This is exactly my use-case for wanting to change FreePool() to allow it to
take a NULL pointer without throwing an ASSERT. Just saying...
On 19 September 2013 18:21, Carsey, Jaben <jaben.car...@intel.com> wrote:
> Reviewed-by: Jaben Carsey <jaben.car...@intel.com>
>
> I will commit this.
>
> -Jaben
>
> -----Original Message-----
> From: Sergei Antonov [mailto:sap...@gmail.com]
> Sent: Thursday, September 19, 2013 3:31 AM
> To: edk2-devel@lists.sourceforge.net
> Subject: Re: [edk2] ShellPkg patch: Handle pool allocation failure
>
> ShellPkg: Handle pool allocation failure
>
> If AllocateZeroPool() returns NULL,
> GetVariable() will return EFI_BUFFER_TOO_SMALL,
> FreePool() will receive NULL.
> So check for NULL before FreePool().
>
> Contributed-under: TianoCore Contribution Agreement 1.0
> Signed-off-by: Sergei Antonov <sap...@gmail.com>
>
> ------------------------------------------------------------------------------
> LIMITED TIME SALE - Full Year of Microsoft Training For Just $49.99!
> 1,500+ hours of tutorials including VisualStudio 2012, Windows 8,
> SharePoint
> 2013, SQL 2012, MVC 4, more. BEST VALUE: New Multi-Library Power Pack
> includes
> Mobile, Cloud, Java, and UX Design. Lowest price ever! Ends 9/20/13.
> http://pubads.g.doubleclick.net/gampad/clk?id=58041151&iu=/4140/ostg.clktrk
> _______________________________________________
> edk2-devel mailing list
> edk2-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/edk2-devel
>
------------------------------------------------------------------------------
LIMITED TIME SALE - Full Year of Microsoft Training For Just $49.99!
1,500+ hours of tutorials including VisualStudio 2012, Windows 8, SharePoint
2013, SQL 2012, MVC 4, more. BEST VALUE: New Multi-Library Power Pack includes
Mobile, Cloud, Java, and UX Design. Lowest price ever! Ends 9/20/13.
http://pubads.g.doubleclick.net/gampad/clk?id=58041151&iu=/4140/ostg.clktrk
_______________________________________________
edk2-devel mailing list
edk2-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/edk2-devel
