Pregunta Cambio inexplicable en la variable C


He escrito un simple programa en C para convertir char dentro Tokens. Las cosas funcionan bien, pero no puedo entender por qué size el valor variable está cambiando.

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

Si code contiene "abcde", la salida es:

BEFORE PARSE: 5
AFTER PARSE: 142786584

El segundo printf Muestra diferentes valores en diferentes ejecuciones.

Por favor ayuda !

PD: soy un C noob!

EDITAR:

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

5
2017-10-16 15:13


origen


Respuestas:


Usted tiene un caso típico de desbordamiento de búfer.

char* code;

Asigna un puntero al carácter (normalmente 8 bytes), no a un búfer para guardar los datos de su archivo.

Lo mismo con

Token *tokens;

Cuando escribes a tokens en parse sobreescribes parte de tu pila y size con eso.

¡Asigne suficiente memoria para ellos!

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

Y pasa el puntero, no su dirección:

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

Aquí está el código corregido:

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

3
2017-10-16 15:23



Es porque los tokens nunca se inicializan. Cambiarlo a:

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

No olvide liberar la memoria cuando haya terminado con ella:

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

free(tokens);

5
2017-10-16 15:23