diff options
| author | Jiaxin Wu <jiaxin.wu@intel.com> | 2017-12-26 11:10:34 +0800 |
|---|---|---|
| committer | Jiaxin Wu <jiaxin.wu@intel.com> | 2017-12-27 16:29:41 +0800 |
| commit | c0a0a5a5c76f7aaef30b5010ae1de6236defb445 (patch) | |
| tree | 4ab28f7ae0f3d899c7fef0619dcb908ea11a296e /NetworkPkg | |
| parent | 0d07e6fb53575ef4dd050a62ed0795575fa3eda9 (diff) | |
| download | edk2-c0a0a5a5c76f7aaef30b5010ae1de6236defb445.tar.gz edk2-c0a0a5a5c76f7aaef30b5010ae1de6236defb445.tar.bz2 edk2-c0a0a5a5c76f7aaef30b5010ae1de6236defb445.zip | |
NetworkPkg/HttpDxe: Fix the memory leak issue in HttpRequest().
Cc: Wang Fan <fan.wang@intel.com>
Cc: Ye Ting <ting.ye@intel.com>
Cc: Fu Siyuan <siyuan.fu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Wu Jiaxin <jiaxin.wu@intel.com>
Reviewed-by: Wang Fan <fan.wang@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>
Diffstat (limited to 'NetworkPkg')
| -rw-r--r-- | NetworkPkg/HttpDxe/HttpImpl.c | 8 |
1 files changed, 7 insertions, 1 deletions
diff --git a/NetworkPkg/HttpDxe/HttpImpl.c b/NetworkPkg/HttpDxe/HttpImpl.c index 57fa39f094..fe1c3b7784 100644 --- a/NetworkPkg/HttpDxe/HttpImpl.c +++ b/NetworkPkg/HttpDxe/HttpImpl.c @@ -472,6 +472,8 @@ EfiHttpRequest ( FreePool (HostName);
+ HttpUrlFreeParser (UrlParser);
+
//
// Queue the HTTP token and return.
//
@@ -656,6 +658,10 @@ EfiHttpRequest ( if (HostName != NULL) {
FreePool (HostName);
}
+
+ if (UrlParser != NULL) {
+ HttpUrlFreeParser (UrlParser);
+ }
return EFI_SUCCESS;
@@ -699,7 +705,7 @@ Error1: if (Wrap != NULL) {
FreePool (Wrap);
}
- if (UrlParser!= NULL) {
+ if (UrlParser != NULL) {
HttpUrlFreeParser (UrlParser);
}
|