[Spice-devel] [PATCH 2/3] Fix "REVERSE_INULL" caught by coverity

Fabiano Fidêncio fidencio at redhat.com
Mon Jul 14 01:12:13 PDT 2014


---
 gtk/channel-cursor.c | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/gtk/channel-cursor.c b/gtk/channel-cursor.c
index fb19536..8d75e95 100644
--- a/gtk/channel-cursor.c
+++ b/gtk/channel-cursor.c
@@ -324,6 +324,9 @@ static display_cursor *set_cursor(SpiceChannel *channel, SpiceCursor *scursor)
 
     size = 4u * hdr->width * hdr->height;
     cursor = spice_malloc(sizeof(*cursor) + size);
+    if (cursor == NULL)
+        return NULL;
+
     cursor->hdr = *hdr;
     cursor->default_cursor = FALSE;
     cursor->refcount = 1;
@@ -389,7 +392,7 @@ static display_cursor *set_cursor(SpiceChannel *channel, SpiceCursor *scursor)
     }
 
 cache_add:
-    if (cursor && (scursor->flags & SPICE_CURSOR_FLAGS_CACHE_ME)) {
+    if (scursor->flags & SPICE_CURSOR_FLAGS_CACHE_ME) {
         cache_add(c->cursors, hdr->unique, display_cursor_ref(cursor));
     }
 
-- 
1.9.3



More information about the Spice-devel mailing list