Проблемы с Error «буфера для» в C

Question

я получаю следующее сообщение об ошибке в моей программе C:

 
Writing to heap after end of help buffer 

Можете ли вы сказать мне, что я не хватает?

char * path_delimiter(char * path) 
{ 
    int i = 0, index = 0, size = 0, length = (int)strlen(path); 
    char *tmp, *ans; 

    for(; i < length; i++) { 
     if(path[i] == PATH_DELIM[0]) { 
      break; 
     } 
    } 
    i++; 
    size = (int)strlen(path) - i; 
    ans = (char*)malloc(sizeof(path)); 
    tmp = (char*)malloc(size); 
    strcpy(ans,path); 
    ans[i-1] = END_ARRAY; 

    if(size > 0) 
    { 
     strcpy(tmp,&path[i]); 
     realloc(path,size); 
     strcpy(path,tmp); 
    } 
    else 
    { 
     strcpy(path,ans); 
    } 
free(tmp); 

return ans; 
} 

Answer 1

Вы не проверять, если malloc и realloc удалось. Что еще более важно, realloc может вернуть другой дескриптор, который вы отбрасываете.

Кроме того, у вас есть:

ans = malloc(sizeof(path)); 
... 
strcpy(ans, path); 

На самой распространенной платформе сегодня, sizeof(path), скорее всего, 4 или 8, может быть, независимо от длины массива символов path указывает.

Answer 2

Это ...

sizeof(path) 

... так же, как ...

sizeof(char *) 

... что размер указателя (не размер буфера, который это указывает на), так что, вероятно, около 4.

так это ...

ans= (char*)malloc(sizeof(path)); 

... это 4-байтный буфер, и поэтому этот ...

strcpy(ans,path); 

... это перезапись (запись в конце прошлого) этого буфера.

Вместо того, чтобы ...

malloc(sizeof(path)); 

... Я думаю, что вы хотите ...

malloc(strlen(path)+1); 

Answer 3

Вы обычно нужно size = strlen(xxx) + 1;, чтобы обеспечить нулевой терминатор на струне.

В этом случае, я думаю, что вам нужно:

size = strlen(path) - i + 1;