v2:
*Revise some codes according to the comments.
1.In HttpResponseWorker, check the HttpHeaders in the first used place.
2.In EfiHttpPoll(), check the HttpInstance state outside of if condition.
Cc: Ye Ting
Cc: Fu Siyuan
Cc: Wu Jiaxin
, Jiaxin; Qiu, Shumin
Subject: RE: [edk2] [PATCH v2] NetworkPkg:Fix NULL pointer dereference issues.
Hi, Lubo
Should HttpHeaders always not be a NULL pointer upon a success call of
HttpTcpReceiveHeader?
If yes, you should use ASSERT here, otherwise the patch is ok with me?
Status
<ting...@intel.com>; Fu, Siyuan <siyuan...@intel.com>; Wu, Jiaxin
<jiaxin...@intel.com>; Qiu, Shumin <shumin....@intel.com>
Subject: [edk2] [PATCH v2] NetworkPkg:Fix NULL pointer dereference issues.
v2:
*Revise some codes according to the comments.
1.In HttpResponseWorker, chec
ng; Wu, Jiaxin; Qiu, Shumin
Subject: RE: [edk2] [PATCH v2] NetworkPkg:Fix NULL pointer dereference issues.
Hi, Lubo
Should HttpHeaders always not be a NULL pointer upon a success call of
HttpTcpReceiveHeader?
If yes, you should use ASSERT here, otherwise the patch is ok with me?
ng; Wu, Jiaxin; Qiu, Shumin
Subject: RE: [edk2] [PATCH v2] NetworkPkg:Fix NULL pointer dereference issues.
Hi, Lubo
Should HttpHeaders always not be a NULL pointer upon a success call of
HttpTcpReceiveHeader?
If yes, you should use ASSERT here, otherwise the patch is ok with me?
5 matches
Mail list logo