Problème de mémoire [C]

Bonjour,

J’ai une fonction f (char* mot, ...) -> int qui parcourt un fichier et effectue des comparaisons entre le mot avec les les lignes du fichier (j’utilise donc beaucoup strcmp dans f). Je ne met pas le code parce-que f utilise d’autres fonctions et donc ça a peu d’intérêt.

Mon problème est le suivant : j’ai une fonction rN (char* word) -> char* qui supprime le caractère \n à la fin d’un mot si il existe. J’ai besoin que rN fasse cela sans modifier le mot en paramètre. J’ai donc ça :

char* rmN (char* word){
    static char* copiedStr = NULL;
    free(copiedStr);
    copiedStr = malloc(strlen(word)+1);
    strcpy(copiedStr, word);
    
    char* newName = copiedStr;
    unsigned long len = strlen(newName)-1;
    
    if (newName[len] == '\n'){
        newName[len] = 0;
    }
    
    return newName;
}

Le problème c’est qu’à la fin je free pas rN. Donc cette fonction n’est pas tip top. Je me suis dis que ça n’était pas grave, pourtant lorsque je fais : f (rN("a\n"), ...) j’obtiens un résultat différent de f ("a", ...) et je ne comprends pas pourquoi ?

Pensez-vous que c’est du à un problème de mémoire ? Si c’est le cas comment améliorer la fonction rN pour éviter cela ?

Merci beaucoup !

Car il faut prendre en compte \0 de fin de chaîne, je suppose ?

J’avais pas fait gaffe au "s’il existe", MB.

Il y a aussi le "free(copiedStr);" qui ne sert à rien, copiedStr étant initialisé à NULL la ligne d’avant, tu ne libérera jamais rien avec cette appel à free().

Peut tu effectivement dire quels résultats tu obtient? Tu devrait normalement obtenir "a\0\0" si il y avait un "\n" et "a\0" si il n’y en avais pas, non?

La variable étant statique, elle n’est initialisée qu’une fois, donc free libère à chaque appel la mémoire allouée par l’appel précédent. C’est peut-être la cause de son problème s’il continue d’utiliser cette mémoire qui a été libérée.

Désolé pour le retard j’ai eu des problèmes de connexions.

Merci beaucoup pour vos réponses.

Vu que manifestement tu ré-initialises la variable copiedStr à chaque call à la fonction, pourquoi diable en faire une variable statique ? (Et contrôle le retour de ton appel à malloc).

Et bien effectivement le problème venait de la variable en static . En enlevant le static ma fonction marche bien. Au début je mettais la variable en static car ça me permettait de faire : free(rmN(""));.

Tu as consciences @InaDeepThink que ta fonction, bien qu’elle fonctionne (🤔), ne permet d’utiliser 2 fois le résultat en même temps ?

Non je n’avais pas remarqué ce comportement et c’est justement pour ça que j’obtenais deux résultats différents du coup.

Il y a aussi un problème si on lui passe une chaîne de caractère vide et comme ça été dit avant la vérification du retour de malloc. Au passage, tu devrais utiliser strcpy_s et gérer les cas où on te passe une valeur NULL.

Oui, mais normalement c’est bon car je m’assure que mes chaînes de caractères ne sont pas vides dans f.

Du coup ta fonction doit ressemblé à ça :

char* rmN (char* word){
    char* copiedStr = NULL;
    unsigned long len = strlen(word);
    copiedStr = malloc(len+1);
    if( copiedStr == NULL )
       return NULL;
    strcpy(copiedStr, word);
    
    
    if (len > 0 && newName[len - 1] == '\n'){
        newName[len - 1] = 0;
    }
    
    return newName;
}

Attention du coup à ne pas oublier de libérer le retour de cette fonction.