Problèmes avec « Tas tampon » Erreur dans C

Question

-je obtenir l'erreur suivante dans mon C programme:

 
Writing to heap after end of help buffer 

Pouvez-vous me dire ce que je suis absent?

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

Vous n'êtes pas vérifier si malloc et realloc réussi. Plus important encore, realloc peut renvoyer une poignée différente dont vous vous débarrassez.

De plus, vous avez:

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

Sur la plate-forme la plus commune aujourd'hui, sizeof(path) est le plus probable 4 ou peut-être 8, quelle que soit la longueur du tableau de caractères path points.

Answer 2

This ...

sizeof(path) 

... est le même que ...

sizeof(char *) 

... qui est la taille du pointeur (pas la taille de la mémoire tampon qui il pointe vers), il est donc probablement environ 4.

... donc, ce

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

... est un tampon de 4 octets, et ainsi cette ...

strcpy(ans,path); 

... écrase par écriture (écriture après la fin de) ce tampon.

Au lieu de ...

malloc(sizeof(path)); 

... Je pense que vous voulez ...

malloc(strlen(path)+1); 

Answer 3

Vous devez normalement size = strlen(xxx) + 1; pour permettre la terminaison nulle sur la chaîne.

Dans ce cas, je pense que vous avez besoin:

size = strlen(path) - i + 1;