Commit: https://github.com/tianocore/edk2/commit/521981ee7608b75b51693ea367c9e1d83687d110
Thanks, Jiaxin > -----Original Message----- > From: Subramanian, Sriram [mailto:srira...@hpe.com] > Sent: Thursday, January 12, 2017 10:40 PM > To: Hegde, Nagaraj P <nagaraj-p.he...@hpe.com>; edk2-devel@lists.01.org > Cc: Wu, Jiaxin <jiaxin...@intel.com>; Carsey, Jaben <jaben.car...@intel.com>; > Ni, Ruiyu <ruiyu...@intel.com>; Zhang, Lubo <lubo.zh...@intel.com> > Subject: RE: [PATCH v1 1/1] ShellPkg/Ifconfig6: Address ASSERT because of a > missing NULL check > > Reviewed-by: Sriram Subramanian <srira...@hpe.com> > > -----Original Message----- > From: Hegde, Nagaraj P > Sent: Thursday, January 12, 2017 9:10 AM > To: edk2-devel@lists.01.org > Cc: jiaxin...@intel.com; jaben.car...@intel.com; ruiyu...@intel.com; > lubo.zh...@intel.com; Subramanian, Sriram <srira...@hpe.com> > Subject: [PATCH v1 1/1] ShellPkg/Ifconfig6: Address ASSERT because of a > missing NULL check > > When we issue 'ifconfig6 -s <interface> auto' system hangs with > an ASSERT in StrLen. in IfConfig6SetInterfaceInfo, for 'auto' case > we added checks to rule out the invalid inputs like 'host', 'gw' > and 'dns'. To parse through this, we do a VarArg = VarArg->Next but > we dont check new VarArg before calling StrCmp. Fix with a check > in this patch. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Hegde Nagaraj P <nagaraj-p.he...@hpe.com> > --- > ShellPkg/Library/UefiShellNetwork2CommandsLib/Ifconfig6.c | 27 > ++++++++++---------- > 1 file changed, 14 insertions(+), 13 deletions(-) > > diff --git a/ShellPkg/Library/UefiShellNetwork2CommandsLib/Ifconfig6.c > b/ShellPkg/Library/UefiShellNetwork2CommandsLib/Ifconfig6.c > index d71688e..72aaee4 100644 > --- a/ShellPkg/Library/UefiShellNetwork2CommandsLib/Ifconfig6.c > +++ b/ShellPkg/Library/UefiShellNetwork2CommandsLib/Ifconfig6.c > @@ -1317,20 +1317,21 @@ IfConfig6SetInterfaceInfo ( > > VarArg= VarArg->Next; > > - if (StrCmp (VarArg->Arg, L"host") == 0) { > - ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN > (STR_IFCONFIG6_ERR_INVALID_IP_CONFIG), gShellNetwork2HiiHandle, Status); > - ShellStatus = SHELL_INVALID_PARAMETER; > - goto ON_EXIT; > - } else if (StrCmp (VarArg->Arg, L"gw") == 0) { > - ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN > (STR_IFCONFIG6_ERR_INVALID_GW_CONFIG), gShellNetwork2HiiHandle, > Status); > - ShellStatus = SHELL_INVALID_PARAMETER; > - goto ON_EXIT; > - } else if (StrCmp (VarArg->Arg, L"dns") == 0) { > - ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN > (STR_IFCONFIG6_ERR_INVALID_DNS_CONFIG), gShellNetwork2HiiHandle, > Status); > - ShellStatus = SHELL_INVALID_PARAMETER; > - goto ON_EXIT; > + if (VarArg != NULL) { > + if (StrCmp (VarArg->Arg, L"host") == 0) { > + ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN > (STR_IFCONFIG6_ERR_INVALID_IP_CONFIG), gShellNetwork2HiiHandle, Status); > + ShellStatus = SHELL_INVALID_PARAMETER; > + goto ON_EXIT; > + } else if (StrCmp (VarArg->Arg, L"gw") == 0) { > + ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN > (STR_IFCONFIG6_ERR_INVALID_GW_CONFIG), gShellNetwork2HiiHandle, > Status); > + ShellStatus = SHELL_INVALID_PARAMETER; > + goto ON_EXIT; > + } else if (StrCmp (VarArg->Arg, L"dns") == 0) { > + ShellPrintHiiEx (-1, -1, NULL, STRING_TOKEN > (STR_IFCONFIG6_ERR_INVALID_DNS_CONFIG), gShellNetwork2HiiHandle, > Status); > + ShellStatus = SHELL_INVALID_PARAMETER; > + goto ON_EXIT; > + } > } > - > } else if (StrCmp (VarArg->Arg, L"man") == 0) { > // > // Set manual config policy. > -- > 2.8.3.windows.1 > _______________________________________________ edk2-devel mailing list edk2-devel@lists.01.org https://lists.01.org/mailman/listinfo/edk2-devel
