Closed
Bug 1320932
Opened 9 years ago
Closed 9 years ago
Typo in $NSS_DISABLE_LIBPKIX check in tests/ssl/ssl.sh
Categories
(NSS :: Test, defect)
NSS
Test
Tracking
(Not tracked)
RESOLVED
FIXED
3.29
People
(Reporter: ueno, Unassigned)
Details
Attachments
(1 file)
|
657 bytes,
patch
|
KaiE
:
review+
|
Details | Diff | Splinter Review |
Hubert noticed that, after bug 1227795, running ssl.sh causes a syntax error in shell script:
./ssl.sh: line 1100: [: -nz: unary operator expected
I suppose it is a typo of -z?
| Reporter | ||
Updated•9 years ago
|
Attachment #8815242 -
Flags: review?(franziskuskiefer)
Comment 1•9 years ago
|
||
Comment on attachment 8815242 [details] [diff] [review]
typo-fix.patch
I think you're right.
r=kaie
Attachment #8815242 -
Flags: review?(franziskuskiefer) → review+
Comment 2•9 years ago
|
||
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.29
Comment 3•9 years ago
|
||
That hasn't worked, it triggered new failures.
But instead of a failure with the "ssl.sh" test, we get a new failure with the "chains.sh" test:
chains.sh: Verifying certificate(s) PayPalEE.cert with flags -d AllDB -pp -o OID.2.16.840.1.114412.1.1
vfychain -d AllDB -pp -vv -o OID.2.16.840.1.114412.1.1 /home/worker/nss/tests/libpkix/certs/PayPalEE.cert
Chain is bad!
PROBLEM WITH THE CERT CHAIN:
CERT 0. PayPalEE :
ERROR -8181: Peer's Certificate has expired.
Returned value is 1, expected result is pass
chains.sh: #4883: RealCerts: Verifying certificate(s) PayPalEE.cert with flags -d AllDB -pp -o OID.2.16.840.1.114412.1.1 - FAILED
This test calls vfychain with option "-pp" which will cause it to use API CERT_PKIXVerifyCert, which is available only when building with the libpkix library enabled.
The test scripts should be changed to skip any tests with "vfychain -pp" when libpkix is disabled.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Comment 4•9 years ago
|
||
backed out until we have a better fix:
https://hg.mozilla.org/projects/nss/rev/5e7b5e3d301d
Comment 5•9 years ago
|
||
I was wrong.
The reason for the test failure is: The "real world paypal certificate", that is checked in to NSS and that we are using during our tests, has expired today. I'll fix that with bug 1323978.
That means, we can reland the patch from this bug.
Comment 6•9 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 9 years ago → 9 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•