Reviewed-by: Fu Siyuan <siyuan...@intel.com>
-----Original Message-----
From: Wu, Jiaxin
Sent: Tuesday, September 8, 2015 9:41 AM
To: edk2-devel@lists.01.org
Cc: Ye, Ting <ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>
Subject: [Patch] NetworkPkg: Fix suspicious dereference of pointer 'FieldCount'
This patch is used to fix suspicious dereference of pointer 'FieldCount'
before NULL check.
Cc: Ye Ting <ting...@intel.com>
Cc: Fu Siyuan <siyuan...@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jiaxin Wu <jiaxin...@intel.com>
---
NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesProtocol.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesProtocol.c
b/NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesProtocol.c
index 8c29f20..a83c963 100644
--- a/NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesProtocol.c
+++ b/NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesProtocol.c
@@ -305,11 +305,10 @@ HttpUtilitiesParse (
CHAR8 *FieldValue;
UINTN Index;
Status = EFI_SUCCESS;
TempHttpMessage = NULL;
- *FieldCount = 0;
Token = NULL;
NextToken = NULL;
FieldName = NULL;
FieldValue = NULL;
Index = 0;
@@ -326,10 +325,11 @@ HttpUtilitiesParse (
CopyMem (TempHttpMessage, HttpMessage, HttpMessageSize);
//
// Get header number
//
+ *FieldCount = 0;
Token = TempHttpMessage;
while (TRUE) {
FieldName = NULL;
FieldValue = NULL;
NextToken = GetFieldNameAndValue (Token, &FieldName, &FieldValue);
--
1.9.5.msysgit.1
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.01.org
https://lists.01.org/mailman/listinfo/edk2-devel
