Closed Bug 1597605 Opened 4 years ago Closed 4 years ago

TestDllInterceptor.exe throws an exception on WOW64

Categories

(Core :: mozglue, defect, P3)

x86_64
Windows
defect

Tracking

()

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: toshi, Assigned: toshi)

Details

Attachments

(1 file)

If we pass an invalid address to NtQueryFullAttributesFile as the first argument, it causes AV on WOW64. The test program does not crash because the exception is caught by a handler of wow64 anyway, but it's annoying to hit this while debugging TestDllInterceptor.exe. Let's fix this.

(32fc.d30): Access violation - code c0000005 (first chance)
First chance exceptions are reported before any exception handling.
This exception may be expected and handled.
wow64!whNtQueryFullAttributesFile+0x102:
00007ffa`50fd2902 0f1103          movups  xmmword ptr [rbx],xmm0 ds:00000000`00000000=????????????????????????????????
0:000> knL
 # Child-SP          RetAddr           Call Site
00 00000000`0414e3f0 00007ffa`50fc7123 wow64!whNtQueryFullAttributesFile+0x102
01 00000000`0414e4b0 00000000`772d1783 wow64!Wow64SystemServiceEx+0x153
02 00000000`0414ed70 00000000`772d1199 wow64cpu!ServiceNoTurbo+0xb
03 00000000`0414ee20 00007ffa`50fcc77a wow64cpu!BTCpuSimulate+0x9
04 00000000`0414ee60 00007ffa`50fcc637 wow64!RunCpuSimulation+0xa
05 00000000`0414ee90 00007ffa`52ad3fb3 wow64!Wow64LdrpInitialize+0x127
06 00000000`0414f140 00007ffa`52ac1db5 ntdll!LdrpInitializeProcess+0x186b
07 00000000`0414f580 00007ffa`52a71853 ntdll!_LdrpInitialize+0x50549
08 00000000`0414f620 00007ffa`52a717fe ntdll!LdrpInitialize+0x3b
09 00000000`0414f650 00000000`00000000 ntdll!LdrInitializeThunk+0xe

Passing a dummy object to NtQueryFullAttributesFile to avoid AV on WOW64.

Pushed by apavel@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/350397304e64
Pass a valid OBJECT_ATTRIBUTES to NtQueryFullAttributesFile. r=handyman
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: