diff options
| author | Doug Cook (WINDOWS) <dcook@microsoft.com> | 2024-12-01 04:32:03 +0300 |
|---|---|---|
| committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2024-12-17 22:01:48 +0300 |
| commit | d1fccbf494a4f38f803cb8288dd6b08ec7b98dcc (patch) | |
| tree | 7b66fd05637e710b5212fda8aa7b320cceac40bd /BaseTools/Source/Python/AutoGen/ModuleAutoGen.py | |
| parent | 30c8a73850b478042cc8e208298f37084940b9e7 (diff) | |
| download | edk2-d1fccbf494a4f38f803cb8288dd6b08ec7b98dcc.tar.xz | |
EmulatorPkg: spurious failure in WriteBlocks on X64
WinNtBlockIoWriteBlocks can spuriously fail on X64. This occurs because
&BytesWritten is a `UINTN*` (i.e. `UINT64*`) but is cast to `LPDWORD`
(i.e. `UINT32*`). Only the low 32 bits are initialized by WriteFile, so
the high 32 bits are uninitialized. This means we will spuriously fail
the `BytesWritten != BufferSize` test.
This doesn't occur on X86-32 since UINTN is the same as DWORD in that
case.
Fix is to declare BytesWritten as DWORD to match the type expected by
WriteFile. This also makes the cast unnecessary.
Signed-off-by: Doug Cook <idigdoug@gmail.com>
Diffstat (limited to 'BaseTools/Source/Python/AutoGen/ModuleAutoGen.py')
0 files changed, 0 insertions, 0 deletions