Re: [PATCH 1/3] BaseTools: Correct GenSec argument dummy free memory issue

[Laszlo Ersek <lersek@redhat.com>]

Hello Yunhua,

On 04/10/18 03:09, Feng, YunhuaX wrote:
> Free DummyFileBuffer and set DummyFileBuffer to NULL.
> 
> Cc: Liming Gao <liming.gao@intel.com>
> Cc: Yonghong Zhu <yonghong.zhu@intel.com>
> Contributed-under: TianoCore Contribution Agreement 1.1
> 
> Signed-off-by: Yunhua Feng <yunhuax.feng@intel.com>
> ---
>  BaseTools/Source/C/GenSec/GenSec.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)

can you please post patch sets with a cover letter, and with shallow
threading enabled?

The following three commands [1] should configure shallow threading
permanently:

git config format.coverletter     true
git config sendemail.chainreplyto false
git config sendemail.thread       true

Thank you,
Laszlo

[1]
https://github.com/tianocore/tianocore.github.io/wiki/Laszlo's-unkempt-git-guide-for-edk2-contributors-and-maintainers#contrib-05