Branch: refs/heads/master
  Home:   https://github.com/tianocore/edk2
  Commit: bd3fc8133b2b17ad2e0427d1bf6b44b08cf2f3b2
      
https://github.com/tianocore/edk2/commit/bd3fc8133b2b17ad2e0427d1bf6b44b08cf2f3b2
  Author: Marvin H?user <[email protected]>
  Date:   2016-05-25 (Wed, 25 May 2016)

  Changed paths:
    M ShellPkg/Application/Shell/Shell.c

  Log Message:
  -----------
  ShellPkg/App: Fix memory leak and save resources.

1) RunSplitCommand() allocates the initial SplitStdOut via
   CreateFileInterfaceMem(). Free SplitStdIn after the swap to fix
   the memory leak.

2) In RunSplitCommand(), SplitStdOut is checked for equality with
   StdIn. This cannot happen due to the if-check within the swap.
   Hence remove it.

3) UefiMain() doesn't free SplitList. Delete all list entries and
   reinitialize the list when in DEBUG. This does not include the
   CreateFileInterfaceMem()-allocated SplitStd mentioned in 1), so
   keep the ASSERT() until resolved.

Contributed-under: TianoCore Contribution Agreement 1.0
Signed-off-by: Marvin Haeuser <[email protected]>
Reviewed-by: Qiu Shumin <[email protected]>


------------------------------------------------------------------------------
Mobile security can be enabling, not merely restricting. Employees who
bring their own devices (BYOD) to work are irked by the imposition of MDM
restrictions. Mobile Device Manager Plus allows you to control only the
apps on BYO-devices by containerizing them, leaving personal data untouched!
https://ad.doubleclick.net/ddm/clk/304595813;131938128;j
_______________________________________________
edk2-commits mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/edk2-commits

Reply via email to