Fonction majuscule

oui, il doit terminer

result

par le caractère

'\0'

, mais il a aussi un dépassement de tampon mémoire, car il alloue

result = malloc((strlen(chaine) - 1) * sizeof(char));

(soit une taille inférieure d'un char à la longueur de la chaîne sans compter le caractère de fin ... donc trop petite de 2 char), alors qu'il devrait allouer

result = malloc((strlen(chaine) + 1));

(la taille de la chaîne avec un char additionnel pour le caractère de fin).

A noter, qu'en C

sizeof(char)

est garantit de valoir 1, il est donc inutile.

char * strToUpper(char * chaine) {
    char * result = NULL;
    int i;

    result = malloc((strlen(chaine) + 1));
    for (i = 0; i < strlen(chaine); i++) {
        result[i] = toupper(chaine[i]);
    }
    result[i] = '\0';

    return result;
}

On peut optimiser le code, aussi, en évitant de mettre

strlen(chaine)

dans la boucle for et en ne l'exécutant qu'une fois au début de la fonction.

Enfin, cela serait bien de tester que malloc a bien alloué la mémoire nécessaire et de décider de ce fait la fonction si ce n'est pas le cas.

Dal

Effectivement, j'avais complétement oublié le

\0

.
Merci !