2012-02-24 1 views
5

Для жизни меня я не могу понять, почему я получаю invalid read size of 1 для этого фрагмента кода, я уверен, что его что-то делать со мной злоупотребляя char *url pointer ...Valgrind Invalid чтение размера 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;  
} 

ошибка Valgrind Я получаю:

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

Может кто-то «точка» меня в правильном направлении?

+0

Вы выделяете память для URL-адреса, а затем устанавливаете его на нулевой указатель. (url = '\ 0';) Вы имели в виду, возможно, установить url [0] = '\ 0'; ? – Joe

+0

Возможно, вы просто хотите использовать strncpy? – Cascabel

+0

У вас также есть утечки памяти с вашими первыми двумя вызовами 'malloc', которые, кстати, сами по себе являются полностью фиктивными, почему вы зарезервируете' sizeof' размер указателя 'char' * *? Ваш третий 'malloc' также показывает признаки потенциальной путаницы: (1) не отбрасывает возврат' malloc', это только скрывает ошибки (2) 'sizeof (char)' is '1' по определению, номер, который вы передаете to 'malloc' - это номер' char', который вы хотите выделить. –

ответ

8

Здесь

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

вы сразу потеряете malloced память, установив url в NULL. Обратите внимание, что '\0' равно 0, что является нулевой константой указателя. И затем вы пытаетесь сделать strncat что-то недопустимое расположение памяти.

вероятно Вы имели в виду установить

*url = '\0'; 

там.

+0

Это приводит к тому, что strncat выполняет условный переход на неинтеллизированное значение, хотя это не так? – DMcB1888

+0

Ну, я действительно ожидал бы segfault при вызове 'strncat (NULL, something, n)', но valgrind может сообщить об этом, если он предпочитает. –