Closed
Bug 1432505
Opened 6 years ago
Closed 6 years ago
Sandbox MinGW Compilation errors: SmartStub causes errors
Categories
(Core :: Security: Process Sandboxing, defect)
Core
Security: Process Sandboxing
Tracking
()
RESOLVED
WONTFIX
People
(Reporter: tjr, Assigned: tjr)
References
Details
(Whiteboard: [tor][mingw-upstream-pending])
Attachments
(1 file)
59 bytes,
text/x-review-board-request
|
Details |
0:02.10 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc: In member function ‘virtual NTSTATUS sandbox::SmartSidestepResolverThunk::Setup(const void*, const void*, const char*, const char*, const void*, void*, size_t, size_t*)’: 0:02.10 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc:111:57: error: invalid conversion from ‘void (*)()’ to ‘const void*’ [-fpermissive] 0:02.10 target_name, NULL, &SmartStub, 0:02.10 ^~~~~~~~~~ 0:02.10 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc:32:10: note: initializing argument 5 of ‘virtual NTSTATUS sandbox::SidestepResolverThunk::Setup(const void*, const void*, const char*, const char*, const void*, void*, size_t, size_t*)’ 0:02.10 NTSTATUS SidestepResolverThunk::Setup(const void* target_module, 0:02.10 ^~~~~~~~~~~~~~~~~~~~~ 0:02.10 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc:118:35: error: invalid conversion from ‘void (*)()’ to ‘const void*’ [-fpermissive] 0:02.10 thunk_storage, &SmartStub); 0:02.10 ^~~~~~~~~~ 0:02.11 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc: In static member function ‘static void sandbox::SmartSidestepResolverThunk::SmartStub()’: 0:02.11 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc:153:9: error: expected ‘(’ before ‘{’ token 0:02.11 __asm { 0:02.11 ^ 0:02.11 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc:154:5: error: ‘push’ was not declared in this scope 0:02.11 push eax // Space for the jump. 0:02.11 ^~~~ I don't really understand this one. But Bob it was okay to obliterate the function. I think I'll put a Release Assert to ensure it's not getting called.
Assignee | ||
Comment 1•6 years ago
|
||
Going to handle this error too: 0:02.05 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc: In member function ‘virtual NTSTATUS sandbox::SmartSidestepResolverThunk::Setup(const void*, const void*, const char*, const char*, const void*, void*, size_t, size_t*)’: 0:02.05 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc:111:57: error: invalid conversion from ‘void (*)()’ to ‘const void*’ [-fpermissive] 0:02.05 target_name, NULL, &SmartStub, 0:02.05 ^~~~~~~~~~ 0:02.05 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc:32:10: note: initializing argument 5 of ‘virtual NTSTATUS sandbox::SidestepResolverThunk::Setup(const void*, const void*, const char*, const char*, const void*, void*, size_t, size_t*)’ 0:02.05 NTSTATUS SidestepResolverThunk::Setup(const void* target_module, 0:02.05 ^~~~~~~~~~~~~~~~~~~~~ 0:02.05 /home/tom/Documents/moz/mingw-work/sandbox/security/sandbox/chromium/sandbox/win/src/sidestep_resolver.cc:118:35: error: invalid conversion from ‘void (*)()’ to ‘const void*’ [-fpermissive] 0:02.05 thunk_storage, &SmartStub); 0:02.05 ^~~~~~~~~~
Comment hidden (mozreview-request) |
Comment 3•6 years ago
|
||
mozreview-review |
Comment on attachment 8944793 [details] Bug 1432505 Do not compile inline assembly on MinGW, instead break and abort https://reviewboard.mozilla.org/r/214952/#review220994 Clearing this, because it should be fixed by the patch in bug 1432381.
Attachment #8944793 -
Flags: review?(bobowencode)
Assignee | ||
Comment 4•6 years ago
|
||
Won't need this one
Status: NEW → RESOLVED
Closed: 6 years ago
Resolution: --- → WONTFIX
You need to log in
before you can comment on or make changes to this bug.
Description
•