char *的分段错误

时间:2013-06-10 00:10:49

标签: c character-encoding char segmentation-fault

我在c中执行一个接收文件名的函数,例如“文件名.net号”,例如: 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

2 个答案:

答案 0 :(得分: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
  • 如果文件中没有任何新行怎么办?
  • 如果文件中没有任何,该怎么办?
  • 如果输入的目录不存在会怎样?或者,输入错误?

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

答案 1 :(得分:1)

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

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

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