sry about that, confirming you addition hereby. Michael
On Mon, May 9, 2016 at 3:47 AM, Tian, Feng <[email protected]> wrote: > Thanks for your contribution, Mike. > > Your commit log misses the license statement, I will help add below in. > > Contributed-under: TianoCore Contribution Agreement 1.0 > Signed-off-by: Michael Zimmermann <[email protected]> > > Here is my response: > Reviewed-by: Feng Tian <[email protected]> > > Thanks > Feng > > -----Original Message----- > From: edk2-devel [mailto:[email protected]] On Behalf Of > Michael Zimmermann > Sent: Saturday, May 7, 2016 11:17 AM > To: [email protected] <[email protected]> > Subject: [edk2] [PATCH] MdeModulePkg: FileExplorerLib: prevent freeing > null pointer > > when there's no volume label 'Info' can be NULL > --- > MdeModulePkg/Library/FileExplorerLib/FileExplorer.c | 3 ++- > 1 file changed, 2 insertions(+), 1 deletion(-) > > diff --git a/MdeModulePkg/Library/FileExplorerLib/FileExplorer.c > b/MdeModulePkg/Library/FileExplorerLib/FileExplorer.c > index 9f75d6e..a9ca432 100644 > --- a/MdeModulePkg/Library/FileExplorerLib/FileExplorer.c > +++ b/MdeModulePkg/Library/FileExplorerLib/FileExplorer.c > @@ -775,7 +775,8 @@ LibFindFileSystem ( > MenuEntry->DisplayString, > NULL > ); > - FreePool (Info); > + if (Info != NULL) > + FreePool (Info); > > OptionNumber++; > InsertTailList (&gFileExplorerPrivate.FsOptionMenu->Head, > &MenuEntry->Link); > -- > 2.8.2 > _______________________________________________ > edk2-devel mailing list > [email protected] > https://lists.01.org/mailman/listinfo/edk2-devel > _______________________________________________ edk2-devel mailing list [email protected] https://lists.01.org/mailman/listinfo/edk2-devel
