char *的分段错误

Question

我在c中执行一个接收文件名的函数，例如“文件名。网络号”，例如：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, ".") 。通过执行以下操作，检查以确保调用成功与否：

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

另一个可能的原因是atoi 。 你在检查red是否包含一个只有数字的空终止字符串？

你在检查以确保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.
• 如果文件的分机号码有字母怎么办？ 即filename.423d2
• 如果文件中没有任何新行怎么办？
• 如果文件中没有任何内容怎么办？
• 如果输入的目录不存在会发生什么？ 或者，输入错误？

重点是：你必须假设你的程序运行完全不可预测的输入。

Answer 2

正如泰勒弗洛雷斯指出的那样，你有一些有趣的记忆分配，但正如你所提到的，你正在搞乱这个，我会假设这就是原因。

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

此外，您将需要使用malloc（如Taylor所提到的）在运行时动态分配和释放内存，而不是这样做，并且通常也会导致问题。