[libvirt] [PATCH] Enforce attribute check of the virRegister functions
Pavel Hrdina
phrdina at redhat.com
Mon Mar 17 17:03:32 UTC 2014
On 17.3.2014 17:47, Daniel P. Berrange wrote:
> On Mon, Mar 17, 2014 at 05:33:58PM +0100, Pavel Hrdina wrote:
>> Signed-off-by: Pavel Hrdina <phrdina at redhat.com>
>> ---
>> src/driver.h | 16 ++++++++--------
>> 1 file changed, 8 insertions(+), 8 deletions(-)
>>
>> diff --git a/src/driver.h b/src/driver.h
>> index fbfaac4..d106d59 100644
>> --- a/src/driver.h
>> +++ b/src/driver.h
>> @@ -2132,15 +2132,15 @@ struct _virNWFilterDriver {
>> * TODO: also need ways to (des)activate a given driver
>> * lookup based on the URI given in a virConnectOpen(ReadOnly)
>> */
>> -int virRegisterDriver(virDriverPtr);
>> -int virRegisterNetworkDriver(virNetworkDriverPtr);
>> -int virRegisterInterfaceDriver(virInterfaceDriverPtr);
>> -int virRegisterStorageDriver(virStorageDriverPtr);
>> -int virRegisterNodeDeviceDriver(virNodeDeviceDriverPtr);
>> -int virRegisterSecretDriver(virSecretDriverPtr);
>> -int virRegisterNWFilterDriver(virNWFilterDriverPtr);
>> +int virRegisterDriver(virDriverPtr) ATTRIBUTE_RETURN_CHECK;
>> +int virRegisterNetworkDriver(virNetworkDriverPtr) ATTRIBUTE_RETURN_CHECK;
>> +int virRegisterInterfaceDriver(virInterfaceDriverPtr) ATTRIBUTE_RETURN_CHECK;
>> +int virRegisterStorageDriver(virStorageDriverPtr) ATTRIBUTE_RETURN_CHECK;
>> +int virRegisterNodeDeviceDriver(virNodeDeviceDriverPtr) ATTRIBUTE_RETURN_CHECK;
>> +int virRegisterSecretDriver(virSecretDriverPtr) ATTRIBUTE_RETURN_CHECK;
>> +int virRegisterNWFilterDriver(virNWFilterDriverPtr) ATTRIBUTE_RETURN_CHECK;
>> # ifdef WITH_LIBVIRTD
>> -int virRegisterStateDriver(virStateDriverPtr);
>> +int virRegisterStateDriver(virStateDriverPtr) ATTRIBUTE_RETURN_CHECK;
>> # endif
>> void virDriverModuleInitialize(const char *defmoddir);
>> void *virDriverLoadModule(const char *name);
>
> ACK
>
> Regards,
> Daniel
>
Thanks, pushed.
Pavel
More information about the libvir-list
mailing list