带有char*的分段故障

Question

我在c中做了一个函数，它接收一个文件名，比如"file name. network number"，例如: matriz.0这表示矩阵与一个代码相关联，例如数字1，0是网络号。这意味着我必须生成一个包含1和0的数字代码，并返回一个int。这个函数编码良好，如果我在c中的另一个程序中单独使用它，它可以工作，但是将它添加到主程序中，我注意到char *字符串操作会产生错误。因为如果我注释函数中的3行：

filename = strtok (string, ".");
red = strtok (NULL, ".");
codigored = atoi (red);

分段故障消失。我试图分配内存并初始化所有变量，但错误仍然存在，所以有人可以帮助找到错误吗？非常感谢。

下面是c中函数的代码：

int generasubPalabra(char* nombre) 
{
   char *nombrearchivo=NULL; 
   int codigoarchivo=0;  
   char *red=NULL;                
   int codigored=0;                 

   int subpalabra=1;
   nombrearchivo = (char*) malloc (sizeof(char)*50);  //50 chars

   red = (char*) malloc (sizeof(char)*4); 

   char cadena[strlen(nombre)+1];

   strcpy(cadena, nombre);          



   //ACA ESTA EL ERROR: alguna de las 3 funciones, o las 3, genera una violacion de segmento
   //romper cadena en nombre - nro de red
   //Primera llamada => Primer token (primer elemento hasta el punto, es el nombre del archivo)  

   nombrearchivo = strtok( cadena, "." );       
   printf( "nombre del archivo %s\n", nombrearchivo );

 //Segunda llamada => Segundo token (segundo elemento, desde el punto, es el nro de red)
   red = strtok( NULL, "." );                   
   printf( "numero de red %s\n", red );

   codigored=atoi(red);                 //obtengo el entero correspodiente a la red (cast)
   printf( "codigo de red %d\n", codigored );

   //con la siguiente secuencia se traduce el string contenido en nombrearchivo, al codigo correspondiente para la palabra


   if (strcmp(nombrearchivo,"matriz_incidencia")==0) {
       codigoarchivo=1;
   } 
   else if (strcmp(nombrearchivo,"matriz_brazos_inhibidores")==0) {
        codigoarchivo=2;
   }
   else if (strcmp(nombrearchivo,"matriz_prioridades_red")==0) {
        codigoarchivo=3;
   }
   else if (strcmp(nombrearchivo,"matriz_relacion_disparos_distribuidos")==0) {
        codigoarchivo=4;
   }
   else if (strcmp(nombrearchivo,"matriz_prioridades_disparos_distribuidos")==0) {
        codigoarchivo=5;
   }
   else if (strcmp(nombrearchivo,"vector_marcado_inicial")==0) {
        codigoarchivo=6;
   }
   else if (strcmp(nombrearchivo,"vector_cotas_plazas")==0) {
        codigoarchivo=7;
   }
   else if (strcmp(nombrearchivo,"vector_transiciones_automaticas")==0) {
        codigoarchivo=8;
   }
   else if (strcmp(nombrearchivo,"vector_transiciones_noinformadas")==0) {
        codigoarchivo=9;
   }
   else if (strcmp(nombrearchivo,"vector_mascara_interrupciones")==0) {
        codigoarchivo=10;
   }
   else if (strcmp(nombrearchivo,"vector_cola_entrada")==0) {
        codigoarchivo=11;
   }
   else if (strcmp(nombrearchivo,"vector_consulta_disparo_especifico")==0) {
        codigoarchivo=12;
   }
   else if (strcmp(nombrearchivo,"vector_consulta_plaza")==0) {
        codigoarchivo=13;
   }
   else {
        codigoarchivo=31;  //se retorna el ultimo valor posible (5bits =11111)
        printf("No existe el archivo, verifique ubicacion y permisos \n");
   }

   printf( "codigo de archivo %d\n", codigoarchivo );   

   //se concatena en los 5 bits mas significativos la red, y en los siguientes 5 bits mas significativos el codigo de archivo

   subpalabra = 0x000;                      

   subpalabra+= 0x080*codigored;            //red

   subpalabra+= 0x004*codigoarchivo;        //codigo de archivo

   printf("la subpalabra generada es: %x\n", subpalabra);

return subpalabra;

}

下面是完整的代码：http://www.mediafire.com/download/q782v5cmrcdgl2b/matriz.c

要运行，请使用以下命令

"./matriz path_where_are_the_matrices“示例：./matriz /home/eduardo/hpn/matrices

在该路径上，放置一个包含名为: matriz_brazos_inhibidores.1的数组的文件，文件矩阵示例如下http://www.mediafire.com/download/ur3ucbf8cdmfpmf/matriz_brazos_inhibidores.1

Answer 1

你的代码的主要问题是你绝对没有错误检查。阅读你正在使用的函数，这样你就可以知道它们是否成功地返回了。而且，你的内存使用是完全倒退的。Malloc主要用于事先不知道内存大小的情况。你应该这样使用它：

char nombrearchivo[50]; // instead of malloc (sizeof (char) * 50); 
char * cadena = malloc (strlen(nombre) + 1);

您的错误检查的缺失从这里开始：red = strtok(NULL, ".").Check通过执行以下操作来确保调用是否成功：

if (nombrearchivo){
    red = strtok (NULL, ".");
    codigored = atoi (red);
}

另一个可能的原因是atoi。您是否正在检查red是否包含一个仅由数字组成的以null结尾的字符串？

