Closed
Bug 1154188
Opened 10 years ago
Closed 10 years ago
security/certverifier/NSSCertDBTrustDomain.cpp:824:3: error: 'UniquePtr' was not declared in this scope (non-SPS)
Categories
(Core :: Security: PSM, defect)
Tracking
()
RESOLVED
FIXED
mozilla40
Tracking | Status | |
---|---|---|
firefox40 | --- | fixed |
People
(Reporter: jbeich, Assigned: jbeich)
References
Details
Attachments
(1 file)
(deleted),
patch
|
briansmith
:
review+
|
Details | Diff | Splinter Review |
SPS has the following bootlegging thanks to bug 1151829
security/certverifier/Unified_cpp_certverifier0.cpp:20:
security/certverifier/NSSCertDBTrustDomain.cpp:15:
security/certverifier/../manager/ssl/src/nsNSSCertificate.h:14:
security/certverifier/../manager/ssl/src/nsNSSShutDown.h:11:
dist/include/mozilla/CondVar.h:16:
dist/include/GeckoProfiler.h:204:
dist/include/GeckoProfilerImpl.h:16:
dist/include/mozilla/UniquePtr.h
on platforms lacking SPS profiler (Linux/PPC, BSDs, Solaris) it leads to
In file included from security/certverifier/Unified_cpp_certverifier0.cpp:20:
security/certverifier/NSSCertDBTrustDomain.cpp:824:3: error:
use of undeclared identifier 'UniquePtr'
UniquePtr<char, void(&)(char*)>
^
security/certverifier/NSSCertDBTrustDomain.cpp:824:17: error:
expected '(' for function-style cast or type construction
UniquePtr<char, void(&)(char*)>
~~~~^
security/certverifier/NSSCertDBTrustDomain.cpp:824:25: error:
expected expression
UniquePtr<char, void(&)(char*)>
^
security/certverifier/NSSCertDBTrustDomain.cpp:824:31: error:
expected '(' for function-style cast or type construction
UniquePtr<char, void(&)(char*)>
~~~~^
security/certverifier/NSSCertDBTrustDomain.cpp:824:32: error:
expected expression
UniquePtr<char, void(&)(char*)>
^
security/certverifier/NSSCertDBTrustDomain.cpp:825:5: error:
use of undeclared identifier 'fullLibraryPath'
fullLibraryPath(PR_GetLibraryName(dir, "nssckbi"), PR_FreeLibraryName);
^
security/certverifier/NSSCertDBTrustDomain.cpp:826:8: error:
use of undeclared identifier 'fullLibraryPath'
if (!fullLibraryPath) {
^
...
inb4 bug 903843 is done more regularly
https://treeherder.mozilla.org/#/jobs?repo=try&revision=151d80ff94c5
Attachment #8592125 -
Flags: review?(brian)
Comment 2•10 years ago
|
||
FFS... failures are just piling on top of each other. Can ppl stop messing up with those damn headers ?
Comment 3•10 years ago
|
||
Comment on attachment 8592125 [details] [diff] [review]
explicit #include
Review of attachment 8592125 [details] [diff] [review]:
-----------------------------------------------------------------
::: security/certverifier/NSSCertDBTrustDomain.cpp
@@ +12,3 @@
> #include "ExtendedValidation.h"
> #include "OCSPRequestor.h"
> #include "certdb.h"
Please put the include of mozilla/UniquePtr.h here (alphabetical order)
Attachment #8592125 -
Flags: review?(brian) → review+
Comment 4•10 years ago
|
||
Comment 5•10 years ago
|
||
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla40
You need to log in
before you can comment on or make changes to this bug.
Description
•