On Tue, Jun 23, 2009 at 9:45 PM, Pierre Bourdondelroth@gmail.com wrote:
On Wed, Jun 24, 2009 at 03:43, Austin Englishaustinenglish@gmail.com wrote:
On Tue, Jun 23, 2009 at 3:58 AM, Michael Stefaniucmstefani@redhat.de wrote:
Hello Austin,
are you reinventing strcpy?
We do this elsewhere throughout the source. It prevents possible crashes/security vulnerabilities, as well as this warning: arb_program_shader.c: In function ‘shader_arb_get_register_name’: arb_program_shader.c:931: warning: format not a string literal and no format arguments arb_program_shader.c:935: warning: format not a string literal and no format arguments
-- -Austin
I think what Michael meant is that sprintf(a, "%s", b);
is doing exactly the same thing as strcpy(a, b);
in a less efficient way.
-- Pierre "delroth" Bourdon delroth@gmail.com Étudiant à l'EPITA / Student at EPITA
Ah, my mistake.
Point taken, resent patch.