Commit Diff

Commit:
151af94ebac619594e1f4599e395b8946cb47d32
From:
Omar Polo <op@omarpolo.com>
Date:
Message:
only memcpy into imsg->data if datalen is not zero Passing a NULL pointer to memcpy is UB no matter if len is 0 (commit by claudio@, backporting for gmid.)
Actions:
Patch | Tree
commit - 12a5e730e975767f3ba4adbb3b0b50236b8f830c
commit + 151af94ebac619594e1f4599e395b8946cb47d32
blob - 3d36a402db11c258417b0ffc9ccd58e45babb78f
blob + 74aead240e41bac72a32b877480e115eb786c957
--- compat/imsg.c
+++ compat/imsg.c
@@ -1,4 +1,4 @@
-/*	$OpenBSD: imsg.c,v 1.16 2017/12/14 09:27:44 kettenis Exp $	*/
+/*	$OpenBSD: imsg.c,v 1.17 2022/01/28 10:41:44 claudio Exp $	*/
 
 /*
  * Copyright (c) 2003, 2004 Henning Brauer <henning@openbsd.org>
@@ -151,7 +151,8 @@ imsg_get(struct imsgbuf *ibuf, struct imsg *imsg)
 	else
 		imsg->fd = -1;
 
-	memcpy(imsg->data, ibuf->r.rptr, datalen);
+	if (datalen != 0)
+		memcpy(imsg->data, ibuf->r.rptr, datalen);
 
 	if (imsg->hdr.len < av) {
 		left = av - imsg->hdr.len;