From d1b4498645fcd6218b6e4d822795a74e123f88a6 Mon Sep 17 00:00:00 2001 From: reyk <> Date: Wed, 19 Dec 2012 13:37:12 +0000 Subject: [PATCH] Fix a purely theoretical NULL-pointer dereference in the case that we would be able to receive multiple SCM_RIGHTS messages. ok claudio@ gilles@ --- src/lib/libutil/imsg.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/lib/libutil/imsg.c b/src/lib/libutil/imsg.c index 05e57c7e..b506297b 100644 --- a/src/lib/libutil/imsg.c +++ b/src/lib/libutil/imsg.c @@ -1,4 +1,4 @@ -/* $OpenBSD: imsg.c,v 1.2 2012/06/02 21:46:53 gilles Exp $ */ +/* $OpenBSD: imsg.c,v 1.3 2012/12/19 13:37:12 reyk Exp $ */ /* * Copyright (c) 2003, 2004 Henning Brauer @@ -103,7 +103,7 @@ again: (char *)CMSG_DATA(cmsg)) / sizeof(int); for (i = 0; i < j; i++) { fd = ((int *)CMSG_DATA(cmsg))[i]; - if (i == 0) { + if (ifd != NULL) { ifd->fd = fd; TAILQ_INSERT_TAIL(&ibuf->fds, ifd, entry);