NSS test "Reloading CRL for group 43 - 48" fails on Android

RESOLVED FIXED in 3.14.2

Status

NSS
Test
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: kaie, Assigned: kaie)

Tracking

3.14
3.14.2
ARM
Android

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 years ago
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

5 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

5 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

5 years ago
Created attachment 649322 [details] [diff] [review]
Patch v1

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

5 years ago
Attachment #649322 - Flags: review? → review?(rrelyea)

Comment 4

5 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

5 years ago
Keywords: checkin-needed
(Assignee)

Comment 5

5 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.
(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

5 years ago
I experienced this bug again, with Android 4.2.1

I'll check it in.
(Assignee)

Updated

5 years ago
Target Milestone: --- → 3.14.2
(Assignee)

Comment 8

5 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
Last Resolved: 5 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.