diff options
author | nelsonb%netscape.com <devnull@localhost> | 2000-09-28 01:55:11 +0000 |
---|---|---|
committer | nelsonb%netscape.com <devnull@localhost> | 2000-09-28 01:55:11 +0000 |
commit | f29cb17602fc9ec2803098993d9e3d58462ffcfc (patch) | |
tree | 8ea5c083ca539a6856e177b0c974f8bbfe45bef8 | |
parent | 2a762e27b84208e42504e1068b89829886fe6fe8 (diff) | |
download | nss-hg-f29cb17602fc9ec2803098993d9e3d58462ffcfc.tar.gz |
Keep NSS_30_BRANCH in sync with trunk. Fix PR_Connect timeout crash.
-rw-r--r-- | security/nss/cmd/strsclnt/strsclnt.c | 11 |
1 files changed, 8 insertions, 3 deletions
diff --git a/security/nss/cmd/strsclnt/strsclnt.c b/security/nss/cmd/strsclnt/strsclnt.c index 059a0696e..71ebf718b 100644 --- a/security/nss/cmd/strsclnt/strsclnt.c +++ b/security/nss/cmd/strsclnt/strsclnt.c @@ -717,8 +717,8 @@ do_connects( { PRNetAddr * addr = (PRNetAddr *) a; PRFileDesc * model_sock = (PRFileDesc *) b; - PRFileDesc * ssl_sock; - PRFileDesc * tcp_sock; + PRFileDesc * ssl_sock = 0; + PRFileDesc * tcp_sock = 0; PRStatus prStatus; SECStatus result; int rv = SECSuccess; @@ -748,6 +748,7 @@ retry: goto retry; } errWarn("PR_Connect"); + rv = SECFailure; goto done; } @@ -771,7 +772,11 @@ retry: } done: - PR_Close(ssl_sock); + if (ssl_sock) { + PR_Close(ssl_sock); + } else if (tcp_sock) { + PR_Close(tcp_sock); + } return SECSuccess; } |