Re: [edk2] [patch] MdeModulePkg/DisplayEngine: Fix memory leak issues in DisplayEngine

2016-05-19 Thread Bi, Dandan
, Dandan ; edk2-devel@lists.01.org Cc: Qiu, Shumin ; Dong, Eric Subject: RE: [edk2] [patch] MdeModulePkg/DisplayEngine: Fix memory leak issues in DisplayEngine Hi Dandan, The InputHandle.c one looks good. About the ProcessOptions.c one, I propose remove the lines starting "FreePool (Stri

Re: [edk2] [patch] MdeModulePkg/DisplayEngine: Fix memory leak issues in DisplayEngine

2016-05-19 Thread Sheng, Cecil (HPS SW)
; Eric Dong Subject: [edk2] [patch] MdeModulePkg/DisplayEngine: Fix memory leak issues in DisplayEngine Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Universal/DisplayEngineDxe/InputHandler.c

[edk2] [patch] MdeModulePkg/DisplayEngine: Fix memory leak issues in DisplayEngine

2016-05-19 Thread Dandan Bi
Cc: Qiu Shumin Cc: Eric Dong Contributed-under: TianoCore Contribution Agreement 1.0 Signed-off-by: Dandan Bi Reviewed-by: Eric Dong --- MdeModulePkg/Universal/DisplayEngineDxe/InputHandler.c | 5 + MdeModulePkg/Universal/DisplayEngineDxe/ProcessOptions.c | 3 ++- 2 files changed, 3 inse