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

Status: RESOLVED FIXED

(NSS :: Test, defect)

Description

[Kai Engert [:KaiE:]]

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.

Comment 1

[Kai Engert [:KaiE:]]

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=[e-mail],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.

Comment 2

[Kai Engert [:KaiE:]]

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

Comment 3

[Kai Engert [:KaiE:]]

Attachment: Patch v1

I used this patch (on top of bug 772144) to make ssl.sh compatible with older Android.

Comment 4

[Robert Relyea]

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).

Comment 5

[Kai Engert [:KaiE:]]

> 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

[Ryan VanderMeulen [:RyanVM]]

(Removing the checkin-needed until comment 5 is resolved so it doesn't show up in my search queries in the mean time)

Comment 7

[Kai Engert [:KaiE:]]

I experienced this bug again, with Android 4.2.1

I'll check it in.

Comment 8

[Kai Engert [:KaiE:]]

Checking in ssl.sh;
/cvsroot/mozilla/security/nss/tests/ssl/ssl.sh,v  <--  ssl.sh
new revision: 1.114; previous revision: 1.113
done