0

以下は私のプログラムです。非常に大きな構造体配列が必要なため、mallocを使用してメモリを割り当て、構造体を作成してから、その構造体配列を関数に渡します。メインプログラムから抜け出せないので、機能は重要ではありません。私の混乱は、コンパイラ（gcc）が、解放しようとするとeach_eventが宣言されていないが、他のどこにも宣言されていないと言っていることです。freeステートメントをコメントアウトするとコンパイルされますが、valgrindは、each_event [i] .timestamp = tim [i]行を実行すると、サイズ4の無効な書き込みがあると言います。その上の行は、下の行を意味する必要があることを知っていたにもかかわらず、valgrindがエラーがあった（gcc -g -O0でコンパイルした）と言っていたため、コメントアウトされています。

#include <stdio.h>
#include <math.h>
#include <stdlib.h>
#include <string.h>

#define event_length 512
#define bits_n_byte 8
#define timestamp_bytes 8

typedef enum type {
    DATA, CLOCK, EXPECTED, SPILL, ALL
} type;

typedef struct event {
char clocbuffer[event_length*bits_n_byte];
char datbuffer[event_length*bits_n_byte];
char expect[event_length];
char spil[event_length];
char clocerror[event_length*bits_n_byte];
char daterror[event_length*bits_n_byte];
long unsigned int timestamp;
} event;

int i, j, k, l, length, nevents;
char **spil, **expect, **dat, **cloc, **clocerror, **daterror;
long unsigned int *tim;
char library[256];
char *runnum, *clocmode, *datmode;

void GetPiece(char*, type, struct event*);
void FindMode(type);
void ErrorPiece(type);

int main(int argc,char **argv) {
if (argc != 2) {fprintf(stderr, "Format: ./program #_patterns\nTry again.\n");}

for (i=0;i<256;i++) library[i]=i;

FILE *IN = NULL;
char *buffer = NULL;

runnum = (char *) malloc(2);
runnum = strncpy(runnum,argv[1],1);
runnum[1] = '\0';
IN=fopen(argv[1], "r");                 /*Open input file.*/

if (IN)
{
    fseek(IN, 0, SEEK_END);                 /*This finds  */
    length  = ftell(IN);                    /*the length  */
    fseek(IN, 0, SEEK_SET);                 /*of the file.*/
    buffer = malloc(length + 2);            /*for buffer.     */
    fread(buffer, 1, length, IN);
    tim = (long unsigned int *) malloc(length + 2*sizeof(long unsigned int));
    fread(tim, sizeof(long unsigned int), length/sizeof(long unsigned int), IN);
    fclose(IN);

    nevents = length/2056;
    struct event* each_event = (struct event *) malloc(nevents*sizeof(struct event));

    for (i=0; i<length/sizeof(unsigned long int); i+=2056/sizeof(unsigned long int))
    {
    tim[i] = __builtin_bswap32 (tim[i]);
    tim[i]-=0x80000000;
    //if (tim[i]<1200000000 || tim[i]>1300000000) fprintf(stderr, "Check timestamp. Either endianness, size of bytes, or size of long ints are different.");
    each_event[i].timestamp = tim[i];
    }

    clocmode = malloc(nevents);
    datmode = malloc(nevents);

    GetPiece(buffer, DATA, each_event);
    GetPiece(buffer, CLOCK, each_event);
    GetPiece(buffer, EXPECTED, each_event);
    GetPiece(buffer, SPILL, each_event);
    GetPiece(buffer, ALL, each_event);
    FindMode(DATA);
    FindMode(CLOCK);
    ErrorPiece(DATA);
    ErrorPiece(CLOCK);
}
else fprintf(stderr,"Error in file naming/opening.\n");   /*error*/
free (buffer);
free (tim);
free (runnum);
free (clocmode);
free (datmode);
free (each_event);
return 0;}
4

1 に答える 1

0

はブロックeach_event内で宣言されているため、試行するときには範囲外です。の最後でそれを解放するか、の最初で他の宣言と一緒に宣言を入れます。ただし、後者を実行すると、初期化されずに使用されている可能性があるという警告が表示される可能性があるため、最初のオプションがおそらく最適です。if { ... }free()if { ... }main()

于 2012-06-20T15:13:29.467 に答える