summaryrefslogtreecommitdiff
path: root/MdeModulePkg/MdeModulePkg.dsc
diff options
context:
space:
mode:
authorMarvin H?user <Marvin.Haeuser@outlook.com>2016-05-20 03:04:02 +0800
committerQiu Shumin <shumin.qiu@intel.com>2016-05-25 10:16:46 +0800
commitbd3fc8133b2b17ad2e0427d1bf6b44b08cf2f3b2 (patch)
treedeff7cd5734e2a08add8dbb4cdf05dc2a9d0ec4d /MdeModulePkg/MdeModulePkg.dsc
parentdc99315b8732b6e3032d01319d3f534d440b43d0 (diff)
downloadedk2-bd3fc8133b2b17ad2e0427d1bf6b44b08cf2f3b2.zip
edk2-bd3fc8133b2b17ad2e0427d1bf6b44b08cf2f3b2.tar.gz
edk2-bd3fc8133b2b17ad2e0427d1bf6b44b08cf2f3b2.tar.bz2
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 <Marvin.Haeuser@outlook.com> Reviewed-by: Qiu Shumin <shumin.qiu@intel.com>
Diffstat (limited to 'MdeModulePkg/MdeModulePkg.dsc')
0 files changed, 0 insertions, 0 deletions