[libvirt] [PATCHv3 6/6] build: add syntax check for proper flags use

Eric Blake eblake at redhat.com
Fri Jul 15 22:39:03 UTC 2011


On 07/15/2011 09:52 AM, Matthias Bolte wrote:
> 2011/7/15 Eric Blake<eblake at redhat.com>:
>> Enforce the recent flags cleanups - we want to use 'unsigned int flags'
>> in any of our APIs (except where backwards compatibility is important,
>> in the public migration APIs), and that all flags are checked for
>> validity (except when there are stub functions that completely
>> ignore the flags argument).
>>
>> There are a few minor tweaks done here to avoid false positives:
>> signed arguments passed to open() are renamed oflags, and flags
>> arguments that are legitimately ignored are renamed flags_unused.
>>
>
> ACK.

I've pushed this now.  I'll post a v4 of patch 5/6, which picks up some 
additional places where libvirt.c is wrongly filtering flags.

-- 
Eric Blake   eblake at redhat.com    +1-801-349-2682
Libvirt virtualization library http://libvirt.org




More information about the libvir-list mailing list