lynxis lazus submitted this change.

View Change

Approvals: Jenkins Builder: Verified laforge: Looks good to me, but someone else must approve pespin: Looks good to me, approved
xua_msg_copy(): add missing check if xua_msg_alloc() fails

Change-Id: I7f2e098dcf7790862eb509357f8d214c87212381
---
M src/xua_msg.c
1 file changed, 2 insertions(+), 0 deletions(-)

diff --git a/src/xua_msg.c b/src/xua_msg.c
index 0652277..1761fee 100644
--- a/src/xua_msg.c
+++ b/src/xua_msg.c
@@ -62,6 +62,8 @@
{
struct xua_msg_part *part;
struct xua_msg *xua_cpy = xua_msg_alloc();
+ if (!xua_cpy)
+ return NULL;

xua_cpy->hdr = xua_in->hdr;
xua_cpy->mtp = xua_in->mtp;

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

Gerrit-MessageType: merged
Gerrit-Project: libosmo-sigtran
Gerrit-Branch: master
Gerrit-Change-Id: I7f2e098dcf7790862eb509357f8d214c87212381
Gerrit-Change-Number: 41205
Gerrit-PatchSet: 2
Gerrit-Owner: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: lynxis lazus <lynxis@fe80.eu>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>