]> xenbits.xensource.com Git - libvirt.git/commitdiff
virDomainMemoryStats: avoid null dereference
authorEric Blake <eblake@redhat.com>
Mon, 14 Feb 2011 23:20:39 +0000 (16:20 -0700)
committerEric Blake <eblake@redhat.com>
Tue, 15 Feb 2011 16:43:05 +0000 (09:43 -0700)
* src/libvirt.c (virDomainMemoryStats): Check domain before flags.

src/libvirt.c

index 479a9b5aa6c077ff037b417586a35ced713293b6..f65cc24f2ce3c6fcee256bb8e658eae947456b24 100644 (file)
@@ -4411,12 +4411,6 @@ int virDomainMemoryStats (virDomainPtr dom, virDomainMemoryStatPtr stats,
 
     VIR_DOMAIN_DEBUG(dom, "stats=%p, nr_stats=%u", stats, nr_stats);
 
-    if (flags != 0) {
-        virLibDomainError(VIR_ERR_INVALID_ARG,
-                           _("flags must be zero"));
-        goto error;
-    }
-
     virResetLastError();
 
     if (!VIR_IS_CONNECTED_DOMAIN (dom)) {
@@ -4424,6 +4418,12 @@ int virDomainMemoryStats (virDomainPtr dom, virDomainMemoryStatPtr stats,
         virDispatchError(NULL);
         return -1;
     }
+    if (flags != 0) {
+        virLibDomainError(VIR_ERR_INVALID_ARG,
+                           _("flags must be zero"));
+        goto error;
+    }
+
     if (!stats || nr_stats == 0)
         return 0;