2012-02-24 2 views
5

Per la vita di me non posso capire perché sto ottenendo un invalid read size of 1 per questo frammento di codice, sono abbastanza sicuro che la sua ha qualcosa a che fare con me abusando del char *url pointer ...Valgrind non valido leggere dimensioni di 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;  
} 

l'errore valgrind sto ottenendo è:

==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 

Can qualcuno 'punto' di me nella giusta direzione?

+0

Alloca memoria per url, quindi imposta un puntatore nullo. (url = '\ 0';) Volevi dire, forse, per impostare url [0] = '\ 0'; ? – Joe

+0

Forse vuoi semplicemente usare strncpy? – Cascabel

+0

Hai anche perdite di memoria con le tue prime due chiamate 'malloc', che tra l'altro sono completamente false, perché dovresti riservare' sizeof' alla dimensione di un puntatore 'char' * *? Anche il terzo 'malloc' mostra segni di potenziale confusione: (1) non eseguire il ritorno di' malloc', questo nasconde solo bug (2) 'sizeof (char)' è '1' per definizione, il numero che passi a 'malloc' è il numero di' char' che si desidera assegnare. –

risposta

8

Qui

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

si perde immediatamente la memoria malloced impostando url-NULL. Notare che '\0' è 0, che è una costante di puntatore nullo. E quindi si prova a strncat qualcosa in una posizione di memoria non valida.

Probabilmente significava per impostare

*url = '\0'; 

lì.

+0

Questo porta quindi a strncat a fare un salto condizionato su un valore non intenzionale sebbene non lo sia? – DMcB1888

+0

Beh, in realtà mi aspetterei un segfault quando si chiama 'strncat (NULL, something, n)', ma valgrind potrebbe riportarlo, se preferisce. –