Re: [edk2-devel] [PATCH 1/1] ArmPkg/SemihostFs: replace SetMem with ZeroMem

2023-03-09 Thread Ard Biesheuvel
On Thu, 9 Mar 2023 at 10:16, Gerd Hoffmann wrote: > > Ping. Two reviews and the freeze is over. Merge this now? > Pong. Thanks for the reminder - I'll go and merge this. > On Wed, Feb 15, 2023 at 03:37:32PM +0100, Gerd Hoffmann wrote: > > SetMem arguments 2+3 are in the wrong order, resulting

Re: [edk2-devel] [PATCH 1/1] ArmPkg/SemihostFs: replace SetMem with ZeroMem

2023-03-09 Thread Gerd Hoffmann
Ping. Two reviews and the freeze is over. Merge this now? On Wed, Feb 15, 2023 at 03:37:32PM +0100, Gerd Hoffmann wrote: > SetMem arguments 2+3 are in the wrong order, resulting in > the call having no effect because Length is zero. > > Fix this by using ZeroMem instead. > > Bugzilla:

Re: [edk2-devel] [PATCH 1/1] ArmPkg/SemihostFs: replace SetMem with ZeroMem

2023-02-21 Thread Sami Mujawar
Hi Gerd, Thank you for this patch. Reviewed-by: Sami Mujawar Regards, Sami Mujawar On 15/02/2023, 14:37, "Gerd Hoffmann" mailto:kra...@redhat.com>> wrote: SetMem arguments 2+3 are in the wrong order, resulting in the call having no effect because Length is zero. Fix this by using

Re: [edk2-devel] [PATCH 1/1] ArmPkg/SemihostFs: replace SetMem with ZeroMem

2023-02-15 Thread Rebecca Cran
Reviewed-by: Rebecca Cran On 2/15/23 07:37, Gerd Hoffmann wrote: SetMem arguments 2+3 are in the wrong order, resulting in the call having no effect because Length is zero. Fix this by using ZeroMem instead. Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=4205 Reported-by: Jeremy

[edk2-devel] [PATCH 1/1] ArmPkg/SemihostFs: replace SetMem with ZeroMem

2023-02-15 Thread Gerd Hoffmann
SetMem arguments 2+3 are in the wrong order, resulting in the call having no effect because Length is zero. Fix this by using ZeroMem instead. Bugzilla: https://bugzilla.tianocore.org/show_bug.cgi?id=4205 Reported-by: Jeremy Boone Signed-off-by: Gerd Hoffmann ---