diff options
author | Laszlo Ersek <lersek@redhat.com> | 2017-09-12 23:54:23 +0200 |
---|---|---|
committer | Laszlo Ersek <lersek@redhat.com> | 2017-09-14 00:05:20 +0200 |
commit | 5afa5b815936e2b45a375b6521764195bed68680 (patch) | |
tree | f8c1b412283ae477d021d2717c2a4a9150c53bcf /OvmfPkg/VirtioNetDxe | |
parent | 998aee899e49b37b217a0c842f6340ca27e6fa73 (diff) | |
download | edk2-5afa5b815936e2b45a375b6521764195bed68680.tar.gz edk2-5afa5b815936e2b45a375b6521764195bed68680.tar.bz2 edk2-5afa5b815936e2b45a375b6521764195bed68680.zip |
MdeModulePkg/UdfDxe: suppress incorrect compiler warning in ReadFile()
When building the driver for DEBUG/RELEASE, GCC48/GCC49 warn about
ReadFile() possibly using "BytesLeft" without initializing it first.
This is not the case. The reads of "BytesLeft" are only reachable if
(ReadFileInfo->Flags == READ_FILE_SEEK_AND_READ). But, in that case, we
also set "BytesLeft" to "ReadFileInfo->FileDataSize", near the top of the
function.
Assign "BytesLeft" zero at the top, and add a comment that conforms to the
pending Coding Style Spec feature request at
<https://bugzilla.tianocore.org/show_bug.cgi?id=607>.
This issue was reported by Ard's and Gerd's CI systems independently.
Cc: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Cc: Eric Dong <eric.dong@intel.com>
Cc: Paulo Alcantara <pcacjr@zytor.com>
Cc: Ruiyu Ni <ruiyu.ni@intel.com>
Cc: Star Zeng <star.zeng@intel.com>
Reported-by: Ard Biesheuvel <ard.biesheuvel@linaro.org>
Reported-by: Gerd Hoffmann <kraxel@redhat.com>
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Star Zeng <star.zeng@intel.com>
Reviewed-by: Paulo Alcantara <pcacjr@zytor.com>
Diffstat (limited to 'OvmfPkg/VirtioNetDxe')
0 files changed, 0 insertions, 0 deletions