Passant char pour casser des tas de tas

Question

Psudo code ci-dessous, mais est-ce que quelqu'un a une idée pourquoi ce serait casser le tas? La fonction urlencode est une fonction de bibliothèque standard téléchargée ailleurs, et semble fonctionner comme prévu. Dans le code actuel, j'utilise des tableaux dynamiques de taille char, donc la raison de l'exigence de malloc dans main.

/* Returns a url-encoded version of str */ 
/* IMPORTANT: be sure to free() the returned string after use */ 
char *urlencode(char *str) { 
    //char *pstr = str, *buf = malloc(strlen(str) * 3 + 1), *pbuf = buf; 
    char *pstr = str, *buf = malloc(strlen(str) * 3 + 1), *pbuf = buf; 
    while (*pstr) { 
    if (isalnum(*pstr) || *pstr == '-' || *pstr == '_' || *pstr == '.' || *pstr == '~') 
     *pbuf++ = *pstr; 
    else if (*pstr == ' ') 
     *pbuf++ = '+'; 
    else 
     *pbuf++ = '%', *pbuf++ = to_hex(*pstr >> 4), *pbuf++ = to_hex(*pstr & 15); 
    pstr++; 
    } 
    *pbuf = '\0'; 
    return buf; 
} 

int testFunction(char *str) { 
    char *tmpstr; 
    tmpstr = urlencode(str); 
    // Now we do a bunch of stuff 
    // that doesn't use str 
    free(tmpstr); 
    return 0; 
    // At the end of the function, 
    // the debugger shows str equal 
    // to "This is a test" 
} 

int main() { 
    char *str = NULL; 
    str = malloc(100); 
    strcpy(str, "This is a test"); 
    testFunction(str); 
    free(str); // Debugger shows correct value for str, but "free" breaks the heap 
    return 0; 
} 

Merci.

Answer 1

Je suppose que str a déjà été libéré par free(tmpstr); - s'il vous plaît jeter un oeil sur le comportement de la fonction urlencode. Il semble qu'il ne génère pas de nouvelle chaîne comme valeur de retour, mais renvoie la chaîne d'entrée (modifiée).

Answer 2

le problème s'est avéré être un problème avec le calcul de la taille pour le malloc initial de str étant exécuté comme 0. Merci pour les commentaires, malheureusement aucun moyen de vraiment marquer une réponse en tant que commentaires.

Si c'est une manière incorrecte de fermer ceci, s'il vous plaît faites le moi savoir.