[libvirt] [PATCH v2 07/10] conf: Optionally keep domains with invalid XML, but don't allow starting them
John Ferlan
jferlan at redhat.com
Fri Dec 11 11:55:50 UTC 2015
[...]
>>> +
>>> + if (virFileReadAll(configFile, 1024*1024*10, &xmlStr) < 0)
>>
>> Any reason to not use MAX_CONFIG_FILE_SIZE? Cannot imagine this
>> failing, but I mention for consistency
>>
>
> This was a copy-paste from somewhere else, but I will add
> MAX_CONFIG_FILE_SIZE both here and to that copy-paste source as well (if
> I can find it ;)
>
Dang it's in a .c file now... I initially cscope searched on
virFileReadAll... That had some examples... Which is why I mentioned it.
Using 'find src -exec grep "1024.*\*1024.*\*10" {} \;' yields
src/util/virconf.c
src/security/security_apparmor.h
Couldn't say the magic words to cscope to take that string and find
anything though ;-)
John
More information about the libvir-list
mailing list