[libvirt] [PATCH v2] util: set OOM in virCopyLastError if error is not set

[John Ferlan]

On 07/19/2018 04:15 AM, Nikolay Shirokovskiy wrote:
> 
> 
> On 17.07.2018 22:28, John Ferlan wrote:
>>
>>
>>>>> +    } else {
>>>>> +        to->code = VIR_ERR_NO_MEMORY;
>>>>> +        to->domain = VIR_FROM_NONE;
>>>>> +        to->level = VIR_ERR_ERROR;
>>>>
>>>> Should we do a VIR_FREE(to->message); so that nothing that was there
>>>> before somehow remains... I don't think either agent or monitor> "lastError" is reset until Dispose time.
>>>
>>> Won't hurt but probably will not be used by monitor or agent. If thread
>>> error is not allocated message is NULL upon return, after error is allocated we never
>>> hit this OOM branch anymore. Of course hypotetical client can bring @to
>>> with message already set so this a bit future proof. 
>>>
>>> I think then we can leave reset and then set these 3 fields.
>>>
>>> Nikolay
>>>
>>
>> You have commit access and my R-by regardless of whether you add the
>> VIR_FREE or not.  I leave the rest to you ;-)
>>
> 
> Thanx! Pushed with reset added.
> 
> Nikolay
> 

Oh no, you pushed a local .gnulib too!

git log -p
...
commit 1bff5bbe25eb7a7e7a4e0067c4ca7cbc1cb34999
Author: Nikolay Shirokovskiy <nshirokovskiy at virtuozzo.com>
Date:   Mon Jul 2 14:16:52 2018 +0300

...
diff --git a/.gnulib b/.gnulib
index cdbf3d385a..d6397dde2e 160000
--- a/.gnulib
+++ b/.gnulib
@@ -1 +1 @@
-Subproject commit cdbf3d385a32ff904c96f20c26f3470bd8345248
+Subproject commit d6397dde2e127e246e3eeb5254a21f42cac783c8
diff --git a/src/util/virerror.c b/src/util/virerror.c
index f198f27957..5f26d59777 100644
--- a/src/util/virerror.c
+++ b/src/util/virerror.c


...

John