]> xenbits.xensource.com Git - libvirt.git/commitdiff
conf: Need to check for glisten before accessing
authorJohn Ferlan <jferlan@redhat.com>
Mon, 18 Jul 2016 18:30:59 +0000 (14:30 -0400)
committerJohn Ferlan <jferlan@redhat.com>
Tue, 19 Jul 2016 11:51:10 +0000 (07:51 -0400)
When formatting the graphics data for TYPE_SPICE, check if the glisten
is NULL before blindly referencing

Found by Coverity

Signed-off-by: John Ferlan <jferlan@redhat.com>
src/conf/domain_conf.c

index 15fd8cab0d871af012142dfc8eb07ff77ff8a3b9..6dfcf81e9e058a3665ba2807a7c396a7c7bc81db 100644 (file)
@@ -22182,6 +22182,12 @@ virDomainGraphicsDefFormat(virBufferPtr buf,
         break;
 
     case VIR_DOMAIN_GRAPHICS_TYPE_SPICE:
+        if (!glisten) {
+            virReportError(VIR_ERR_INTERNAL_ERROR, "%s",
+                           _("missing listen element for spice graphics"));
+            return -1;
+        }
+
         switch (glisten->type) {
         case VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_ADDRESS:
         case VIR_DOMAIN_GRAPHICS_LISTEN_TYPE_NETWORK: