2012-02-24 14 views
5

Para la vida de mí no puedo entender por qué estoy recibiendo un invalid read size of 1 de este fragmento de código, estoy bastante seguro de que su tiene algo que ver conmigo abusar de la char *url pointer ...Valgrind no válida leer tamaño de 1

char *extractURL(char request[]) 
{ 
char *space = malloc(sizeof(char *)); 
space = strchr(request, ' ')+1; 
char *hostend = malloc(sizeof(char *)); 
hostend = strchr(request, '\r'); 
int length = hostend - space; 
if (length > 0) 
{ 
    printf("Mallocing %d bytes for url\n.", length+1); 
    char *url = (char *)malloc((length+1)*sizeof(char)); 
    url = '\0'; 
    strncat(url, space, length); 
    return url; 
} 
//else we have hit an error so return NULL 
return NULL;  
} 

el error valgrind que estoy recibiendo es:

==4156== Invalid read of size 1 

==4156== at 0x4007518: strncat (mc_replace_strmem.c:206) 

==4156== by 0x8048D25: extractURL() 

==4156== by 0x8048E59: processRequest() 

==4156== by 0x8049881: main() 

==4156== Address 0x0 is not stack'd, malloc'd or (recently) free'd 

Puede alguien 'punto' a mí en la dirección correcta?

+0

Asigna memoria para url, luego establézcala en un puntero nulo. (url = '\ 0';) ¿Quiso decir, tal vez, para establecer url [0] = '\ 0'; ? – Joe

+0

¿Quizás solo quiera usar strncpy? – Cascabel

+0

También tiene fugas de memoria con sus dos primeras llamadas 'malloc', que por cierto son completamente falsas, ¿por qué reservar' sizeof' del tamaño de un puntero 'char' *? Su tercer 'malloc' también muestra signos de posible confusión: (1) no arrojar el retorno de' malloc', esto solo oculta errores (2) 'sizeof (char)' es '1' por definición, el número que pasa a 'malloc' es el número de' char' que desea asignar. –

Respuesta

8

Aquí

char *url = malloc((length+1)*sizeof(char)); 
url = '\0'; 
strncat(url, space, length); 

pierden de forma inmediata la memoria malloced estableciendo url-NULL. Tenga en cuenta que '\0' es 0, que es una constante de puntero nulo. Y luego intenta strncat algo en una ubicación de memoria no válida.

Probablemente significaba para establecer

*url = '\0'; 

allí.

+0

Eso lleva a que strncat haga un salto condicional en un valor sin pintar, ¿no es así? – DMcB1888

+0

Bueno, realmente esperaría una segfault al llamar a 'strncat (NULL, something, n)', pero valgrind podría informar eso, si así lo prefiere. –

Cuestiones relacionadas