0

たとえば、「ファイル名.ネットワーク番号」などのファイル名を受け取る関数を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 ファイル マトリックスの例は次のとおり です

4

2 に答える 2

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);
}

もう 1 つの考えられる原因は次のとおりですatoi。redヌルで終了する数字だけの文字列が含まれていることを確認していますか?

nombreそれがnullでないことを確認していますか？これを追加：

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;
}

アップデート：

プログラムのソース全体を取得した後、セグメンテーション違反を引き起こしているいくつかのことがわかりました。プログラムをテストするために、私はこれを行いました：

  • その中に1つのファイルを含むテストディレクトリを作成しました
  • ファイルに testname.1 というラベルを付けました
  • それにいくつかのランダムな単語を追加しました

きれいに実行するために行った変更の一部を次に示します。

  • 既にある関数の代わりに編集した関数を使用しました (1 つの障害を修正)
  • forループの 1 つを に変更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
  • ファイルに新しい行がない場合はどうなりますか?
  • ファイルに何もない場合はどうなりますか?
  • 入力したディレクトリが存在しない場合はどうなりますか? それとも、タイプミスですか？

つまり、プログラムが完全に予測不可能な入力で実行されていると想定する必要があります。

于 2013-06-10T00:19:06.960 に答える
1

テイラー・フローレスが指摘したように、そこにはいくつかの興味深いメモリ割り当てがありますが、あなたがそれをいじっていると述べたように、それが理由だと思います。

カデナの最後のインデックスが初期化されていることを確認してください。そうでない場合 (nombre 文字列に null 終端文字がない場合)、null 文字をスローするか、strtok がメモリ内の void にアクセスしようとします。

また、実行時にメモリを動的に割り当てて解放するには、malloc（テイラーも言及）を使用する必要があります。これを行わないと、問題が発生する可能性があり、多くの場合、問題が発生します。

于 2013-06-10T00:27:25.527 に答える