portability: memcpy of size zero is undefined behaviour

frida · Jan 1, 2023 · f378d63 · f378d63
1 parent a3303a2
commit f378d63
Showing 1 changed file with 4 additions and 2 deletions.
diff --git a/quickjs.c b/quickjs.c
@@ -32773,8 +32773,10 @@ static JSValue js_create_function(JSContext *ctx, JSFunctionDef *fd)
             }
         } else {
             b->vardefs = (void *)((uint8_t*)b + vardefs_offset);
-            memcpy(b->vardefs, fd->args, fd->arg_count * sizeof(fd->args[0]));
-            memcpy(b->vardefs + fd->arg_count, fd->vars, fd->var_count * sizeof(fd->vars[0]));
+            if (fd->arg_count > 0) {
+                memcpy(b->vardefs, fd->args, fd->arg_count * sizeof(fd->args[0]));
+                memcpy(b->vardefs + fd->arg_count, fd->vars, fd->var_count * sizeof(fd->vars[0]));
+	    }
         }
         b->var_count = fd->var_count;
         b->arg_count = fd->arg_count;