On Tue, Jun 23, 2009 at 9:45 PM, Pierre Bourdon<delr...@gmail.com> wrote:
> On Wed, Jun 24, 2009 at 03:43, Austin English<austinengl...@gmail.com> wrote:
>> On Tue, Jun 23, 2009 at 3:58 AM, Michael Stefaniuc<mstef...@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 <delr...@gmail.com>
> Étudiant à l'EPITA / Student at EPITA
>

Ah, my mistake.

Point taken, resent patch.

-- 
-Austin


Reply via email to