diff options
| author | Gerd Hoffmann <kraxel@redhat.com> | 2023-02-15 15:37:32 +0100 |
|---|---|---|
| committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2023-03-09 09:45:39 +0000 |
| commit | 2fccd1f153e1b7146d6fe9d9f8cfaeabdf636c44 (patch) | |
| tree | 172da5a0226f2248a36a1e6086bbeb291d1dce56 /ArmPkg/Filesystem/SemihostFs/Arm/SemihostFs.c | |
| parent | 69da506c927f8092ea8f783a092a694a3582e3ef (diff) | |
| download | edk2-2fccd1f153e1b7146d6fe9d9f8cfaeabdf636c44.tar.gz edk2-2fccd1f153e1b7146d6fe9d9f8cfaeabdf636c44.tar.bz2 edk2-2fccd1f153e1b7146d6fe9d9f8cfaeabdf636c44.zip | |
ArmPkg/SemihostFs: replace SetMem with ZeroMem
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 <jeremy.boone@nccgroup.com>
Signed-off-by: Gerd Hoffmann <kraxel@redhat.com>
Reviewed-by: Rebecca Cran <rebecca@bsdio.com>
Reviewed-by: Sami Mujawar <sami.mujawar@arm.com>
Diffstat (limited to 'ArmPkg/Filesystem/SemihostFs/Arm/SemihostFs.c')
| -rw-r--r-- | ArmPkg/Filesystem/SemihostFs/Arm/SemihostFs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/ArmPkg/Filesystem/SemihostFs/Arm/SemihostFs.c b/ArmPkg/Filesystem/SemihostFs/Arm/SemihostFs.c index 39a30533ee..9cc9ed7d36 100644 --- a/ArmPkg/Filesystem/SemihostFs/Arm/SemihostFs.c +++ b/ArmPkg/Filesystem/SemihostFs/Arm/SemihostFs.c @@ -574,7 +574,7 @@ ExtendFile ( }
Remaining = Size;
- SetMem (WriteBuffer, 0, sizeof (WriteBuffer));
+ ZeroMem (WriteBuffer, sizeof (WriteBuffer));
while (Remaining > 0) {
WriteNb = MIN (Remaining, sizeof (WriteBuffer));
WriteSize = WriteNb;
|