avoid NULL argument to memcpy()

gcc 4.9 takes advantage of the specification of undefined behavior if you pass a NULL to memcpy(), even if the last argument is 0
racket · Aug 12, 2014 · df375da · df375da
1 parent 3c8b5b6
commit df375da
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/racket/src/racket/sgc/sgc.c b/racket/src/racket/sgc/sgc.c
@@ -1653,11 +1653,11 @@ void GC_add_roots(void *start, void *end)
 
     mem_real_use += (sizeof(uintptr_t) * roots_size);
 
-    memcpy((void *)naya, (void *)roots, 
-	   sizeof(uintptr_t) * roots_count);
-
-    if (roots)
+    if (roots) {
+      memcpy((void *)naya, (void *)roots,
+             sizeof(uintptr_t) * roots_count);
       free_managed(roots);
+    }
 
     roots = naya;
   }