pespin submitted this change.

Approvals:
  osmith: Looks good to me, but someone else must approve
  Jenkins Builder: Verified
  laforge: Looks good to me, approved

osmo_io: Validate msghdr!=NULL after allocation

Related: Coverity CID#556925
Change-Id: Ib90e6b5bc6c03cbbf40041463f3d05d85f55a68a
---
M src/core/osmo_io_poll.c
1 file changed, 5 insertions(+), 0 deletions(-)

diff --git a/src/core/osmo_io_poll.c b/src/core/osmo_io_poll.c
index c37d436..b861675 100644
--- a/src/core/osmo_io_poll.c
+++ b/src/core/osmo_io_poll.c
@@ -106,6 +106,11 @@
 		}
 
 		msghdr = iofd_msghdr_alloc(iofd, action, NULL, iofd->cmsg_size);
+		if (!msghdr) {
+			LOGPIO(iofd, LOGL_ERROR, "Could not allocate msghdr for reading\n");
+			OSMO_ASSERT(0);
+		}
+
 		for (idx = 0; idx < msghdr->io_len; idx++) {
 			msghdr->iov[idx].iov_base = msghdr->msg[idx]->tail;
 			msghdr->iov[idx].iov_len = msgb_tailroom(msghdr->msg[idx]);

To view, visit change 41840. To unsubscribe, or for help writing mail filters, visit settings.