P...n, tu m'étonnes après qu'on dit que le C est un langage de goret. Bon, je vois 3 fautes monumentales dans ces 5 pauvres lignes :
- Ta conversion de param en va_list n'est pas du tout portable, j'espère que tu n'auras pas à le tester sur d'autres archie. Sympa pour ceux qui vont maintenir ça après ...
- Utilise au moins vsnprintf, ça coute pas beaucoup plus cher, et tu éviteras un petit buffer overflow (surtout avec une taille de 100 octets). J'espère qu'il n'est pas trop critique ce code (pas d'accès public).
- Tes paramètres passés directement à sprintf, j'espère que tu es sûr de leur contenu, parce que c'est du pain bénit pour l'injection SQL, bref une faille de sécurité monumentale. Clairement, il ne vaut mieux pas qu'il soit public ton code.
[^] # Re: .
Posté par pierthi . En réponse au message tableau comme argument à vsprintf. Évalué à 2.
- Ta conversion de param en va_list n'est pas du tout portable, j'espère que tu n'auras pas à le tester sur d'autres archie. Sympa pour ceux qui vont maintenir ça après ...
- Utilise au moins vsnprintf, ça coute pas beaucoup plus cher, et tu éviteras un petit buffer overflow (surtout avec une taille de 100 octets). J'espère qu'il n'est pas trop critique ce code (pas d'accès public).
- Tes paramètres passés directement à sprintf, j'espère que tu es sûr de leur contenu, parce que c'est du pain bénit pour l'injection SQL, bref une faille de sécurité monumentale. Clairement, il ne vaut mieux pas qu'il soit public ton code.