5

我编写了一个简单的 C 程序来转换char为Tokens。一切正常，但我无法理解为什么size变量值会发生变化。

typedef struct _token {
    int val;
} Token;

void parse( char* code, int size, Token** tokens ) {
    int i = 0;
    for (; i < size; i++) {
        tokens[i] = malloc(sizeof(Token));
        tokens[i]->val = code[i];  
    }
}

int execute( char *path ) {
    char* code;
    if ( read_file( path, &code ) != 0 ) {
        return -1;
    }
    int size = strlen(code) - 1;
    printf("BEFORE PARSE: %d\n", size);    // 1st printf
    Token *tokens;
    parse( code, size, &tokens );        
    printf("AFTER PARSE: %d\n", size);     // 2nd printf
    return 0;
}

如果code包含"abcde"，则输出为：

BEFORE PARSE: 5
AFTER PARSE: 142786584

第二个printf在不同的运行中显示不同的值。

请帮忙 ！

PS：我是C菜鸟！

编辑：

int read_file(char* path, char** code) {
    FILE* fp = fopen ( path , "rb" );
    if( !fp ) {
        return -1;
    }

    fseek( fp , 0L , SEEK_END);
    long lSize = ftell( fp );
    rewind( fp );

    /* allocate memory for entire content */
    *code = calloc( 1, lSize+1 );
    if( !*code ) {
        fclose( fp );
        return -1;
    }

    /* copy the file into the buffer */
    if( 1 != fread( *code , lSize, 1 , fp) ) {
        fclose(fp);
        return -1;
    }

    fclose( fp );
    return 0;
}
4

2 回答 2

4

这是因为令牌永远不会被初始化。将其更改为：

Tokens **tokens = malloc(sizeof(Tokens *) * size);

完成后不要忘记释放内存：

for (; i < size; i++) {
    free(tokens[i]);
}

free(tokens);
于 2013-10-16T15:23:24.833 回答
4

您有一个典型的缓冲区溢出案例。

char* code;

分配指向字符的指针（通常为 8 个字节），而不是用于保存文件数据的缓冲区。

与

Token *tokens;

当您写入时tokens，parse您会覆盖堆栈的一部分并size使用它。

为他们分配足够的内存！

char * code = malloc(0x1000);
Token *tokens = malloc(0x100 * sizeof(Token *));

并传递指针，而不是地址：

read_file( path, code );
parse( code, size, tokens );

这是更正的代码：

typedef struct _token {
    int val;
} Token;

void parse( char* code, int size, Token* tokens ) {
    int i = 0;
    for (; i < size; i++) {
            // you already have memory now
        tokens[i]->val = code[i];  
    }
}

int execute( char *path ) {
    char* code =  malloc(0x1000);
    if ( read_file( path, code ) != 0 ) {
        return -1;
    }
    int size = strlen(code) - 1;
    printf("BEFORE PARSE: %d\n", size);    // 1st printf
    Token *tokens = calloc(sizeof(Token), 0x100);
    parse( code, size, tokens );        
    printf("AFTER PARSE: %d\n", size);     // 2nd printf
    return 0;
}
于 2013-10-16T15:23:37.530 回答