Discussion :

[Fahya]

Bonjour,
Voici ce que contient ma fonction main :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
char buf[1024] = "put 394c8a052d ::1";
  char get_or_put[3];
  int j;
  for (j = 0; j < 3; j++) get_or_put[j] = buf[j];
  printf("%s\n", get_or_put);

Et j'obiens comme résultat : "puput 394c8a052d ::1", je ne comprend vraiment pas pourquoi j'ai ce résultat.

Bonne journée et merci d'avance.

[Matt_Houston]

Comment pourrait-on déterminer la fin de la chaîne que tu cherches à afficher ?

[Fahya]

Comment pourrait-on déterminer la fin de la chaîne que tu cherches à afficher ?

J'essaye simplement de copier les trois premiers caractères de buf dans get_or_put. L'affichage est censé me donner 'put'

[Matt_Houston]

Alors je reformule ma question initiale : comment printf pourrait-il déterminer que la chaîne que tu lui présentes contient trois octets, et pas deux ou six ?

[Fahya]

Alors je reformule ma question initiale : comment printf pourrait-il déterminer que la chaîne que tu lui présentes contient trois octets, et pas deux ou six ?

D'accord, étant donné que sa taille est de trois je pense qu'il doit en afficher trois non ?

[ternel]

Comment détermine-t-il que la taille est trois?
Sachant qu'un argument de fonction n'est jamais un tableau, mais un pointeur.

Qu'est-ce qu'une chaine de caractère en C?

[Sve@r]

D'accord, étant donné que sa taille est de trois je pense qu'il doit en afficher trois non ?

Bonjour

Ben non. Le C n'est pas assez évolué pour faire ce rapprochement. D'ailleurs ce serait assez malheureux qui'il bosse de cette façon parce qu'alors l'affichage de "buf" afficherait 1024 caractères (tu aurais pu t'en rendre compte tout seul !!!)
Pour afficher une chaine, il part du début de la chaine et boucle jusqu'à ce qu'il trouve la "fin de la chaine". A toi de reprendre ton cours sur "ce qu'est une chaine de caractères en C" (cf post de ternel) pour comprendre comment il trouve cette "fin de chaine"...

[Fahya]

Je sais ce qu'est une fin de chaîne. Mais j'ai vu dans tous mes cours que si on fait printf("%s\n", chaine); cela affichait la chaine. Je n'ai pas besoin d'afficher buf justement, je veux juste récupérer ses trois premiers caractères en les copiant dans une chaîne. J'ai tout de même suivi vos conseils en ajoutant dans ma fonction "get_or_put[3] = '\0'", mais cela ne marche toujours pas.

[Kannagi]

je veux juste récupérer ses trois premiers caractères en les copiant dans une chaîne

Alors sprintf est la solution : sprintf(copy,"%.3s", chaine);
D’ailleurs si ton but est seulement d'afficher 3 caractère alors printf("%.3s\n", chaine);

Par contre :

J'ai tout de même suivi vos conseils en ajoutant dans ma fonction "get_or_put[3] = '\0'", mais cela ne marche toujours pas.

Il faudrait faire une révision des tableaux en C !
Quand tu déclare get_or_put[3] , cela veut dire qu'il existe : get_or_put[0] , get_or_put[1] , get_or_put[2] et donc écrire sur get_or_put[3] c'est un buffer overflow donc oui normal que cela ne marche pas !

[Sve@r]

Je sais ce qu'est une fin de chaîne.

Certainement. Mais tu n'as pas fait le rapprochement entre "ce que tu sais" et ton code et c'est ce qu'on te reproche. Autrement dit en plus condensé: si tu le sais, pourquoi tu ne l'as pas mis ?

Mais j'ai vu dans tous mes cours que si on fait printf("%s\n", chaine); cela affichait la chaine.

Effectivement, quand tu as une vraie chaine, contenant toutes les caractéristiques appropriées d'une chaine, alors printf("%s\n", chaine) ça l'affiche proprement. Malheureusement "get_or_put" ce n'est pas une chaine donc printf("%s\n", get_or_put) ça ne te l'affiche pas proprement.
Ainsi, le résultat de ton code correspond exactement à ce qui est indiqué dans tous tes cours (ou plutôt à ce qu'on peut en déduire parce que, en logique mathématique, si A implique B, alors non(B) implique non(A) => si j'ai une chaine alors printf("%s\n", chaine) l'affiche proprement donc si printf("%s\n", get_or_put) ne l'affiche pas proprement c'est que c'est pas une chaine)...

