diff options
author | Daryl McDaniel <edk2-lists@mc2research.org> | 2016-02-16 10:47:30 -0800 |
---|---|---|
committer | Daryl McDaniel <edk2-lists@mc2research.org> | 2016-02-17 16:11:29 -0800 |
commit | 7053c3b3b35275a052b2411bf73c1c4f578a0322 (patch) | |
tree | 814ab346947659e8781653d659f11c0913d1d3b9 /AppPkg | |
parent | ea3e924a0c91e2dd7fbb5e2f79899367222f27eb (diff) | |
download | edk2-7053c3b3b35275a052b2411bf73c1c4f578a0322.tar.gz edk2-7053c3b3b35275a052b2411bf73c1c4f578a0322.tar.bz2 edk2-7053c3b3b35275a052b2411bf73c1c4f578a0322.zip |
StdLib/BsdSocketLib: Fix minor memory leak by freeing rrecp on error return.
The error return is triggered by one of two conditions:
1. rrecp is NULL (calloc failed)
2. strdup(dname) returns NULL
Previously, the function just returned NULL. This patch adds a call to
free rrecp before returning NULL. Since the free() function will properly
do nothing when called with a NULL parameter, it is not necessary to
separate the two tests into separate if clauses.
Reported-by: Colin King <colin.king@canonical.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Daryl McDaniel <edk2-lists@mc2research.org>
Reviewed-by: Jaben Carsey <jaben.carsey@intel.com>
Diffstat (limited to 'AppPkg')
0 files changed, 0 insertions, 0 deletions