[PATCH] Add __builtin_va_arg_pack_len () builtin for {open{,at}{,64},mq_open} fortification

Tom Tromey tromey@redhat.com
Sat Sep 8 21:17:00 GMT 2007


>>>>> "Richard" == Richard Guenther <richard.guenther@gmail.com> writes:

Richard> On 9/8/07, Tom Tromey <tromey@redhat.com> wrote:
>> >>>>> "Jakub" == Jakub Jelinek <jakub@redhat.com> writes:
>> 
Jakub> We have discussed several possibilities, including:
Jakub> 1) hardwiring this kind of checking into gcc with
Jakub> __attribute__((__open_check__ (O_CREAT, 2, __open_2)))
>> 
>> I wanted to point out that this is a situation where plugins would be
>> helpful.  The same applies to the recent alloca warning patch.

Richard> I think the __builtin_va_arg_pack and this natural extension
Richard> of it are generally useful as a tool.  Heh, we even got
Richard> variadic templates now! :P

I didn't mean to imply anything untoward about this patch.  And
certainly it, or any patch, shouldn't be held up waiting for plugins,
since AFAIK we don't even know whether that will ever merge :-(.

Tom



More information about the Gcc-patches mailing list