您是否正在检查以确保nombre不为空？添加以下内容：

if (!nombre) return -1; /* Or just check that you aren't sending a null
                         * pointer in the calling function
                         */

我用一些标准实践重写了你的函数，没有得到任何错误：

int generasubPalabra (char* nombre){

    if (!nombre) return -1;

    char    * nombrearchivo = 0,
            * red = 0,
            * cadena = malloc (strlen (nombre)); /* strlen could lead to
                                                  * a seg fault if it
                                                  * isn't null-terminated.
                                                  */

    int codigoarchivo  = 0,
            codigored  = 0,
            subpalabra = 1;


    if (!cadena) return -1;

    strcpy(cadena, nombre); /* A seg fault wouldn't happen here without 
                             * first happening above, at strlen
                             */

    nombrearchivo = strtok (cadena, ".");

    if (!nombrearchivo) return -1;

    printf ("nombre del archivo %s\n", nombrearchivo);

    red = strtok (0, ".");

    if (!red) return -1;

    printf ("numero de red %s\n", red);

    codigored = atoi (red);
    printf ("codigo de red %d\n", codigored);

         if (!strcmp(nombrearchivo,"matriz_incidencia"))            codigoarchivo = 1;
    else if (!strcmp(nombrearchivo,"matriz_brazos_inhibidores"))    codigoarchivo = 2;
    else if (!strcmp(nombrearchivo,"matriz_prioridades_red"))       codigoarchivo = 3;
    else if (!strcmp(nombrearchivo,"matriz_relacion_disparos_distribuidos"))        codigoarchivo = 4;
    else if (!strcmp(nombrearchivo,"matriz_prioridades_disparos_distribuidos"))     codigoarchivo = 5;
    else if (!strcmp(nombrearchivo,"vector_marcado_inicial"))               codigoarchivo = 6;
    else if (!strcmp(nombrearchivo,"vector_cotas_plazas"))                  codigoarchivo = 7;
    else if (!strcmp(nombrearchivo,"vector_transiciones_automaticas"))      codigoarchivo = 8;
    else if (!strcmp(nombrearchivo,"vector_transiciones_noinformadas"))     codigoarchivo = 9;
    else if (!strcmp(nombrearchivo,"vector_mascara_interrupciones"))        codigoarchivo = 10;
    else if (!strcmp(nombrearchivo,"vector_cola_entrada"))                  codigoarchivo = 11;
    else if (!strcmp(nombrearchivo,"vector_consulta_disparo_especifico"))   codigoarchivo = 12;
    else if (!strcmp(nombrearchivo,"vector_consulta_plaza"))                codigoarchivo = 13;
    else {
            codigoarchivo = 31;  //se retorna el ultimo valor posible (5bits =11111)
            printf ("No existe el archivo, verifique ubicacion y permisos \n");
    }
    printf ("codigo de archivo %d\n", codigoarchivo);

    subpalabra  = 0x000;
    subpalabra += 0x080*codigored;            //red
    subpalabra += 0x004*codigoarchivo;        //codigo de archivo

    printf ("la subpalabra generada es: %x\n", subpalabra);

    return subpalabra;
}

int main (){

    int ret = generasubPalabra ("matriz_brazos_inhibidores.1");

    if (ret < 0)
            printf ("Error occured\n");
    else    printf ("Result = %d\n", ret);

    return 0;
}

更新：

在得到你的程序的全部源代码后，我发现了一些产生分段错误的东西。为了测试这个程序，我这样做：

• 制作了一个包含一个文件的测试目录
• 标记文件 testname.1
• 添加了一些随机单词

以下是我为获得一个干净的运行所做的一些更改：

• 我使用的是我编辑的函数，而不是已经存在的函数(修复了一个错误)
• 通过替换<=，我将其中一个for循环更改为for ( j = 0; j < espacios; j++)

在该循环的末尾，我添加了以下内容：

     /* other code went here */
     printf("el valor a ser escrito long int: %s\n", msg);
  }
  // strtok right here was causing another seg fault
  pch = strtok (NULL, "\n "); //esto es como un reset de pch
  if (!pch){ // your solution to this problem might be different
     fclose (pFile); // just as long as it's taken care of before
     return 0; // the next iteration of the loop
  }

这可能不能完全解决你的程序的问题。我无法添加最有意义的解决方案，因为我并不真正理解您的所有代码是如何组合在一起的(我不太懂西班牙语)。但是，如果你只是在你的代码中添加更多的错误检查，你将能够修复这个问题，或者至少确切地说是什么导致了这个问题。我看到您确实检查了一些函数中的错误，但这还不够。你必须表现得好像什么事情都不会按计划进行。问问你自己：

• 如果目录中有一个没有.符号的文件名怎么办？
• 如果有一个文件没有扩展名怎么办？如果文件的扩展名包含字母，是否类似于filename.
• What？ie filename.423d2
• What如果文件中没有任何新的行？
• 如果文件中没有任何内容怎么办？
• 如果输入的目录不在那里会发生什么？或者，输入错误？

要点是:你必须假设你的程序正在以完全不可预测的输入运行。

Answer 2

正如Taylor Flores指出的，你有一些有趣的内存分配，但正如你提到的那样，我假设这就是为什么。

确保cadena的最后一个索引已初始化。如果不是(f.e.nombre字符串中没有空的终止字符)，抛出一个空字符，否则strtok将尝试访问内存中的空。

此外，您还需要使用malloc ( Taylor也提到过)在运行时动态分配和释放内存，不这样做也会导致问题。