On 07/06/2011 08:04 AM, Matthias Bolte wrote:
> 2011/7/6 Eric Blake <eblake redhat com>:
>> On 07/06/2011 07:47 AM, Matthias Bolte wrote:
>>> Move stat and mkdir to virFileMakePathHelper.
>>>
>>> Also use the stat result to detect whether the existing path
>>> is a directory and set errno accordingly if it's not.
>>> ---
>>> src/util/util.c | 44 ++++++++++++++------------------------------
>>> 1 files changed, 14 insertions(+), 30 deletions(-)
>>
>> ACK with one nit:
>>
>>> - if ((parent = strdup(path)) == NULL) {
>>> + if ((tmp = strdup(path)) == NULL) {
>>> errno = ENOMEM;
>>
>> strdup() already sets errno to ENOMEM on failure, so this line can be
>> deleted.
>
> Removed that line and pushed, thanks.
Shoot. I just realized that mingw does _not_ set errno on strdup
failure; to get that, we need to import the gnulib module strdup-posix.
I'll amend my pending gnulib update patch for yet another mingw
workaround...
--
Eric Blake eblake redhat com +1-801-349-2682
Libvirt virtualization library http://libvirt.org
Attachment:
signature.asc
Description: OpenPGP digital signature