Closed
Bug 777699
Opened 12 years ago
Closed 12 years ago
NSS test "Reloading CRL for group 43 - 48" fails on Android
Categories
(NSS :: Test, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
3.14.2
People
(Reporter: KaiE, Assigned: KaiE)
References
Details
Attachments
(1 file)
1.17 KB,
patch
|
rrelyea
:
review+
|
Details | Diff | Splinter Review |
I'm following the instructions from bug 772144 and test on Android 2.3.7 (CM 7.2.0). I started the tests using make android_run_tests NSS_TESTS=ssl The tests run fine, until it gets stuck at the following test: Reloading CRL for group 43 - 48 tstclnt -p 8443 -h localhost -f -d ../client -v -2 -w nss -n TestUser41 GET crl:///data/data/berserker.android.apps.sshdroid/home/nsstest/tests_results/security/localhost.5/server/root.crl_43-48 RELOAD time tstclnt continues to run, but never succeeds. While it runs, it keeps eating 100% cpu and keeps writing data into a tmpfile (localhost.tmp.5649) until the filesystem is full, but tstclnt keeps running even after that.
Assignee | ||
Comment 1•12 years ago
|
||
The tmp file contents starts with tstclnt: connecting to localhost:8443 (address=127.0.0.1) tstclnt: connect: Operation is still in progress (probably a non-blocking connect) tstclnt: ready... tstclnt: stdin read 124 bytes tstclnt: Writing 124 bytes to server tstclnt: using asynchronous certificate validation tstclnt: handshake was paused by auth certificate hook tstclnt: SSL version 3.1 using 128-bit RC4 with 128-bit MD5 MAC tstclnt: Server Auth: 1024-bit RSA, Key Exchange: 1024-bit RSA Compression: NULL subject DN: CN=localhost,E=localhost@bogus.com,O=BOGUS NSS,L=Mountain View,ST=California,C=US issuer DN: CN=NSS Test CA,O=BOGUS NSS,L=Mountain View,ST=California,C=US 0 cache hits; 1 cache misses, 0 cache not reusable 0 stateless resumes tstclnt: Read from server 0 bytes The remainder are messages about polling.
Assignee | ||
Comment 2•12 years ago
|
||
well, strange, it seems the following syntax doesn't work in the Android shell: command <<_DELIM input _DELIM I rewrote it to start by creating a new temporary file with the input and started using the following command, which works: command < tempfile
Assignee | ||
Comment 3•12 years ago
|
||
I used this patch (on top of bug 772144) to make ssl.sh compatible with older Android.
Assignee: nobody → kaie
Attachment #649322 -
Flags: review?
Assignee | ||
Updated•12 years ago
|
Attachment #649322 -
Flags: review? → review?(rrelyea)
Comment 4•12 years ago
|
||
Comment on attachment 649322 [details] [diff] [review] Patch v1 r+ Though it's weird that << works for cat but not tstclnt. Also, which version of android bash are you running (I remember some bash shells had problems with the tests).
Attachment #649322 -
Flags: review?(rrelyea) → review+
Assignee | ||
Updated•12 years ago
|
Keywords: checkin-needed
Assignee | ||
Comment 5•12 years ago
|
||
> r+ Though it's weird that << works for cat but not tstclnt.
>
> Also, which version of android bash are you running (I remember some bash
> shells had problems with the tests).
This bug only happened with the old Android 2.3.7 version.
I don't know which bash version was used by it.
I no longer use that device for testing.
I no longer require that fix. If you prefer, we could WONTFIX it.
Comment 6•12 years ago
|
||
(Removing the checkin-needed until comment 5 is resolved so it doesn't show up in my search queries in the mean time)
Keywords: checkin-needed
Assignee | ||
Comment 7•12 years ago
|
||
I experienced this bug again, with Android 4.2.1 I'll check it in.
Assignee | ||
Updated•12 years ago
|
Target Milestone: --- → 3.14.2
Assignee | ||
Comment 8•12 years ago
|
||
Checking in ssl.sh; /cvsroot/mozilla/security/nss/tests/ssl/ssl.sh,v <-- ssl.sh new revision: 1.114; previous revision: 1.113 done
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Summary: NSS test "Reloading CRL for group 43 - 48" fails on Android 2.3.7 → NSS test "Reloading CRL for group 43 - 48" fails on Android
You need to log in
before you can comment on or make changes to this bug.
Description
•