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
