Closed
Bug 1432790
Opened 7 years ago
Closed 7 years ago
Sandbox MinGW Compilation error: undefined reference to `_ReturnAddress'
Categories
(Core :: Security: Process Sandboxing, defect)
Core
Security: Process Sandboxing
Tracking
()
RESOLVED
FIXED
mozilla60
Tracking | Status | |
---|---|---|
firefox60 | --- | fixed |
People
(Reporter: tjr, Assigned: tjr)
References
Details
(Whiteboard: [tor][mingw-upstream-pending])
Attachments
(1 file)
In Bug 1431621 I thought I had fixed it by calling mingw's _ReturnAddress() function, but this caused an error on the linking step (below). _ReturnAddress() was just a #define in MinGW to __builtin_return_address - so I'm just going to restore the original code but fix the stray \ [task 2018-01-24T00:44:34.562Z] 00:44:34 INFO - ../../security/sandbox/process_handle_win.o: In function `ZN4base3win19GenericScopedHandleINS0_12HandleTraitsENS0_14VerifierTraitsEE3SetEPv': [task 2018-01-24T00:44:34.562Z] 00:44:34 INFO - /builds/worker/workspace/build/src/security/sandbox/chromium/base/win/scoped_handle.h:70: undefined reference to `_ReturnAddress' [task 2018-01-24T00:44:34.562Z] 00:44:34 INFO - ../../security/sandbox/process_handle_win.o: In function `ZN4base3win19GenericScopedHandleINS0_12HandleTraitsENS0_14VerifierTraitsEE5CloseEv': [task 2018-01-24T00:44:34.563Z] 00:44:34 INFO - /builds/worker/workspace/build/src/security/sandbox/chromium/base/win/scoped_handle.h:95: undefined reference to `_ReturnAddress' (This does compile btw)
Assignee | ||
Updated•7 years ago
|
Summary: Sandbox MinGW Compilation error: → Sandbox MinGW Compilation error: undefined reference to `_ReturnAddress'
Comment hidden (mozreview-request) |
Comment hidden (mozreview-request) |
Comment 3•7 years ago
|
||
mozreview-review |
Comment on attachment 8945080 [details] Bug 1432790 Fix MinGW Sandbox linking error about not being able to find _ReturnAddress() https://reviewboard.mozilla.org/r/215302/#review220878 ::: security/sandbox/chromium-shim/patches/with_update/mingw_base_win_get_caller.patch:7 (Diff revision 2) > -Bug 1431621 Remove the gcc variant of BASE_WIN_GET_CALLER, because MinGW defines _ReturnAddress() ?rbobowen > +Bug XXXX Fix a stray \ in MinGW code > > -Note that MinGW defines it without __builtin_extract_return_addr which > -means we're dropping that, but the gcc documentation indicates that > +This is a bastardized pseudo-commit of Bug 1431621 and Bug 1432790. > +It removes a stray \ in the code that causes MinGW to not compile. Please make this just be a commit message (and patch) as if it had been done like this from the start for bug 1431621. ::: security/sandbox/chromium-shim/patches/with_update/mingw_base_win_get_caller.patch:19 (Diff revision 2) > MozReview-Commit-ID: 4D5bIT9Fei4 > > diff --git a/security/sandbox/chromium/base/win/scoped_handle.h b/security/sandbox/chromium/base/win/scoped_handle.h > --- a/security/sandbox/chromium/base/win/scoped_handle.h > +++ b/security/sandbox/chromium/base/win/scoped_handle.h > @@ -1,31 +1,26 @@ @@ -13,17 +13,17 @@ ::: security/sandbox/chromium-shim/patches/with_update/mingw_base_win_get_caller.patch:20 (Diff revision 2) > // Copyright (c) 2012 The Chromium Authors. All rights reserved. > // Use of this source code is governed by a BSD-style license that can be > // found in the LICENSE file. > > #ifndef BASE_WIN_SCOPED_HANDLE_H_ > #define BASE_WIN_SCOPED_HANDLE_H_ > > -+#include <intrin.h> > #include <windows.h> > > #include "base/base_export.h" > #include "base/gtest_prod_util.h" > #include "base/location.h" Delete these ... ::: security/sandbox/chromium-shim/patches/with_update/mingw_base_win_get_caller.patch:50 (Diff revision 2) > // the ScopedFILE class with two additions: > // - IsValid() method can tolerate multiple invalid handle values such as NULL ... and these.
Attachment #8945080 -
Flags: review?(bobowencode) → review+
Comment hidden (mozreview-request) |
Assignee | ||
Comment 5•7 years ago
|
||
Okay I think I corrected it. I generated the patch file using real hg commands instead of editing it manually, but I guess I'd prefer you to double check it before I commit it.
Flags: needinfo?(bobowencode)
Assignee | ||
Updated•7 years ago
|
Keywords: checkin-needed
Pushed by ryanvm@gmail.com: https://hg.mozilla.org/integration/autoland/rev/112289b0c784 Fix MinGW Sandbox linking error about not being able to find _ReturnAddress() r=bobowen
Keywords: checkin-needed
Comment 8•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/112289b0c784
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox60:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla60
You need to log in
before you can comment on or make changes to this bug.
Description
•