[libvirt] [PATCH 3/3] conf: add missing error on OOM

[Eric Blake]

On 04/26/2013 10:31 AM, Ján Tomko wrote:
> I removed it in 5c3d5b2 by accident.
> ---
>  src/conf/domain_conf.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/src/conf/domain_conf.c b/src/conf/domain_conf.c
> index 3c6d260..88e3d93 100644
> --- a/src/conf/domain_conf.c
> +++ b/src/conf/domain_conf.c
> @@ -9924,6 +9924,7 @@ virDomainDefMaybeAddController(virDomainDefPtr def,
>  
>      if (VIR_APPEND_ELEMENT(def->controllers, def->ncontrollers, cont) < 0) {
>          VIR_FREE(cont);
> +        virReportOOMError();
>          return -1;

We may be removing it again once the VIR_STRDUP stuff lands to push OOM
reporting into viralloc.c; but for now this patch is right.

ACK; series belongs in 1.0.5.

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130426/84b2a335/attachment-0001.sig>