Ayuda con segmentation fault

[puppetz]

Buenas, soy nuevo por aca y ando necesitando ayuda para un programa.
La idea del programa es que remplace palabras "prohibidas" ingresadas por usuario en texto que se ingresa como argumento de la funcion. El codigo va algo asi:

Código (c) [Seleccionar] Expandir

#include "my_string.h"

int moderar(char a[], int j, char b[])
{
 int k=0,l=0,m=0,w=0,cant=0;

 for(k=1; k<j; k++)
 {
for(l=0;l<TAM;l++)
{
 m=my_strequ( &a[k], &b[l]);

 if (m=1)
 {
w=my_strlen((&a[l]));
int t=0;

for(t=0; t<w; t++)
{
 ((a[l+t]))='x';
}
 }
}
printf("que ***** cuenta: %d",w);
cant=w;
 }

 return cant;
}

int main (int argc, char* argv[])
{
 char* palprob[TAM];
 int i=0, nuevasx, q=argc;

 printf("\nIngresar hasta 10 palabras prohibidas a moderar (pulse ENTER si desea finalizar antes de llegar a 10 palabras)\n");

 do
 {
palprob[i]= (char*)calloc(1, sizeof(char*));
gets(palprob[i]);
i++;
 } while((i<10)&&(*(palprob[i-1])!='\0'));

nuevasx= moderar(argv, argc, palprob);

printf("\nMensaje moderado: \n");

for(i=0; i<argc-1; i++)
  printf("%s ", argv[i+1]);

printf("\n\nCantidad de caracteres modificados: %d \n",nuevasx);

 return 0;
}

Aclaro, en my_string.h se encuentran funciones propias de strequ y strlen y estan funcionando correctamente (ya las probe con otro programa).

[Eternal Idol]

Es tu codigo y deberias depurarlo. La linea 13 esta mal seguro; no se como te compila el resto, los parametros de moderar deben ser dobles punteros a char.

Ej.:
int moderar(char **a, int j, char *b[])

Linea 9 for(l=0;l<TAM;l++)

¿Y si ingreso menos de 10 - TAM asumo - palabras que pasa? Segmentation fault. Tenes que comprobar que haya una cadena valida (deberias inicializar todos los elementos a NULL) o pasar el numero de cadenas ingresadas ... en fin, revisalo todo de nuevo.

[engel lex]

Efectivamente como dice eternal, te ayudo "=" es asignación y "==" es comparación

[puppetz]

Se que es mi codigo, y uso el kdbg para depurarlo pero no se como corregirlo, ese es el problema.

Entonces tengo que completar toda la cadena de 10 elementos con ='NULL' ?
Voy a probar las correciones y les digo como me fue

[puppetz]

Bueno gente, la clave era como dijo idol, los dobles punteros a char y lo del = y el == me lo comi por distraido  

Hasta ahora tengo esto de codigo, pero no logra cumplir el objetivo la funcion moderar ya que no logro hacer que las cadenas sean identicas.
Mi funcion de my_strequ funciona en otros programas pero no en este  

Código [Seleccionar] Expandir


#include "my_string.h"

int moderar(char** a, int j, char** b, int c)
{
 int i,k,l,m,w=0,cant=0;
 
 for(k=1; k<j; k++)
 {
for(l=0;l<c;l++)
{
 m=my_strequ(a[k],b[l]);
 
 if (m==1)
 {
w=my_strlen((a[k]));
int t=0;

for(t=0; t<w; t++)
{
 a[k][t]='x';
}
 }
}
cant+=w;
 }

 return cant;
}

int main (int argc, char* argv[])
{
 char* palprob[TAM]; char auxi[30];
 int i=0, nuevasx=0;
 int tama, cantpalprob=0;
 
 
 printf("\nIngresar hasta 10 palabras prohibidas a moderar (pulse ENTER si desea finalizar antes de llegar a 10 palabras):\n");
 
for (i=0;i<10 && *auxi!='\0';i++)
 {
   gets(auxi);
   cantpalprob=i;
   if (*auxi!='\0')
   {
     tama=my_strlen(auxi);
     palprob[i] = (char*) calloc (tama+1,sizeof(char));
     my_strcpy(palprob[i],auxi);
   }
 }
 
 printf("Palabras prohibidas: ");

 for(i=0; i<cantpalprob; i++)
  printf("\n\t\t\t%s ", palprob[i]);

 nuevasx+= moderar(argv, argc, palprob, cantpalprob);

printf("\nMensaje moderado: \n\t\t\t");

for(i=0; i<argc-1; i++)
  printf("%s ", argv[i+1]);

printf("\n\nCantidad de caracteres modificados: %d \n\n",nuevasx);
 
 return 0;
}


[Eternal Idol]

No, el problema esta en alguna de esas funciones my_, simplemente las reemplace por las standard y el programa funciono. En el bucle estas accediendo a un valor indefinido del primer caracter de auxi en la primera iteracion ...

[puppetz]

El programa funciona a la perfeccion! Gracias a Eternal y a Engel por su ayuda.
Al final el problema estaba en my_strequ.

Saludos a todos

[Eternal Idol]

De nadas