Revision: 14813
          http://sourceforge.net/p/edk2/code/14813
Author:   jiaxinwu
Date:     2013-10-29 08:51:16 +0000 (Tue, 29 Oct 2013)
Log Message:
-----------
Fix a K9 error.Result of GetVariable2() function that may return NULL will be 
dereferenced.

Signed-off-by: Wu Jiaxin <[email protected] >
Reviewed-by: Fu Siyuan <[email protected]>
Reviewed-by: Dong Guo <[email protected]>

Modified Paths:
--------------
    trunk/edk2/MdeModulePkg/Universal/Network/IScsiDxe/IScsiMisc.c

Modified: trunk/edk2/MdeModulePkg/Universal/Network/IScsiDxe/IScsiMisc.c
===================================================================
--- trunk/edk2/MdeModulePkg/Universal/Network/IScsiDxe/IScsiMisc.c      
2013-10-29 06:52:25 UTC (rev 14812)
+++ trunk/edk2/MdeModulePkg/Universal/Network/IScsiDxe/IScsiMisc.c      
2013-10-29 08:51:16 UTC (rev 14813)
@@ -656,10 +656,9 @@
              (VOID**)&ConfigDataTmp,
              NULL
              );
-  if (EFI_ERROR (Status)) {
+  if (ConfigDataTmp == NULL || EFI_ERROR (Status)) {
     return FALSE;
   }
-  
 
   if (ConfigDataTmp->Enabled && ConfigDataTmp->InitiatorInfoFromDhcp) {
     FreePool (ConfigDataTmp);

This was sent by the SourceForge.net collaborative development platform, the 
world's largest Open Source development site.


------------------------------------------------------------------------------
Android is increasing in popularity, but the open development platform that
developers love is also attractive to malware creators. Download this white
paper to learn more about secure code signing practices that can help keep
Android apps secure.
http://pubads.g.doubleclick.net/gampad/clk?id=65839951&iu=/4140/ostg.clktrk
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to