adjust assertion in alloc_heap_pages()
authorJan Beulich <jbeulich@suse.com>
Thu, 23 Apr 2015 11:08:40 +0000 (13:08 +0200)
committerJan Beulich <jbeulich@suse.com>
Thu, 23 Apr 2015 11:08:40 +0000 (13:08 +0200)
Older gcc warns (and due to -Werror fails) on this ASSERT() now that
"node" is of unsigned type. Make it more useful at once.

Coverity-ID: 1055630

Signed-off-by: Jan Beulich <jbeulich@suse.com>
Acked-by: Tim Deegan <tim@xen.org>
Reviewed-by: Andrew Cooper <andrew.cooper3@citrix.com>
xen/common/page_alloc.c

index 986895d5d0538d5f6c1532c1e7b2205dfeaa1bc3..8500ed7c79a0204470d0026f5f3303f063a75e8f 100644 (file)
@@ -604,7 +604,7 @@ static struct page_info *alloc_heap_pages(
     }
     first_node = node;
 
-    ASSERT(node >= 0);
+    ASSERT(node < MAX_NUMNODES);
     ASSERT(zone_lo <= zone_hi);
     ASSERT(zone_hi < NR_ZONES);