From: Jiaxin Wu <jiaxin.wu@intel.com>
Date: Tue, 8 Sep 2015 02:01:05 +0000 (+0000)
Subject: NetworkPkg: Fix suspicious dereference of pointer 'FieldCount'
X-Git-Tag: edk2-stable201903~8906
X-Git-Url: https://git.proxmox.com/?p=mirror_edk2.git;a=commitdiff_plain;h=2ba88dc554f074946189ec178b82ae2a9974247a

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.ye@intel.com>
Cc: Fu Siyuan <siyuan.fu@intel.com>
Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Jiaxin Wu <jiaxin.wu@intel.com>
Reviewed-by: Fu Siyuan <siyuan.fu@intel.com>

git-svn-id: https://svn.code.sf.net/p/edk2/code/trunk/edk2@18409 6f19259b-4bc3-4df7-8a09-765794883524
---

diff --git a/NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesProtocol.c b/NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesProtocol.c
index 8c29f20adf..a83c9633fc 100644
--- a/NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesProtocol.c
+++ b/NetworkPkg/HttpUtilitiesDxe/HttpUtilitiesProtocol.c
@@ -307,7 +307,6 @@ HttpUtilitiesParse (
   
   Status          = EFI_SUCCESS;
   TempHttpMessage = NULL;
-  *FieldCount     = 0;
   Token           = NULL;
   NextToken       = NULL;
   FieldName       = NULL;
@@ -328,6 +327,7 @@ HttpUtilitiesParse (
   //
   // Get header number
   //
+  *FieldCount = 0;
   Token = TempHttpMessage;
   while (TRUE) {
     FieldName     = NULL;