...parce qu'alors l'affichage de "buf" afficherait 1024 caractères

Je n'ai pas besoin d'afficher buf justement,

Je ne te parlais pas réellement d'afficher buf (t'as pas remarqué le conditionnel de "afficherait" ?). Je te donnais cet exemple pour te montrer que ton hypothèse "si la taille de ma variable est de trois ça affiche trois caractères" ne tenait pas la route parce que pour buf la taille est de 1024 donc si on affichait buf ça afficherait (là aussi un conditionnel) 1024c !!!

je veux juste récupérer ses trois premiers caractères en les copiant dans une chaîne.

Oui on a bien compris. Le problème, c'est que tu ne les copies pas dans une "chaine" mais dans un simple "tableau" !!!!!!!!!

J'ai tout de même suivi vos conseils en ajoutant dans ma fonction "get_or_put[3] = '\0'"

Là ça commence à venir. Mais si tu y réfléchis, tu devrais te rendre compte que "nos" conseils suivent la logique. Si une chaine c'est un tableau contenant impérativement quelque part un '\0' ; mais que rien dans ton code ne prouve qu'un des éléments du tableau "get_or_put" vaut '\0', alors c'est qu'il n'y est pas et qu'il faut le mettre.

mais cela ne marche toujours pas.

Voir post de Kannagi. 'p', 'u', 't' et '\0' ne peuvent pas tenir dans un char[3] et ça aussi tu aurais pu t'en rendre compte tout seul (au pire compter avec tes doigts...)

[ternel]

Comment détermine-t-il que la taille est trois?
Sachant qu'un argument de fonction n'est jamais un tableau, mais un pointeur.

Qu'est-ce qu'une chaine de caractère en C?

La réponse à la première question est: pas.
Une adresse ne désigne pas un tableau, mais un de ses éléments. On peut toujours calculer des adresses d'éléments du même type, et voisins.
Par exemple, si p est un pointeur (un argument de fonction) vers un type T, alors (p+1) et (p-4) sont aussi des pointeurs vers des T.
Mais rien ne peut dire à la fonction qu'il y a bien un T à cet endroit là.
Du coup, la fonction peut manipuler la mémoire à cet endroit, aveuglément.
C'est ce défaut qui est la cause/raison des failles de sécurité de type "buffers overflow" (et gets() a été retirée du langage pour cette raison précise).

Il n'y a que trois solutions possibles:

• Prendre la taille en argument supplémentaire.
• Utiliser une valeur sentinelle. La mémoire pointée est valide dans une direction donnée (généralement, ++), tant qu'on ne tombe pas dessus.
• Avoir la taille intégrée dans la zone mémoire (en premier élément, peut-être, ou dans la structure donnée en argument)

Les fonctions manipulant des chaines parcourent la mémoire jusqu'à tomber sur un '\0'.
Cela signifie que pour pouvoir être utilisées, il faut s'assurer qu'un '\0' est en place.
Dans ton cas, ce n'est pas le cas.

Je peux même dire que ton système est compilé avec une pile dont le haut est dirigé vers les adresses basses.
Dans la pile, le second tableau est placé au dessus du premier. C'est à dire aux adresses juste inférieures, et jointives.
Comme tu n'as pas de '\0' à la fin du petit tableau, la fonction continue à parcourir. Manque de chance pour toi, il tombe sur une autre chaine "valide".
Avec un peu de chance, tu affichais de la mémoire arbitraire.

[Sve@r]

Et j'obiens comme résultat : "puput 394c8a052d ::1"

Dans la pile, le second tableau est placé au dessus du premier. C'est à dire aux adresses juste inférieures, et jointives.

J'y avais pensé aussi. Je me suis même dit qu'avec ce code il était totalement impossible d'avoir comme résultat "puput 394c8a052d ::1" et que le résultat réel était probablement "putput 394c8a052d ::1"...

Avec un peu de chance, tu affichais de la mémoire arbitraire.

Personnellement je pense plutôt que ça c'est de la malchance et que la chance c'est d'avoir un "segfault" montrant ainsi qu'il y a un gros souci ; plutôt que d'avoir un programme qui ne plante pas et qui, tout en montrant éventuellement le bug, continue à tourner et à produire des résultats faux...