diff options
author | Skylar Chang <chiaweic@codeaurora.org> | 2014-07-25 03:48:43 -0700 |
---|---|---|
committer | Skylar Chang <chiaweic@codeaurora.org> | 2014-07-25 04:04:43 -0700 |
commit | e8c9d249798d419e687fcf6a4d0eff2a734f476f (patch) | |
tree | 8e7f5e90f18a0b21b75dbff1d1fc805ef044dbe3 /ipacm/src/IPACM_Netlink.cpp | |
parent | 87735cb4a198e744df3dc22eebdecea5cd8b4f07 (diff) | |
download | ipacfg-mgr-e8c9d249798d419e687fcf6a4d0eff2a734f476f.tar.gz |
IPACM: Fix klocwork issue on master branch
Fix some potential memory overflow/leak issue
reported by Klocwork tool.
Change-Id: I63ff06ffbf4920d733f4e82d0823666874e79d19
Diffstat (limited to 'ipacm/src/IPACM_Netlink.cpp')
-rw-r--r-- | ipacm/src/IPACM_Netlink.cpp | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/ipacm/src/IPACM_Netlink.cpp b/ipacm/src/IPACM_Netlink.cpp index 34c1139..b5d7060 100644 --- a/ipacm/src/IPACM_Netlink.cpp +++ b/ipacm/src/IPACM_Netlink.cpp @@ -1502,6 +1502,12 @@ int ipa_nl_recv_msg(int fd) goto error; } + if(msghdr== NULL) + { + IPACMERR(" failed to get msghdr\n"); + goto error; + } + iov = msghdr->msg_iov; memset(nlmsg, 0, sizeof(ipa_nl_msg_t)); @@ -1513,12 +1519,12 @@ int ipa_nl_recv_msg(int fd) /* Release NetLink message buffer */ if(msghdr) { - ipa_nl_release_msg(msghdr); + ipa_nl_release_msg(msghdr); } if(nlmsg) { - free(nlmsg); - } + free(nlmsg); + } } return IPACM_SUCCESS; |