Discussion :

[MogDeChNord]

Bonjour,

J'ai souvent utilisé ce type de syntaxe:

Code :

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

1
2
3
 
char buffer[BUFF_SIZE] = "titi";
sprintf(buffer, "%s toto", buffer);

Où ici buffer est en même source et destination.

Je n'ai jamais rencontré le moindre pb.

En utilisant l'outil "valgrind" pour détecter des fuites memoire, ce dernier m'avertissait de possible "overlapped copy" dans ce cas là.

Est-ce du zèle de sa part où y-a-t'il un réel risque en fonction des compilateurs ?
(En effet rien n'assure que la source ne sera pas en partie écrasé avant la fin de l'opération)

Merci d'avance pour lumières et explications

[Médinoc]

Je pense qu'il y a un réel risque (on ne sait pas ce que sprintf() fait, sans compter qu'une bibliothèque en version debug peut commencer par vider un buffer ou un truc du genre).

Et puis, dans le cas simple montré ici, un strcat() suffit...

[Jean-Marc.Bourguet]

The sprintf function is equivalent to fprintf, except that the output is written into an array (specified by the argument s) rather than to a stream. A null character is written at the end of the characters written; it is not counted as part of the returned value. If copying takes place between objects that overlap, the behavior is undefined.

[MogDeChNord]

OK vous m'otez d'un doute.

Merci

[Emmanuel Delahaye]

Code :

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

1
2
3
 
char buffer[BUFF_SIZE] = "titi";
sprintf(buffer, "%s toto", buffer);

Où ici buffer est en même source et destination.

Comportement indéterminé.

Je n'ai jamais rencontré le moindre pb.

Possible. Par définition, on ne peut pas prouver l'existence d'un comportement indéterminé a posteriori. C'est au codeur de connaitre et d'appliquer les regles. Des outils d'analyse (comme valgrind ou lint) peuvent aider.

En utilisant l'outil "valgrind" pour détecter des fuites memoire, ce dernier m'avertissait de possible "overlapped copy" dans ce cas là.

Normal, il connait la norme.

7.19.6.6 The sprintf function
Synopsis
1 #include <stdio.h>
int sprintf(char * restrict s,
const char * restrict format, ...);
Description
2 The sprintf function is equivalent to fprintf, except that the output is written into
an array (specified by the argument s) rather than to a stream. A null character is written
at the end of the characters written; it is not counted as part of the returned value. If
copying takes place between objects that overlap, the behavior is undefined.
Returns
3 The sprintf function returns the number of characters written in the array, not
counting the terminating null character, or a neg ative value if an encoding error occurred.

Est-ce du zèle de sa part où y-a-t'il un réel risque en fonction des compilateurs ?
(En effet rien n'assure que la source ne sera pas en partie écrasé avant la fin de l'opération)

Il n'y a aucun zèle. Une regle d'utilisation d'une fonction de la bibliothèque standard du C a été brisée. Tout peut arriver.