[libvirt] [PATCH v3 34/34] Introduce syntax-check rule to prefer VIR_STRDUP over strdup
Michal Privoznik
mprivozn at redhat.com
Thu May 9 10:39:18 UTC 2013
On 07.05.2013 05:54, Eric Blake wrote:
> On 05/03/2013 08:53 AM, Michal Privoznik wrote:
>> ---
>> cfg.mk | 8 ++++++++
>> 1 file changed, 8 insertions(+)
>>
>> diff --git a/cfg.mk b/cfg.mk
>> index 1d63bbd..4ed2c3a 100644
>> --- a/cfg.mk
>> +++ b/cfg.mk
>> @@ -387,6 +387,11 @@ sc_prohibit_asprintf:
>> halt='use strdup instead of virAsprintf with "%s"' \
>> $(_sc_search_regexp)
>>
>> +sc_prohibit_strdup:
>> + @prohibit='\<strn?dup\> *\(' \
>> + halt='use VIR_STRUP, not strdup' \
>
> s/VIR_STRUP/VIR_STRDUP/
>
>> + $(_sc_search_regexp)
>> +
>> # Prefer virSetUIDGID.
>> sc_prohibit_setuid:
>> @prohibit='\<set(re)?[ug]id\> *\(' \
>> @@ -864,6 +869,9 @@ exclude_file_name_regexp--sc_prohibit_always_true_header_tests = \
>> exclude_file_name_regexp--sc_prohibit_asprintf = \
>> ^(bootstrap.conf$$|src/util/virstring\.c$$|examples/domain-events/events-c/event-test\.c$$|tests/vircgroupmock\.c$$)
>>
>> +exclude_file_name_regexp--sc_prohibit_strdup = \
>> + ^(docs/|examples/|python/|src/util/virstring\.c$$|tools/virsh\.c$$)
>
> Do we really need to allow strdup() in the python c code? Likewise for
> tools/virsh.c?
Do you mean we should s/strdup/VIR_STRDUP/ within vshStrdup?
And I'll fix python surely.
Michal
More information about the libvir-list
mailing list