162 Commits

Author SHA1 Message Date
kaie%kuix.de
f1f6b33c41 Bug 542832 - SSL_RestartHandshakeAfterServerCert is broken, patch by Brian Smith, r=rrelyea
git-svn-id: svn://10.0.0.236/trunk@263386 18797224-902f-48f8-a5cc-f745e15eee43
2012-02-11 13:03:08 +00:00
kaie%kuix.de
a80bdaa107 Bug 542832 - SSL_RestartHandshakeAfterServerCert is broken, patch by Brian Smith, r=rrelyea
git-svn-id: svn://10.0.0.236/trunk@263385 18797224-902f-48f8-a5cc-f745e15eee43
2012-02-11 12:58:48 +00:00
kaie%kuix.de
f0bd270bae Bug 542832 - SSL_RestartHandshakeAfterServerCert is broken, patch by Brian Smith, r=rrelyea
git-svn-id: svn://10.0.0.236/trunk@263384 18797224-902f-48f8-a5cc-f745e15eee43
2012-02-11 12:58:47 +00:00
kaie%kuix.de
9d33576904 Bug 542832 - SSL_RestartHandshakeAfterServerCert is broken, patch by Brian Smith, r=rrelyea
git-svn-id: svn://10.0.0.236/trunk@263382 18797224-902f-48f8-a5cc-f745e15eee43
2012-02-11 12:55:58 +00:00
bsmith%mozilla.com
9b720db9f7 Bug 700516: Remove SSL2 support for SSL_RestartHandshakeAfterServerCert and SSL_RestartHandshakeAfterCertReq, r=rrelyea
git-svn-id: svn://10.0.0.236/trunk@263089 18797224-902f-48f8-a5cc-f745e15eee43
2011-11-19 21:58:21 +00:00
bsmith%mozilla.com
24363aa2e6 Bug 651523: Remove step-up code, v4, removing EXPORT_VERSION build option, r=rrelyea
git-svn-id: svn://10.0.0.236/trunk@263080 18797224-902f-48f8-a5cc-f745e15eee43
2011-11-17 00:20:22 +00:00
kaie%kuix.de
51ddd562f1 Fix Tinderbox bustage, backout all patches from 2011-11-11
git-svn-id: svn://10.0.0.236/trunk@263077 18797224-902f-48f8-a5cc-f745e15eee43
2011-11-16 19:12:36 +00:00
bsmith%mozilla.com
4778bbc800 Bug 700516: Remove SSL2 support for SSL_RestartHandshakeAfterServerCert and SSL_RestartHandshakeAfterCertReq, r=rrelyea
git-svn-id: svn://10.0.0.236/trunk@263067 18797224-902f-48f8-a5cc-f745e15eee43
2011-11-11 19:06:52 +00:00
bsmith%mozilla.com
7a0edbfd6a Bug 651523 - Remove SSL step-up code from libssl and remove step-up extended key usage support, r=rrelyea
git-svn-id: svn://10.0.0.236/trunk@263066 18797224-902f-48f8-a5cc-f745e15eee43
2011-11-11 18:47:20 +00:00
bsmith%mozilla.com
0601ca68ad Bug 547312: Implement client-side support for NPN; original patch by agl r=wtc; changes by bsmith r=agl
git-svn-id: svn://10.0.0.236/trunk@263024 18797224-902f-48f8-a5cc-f745e15eee43
2011-10-29 00:29:11 +00:00
bsmith%mozilla.com
2e2793892a Bug 665814: Prevent chosen plaintext attacks on SSL 3.0 and TLS 1.0 connections, r=wtc, sr=rrelyea
git-svn-id: svn://10.0.0.236/trunk@262945 18797224-902f-48f8-a5cc-f745e15eee43
2011-10-01 03:59:54 +00:00
wtc%google.com
638601ef20 Bug 673477: Expose the error code set by CERT_ExtractPublicKey in
ssl3_VerifySignedHashes and ssl3_SendClientKeyExchange.  r=wtc.


git-svn-id: svn://10.0.0.236/trunk@262531 18797224-902f-48f8-a5cc-f745e15eee43
2011-07-26 02:13:37 +00:00
wtc%google.com
0f73ee0fe1 Bug 616757: in ssl3_SendCertificateVerify, we must destroy
ss->ssl3.clientPrivateKey for all key exchange algorithms, otherwise we
will send a Certificate message in renegotiation even if the renegotiation
doesn't request client auth.  Move the cleanup of clientCertChain and
clientPrivateKey from ssl3_HandleCertificateRequest to
ssl3_HandleServerHello as a second defense.  The patch is contributed by
Ryan Sleevi <ryan.sleevi@gmail.com>.  r=wtc.


git-svn-id: svn://10.0.0.236/trunk@261791 18797224-902f-48f8-a5cc-f745e15eee43
2011-01-22 02:25:06 +00:00
wtc%google.com
b41a67ad55 Bug 606049: Add the SSL peer's CA certificates to ss->ssl3.peerCertChain in
the correct order.  r=nelson.


git-svn-id: svn://10.0.0.236/trunk@261441 18797224-902f-48f8-a5cc-f745e15eee43
2010-10-21 17:31:36 +00:00
wtc%google.com
e1d2b6d850 Bug 587234: SSL_ERROR_WEAK_SERVER_KEY is renamed
SSL_ERROR_WEAK_SERVER_EPHEMERAL_DH_KEY to be specific about key type.
TBR=kaie.
Modified Files:
	cmd/lib/SSLerrs.h lib/ssl/ssl3con.c lib/ssl/sslerr.h


git-svn-id: svn://10.0.0.236/trunk@261157 18797224-902f-48f8-a5cc-f745e15eee43
2010-09-01 19:43:48 +00:00
nelson%bolyard.com
d36b5e1607 Bug 586697 - ssl3_DeriveMasterSecret must not request pVersion when it does
Master key derivation for Diffie-Hellman through pkcs11.
Patch contributed by Alexei Volkov <alexei.volkov.bugs@sun.com>, r=nelson


git-svn-id: svn://10.0.0.236/trunk@261124 18797224-902f-48f8-a5cc-f745e15eee43
2010-08-28 18:22:07 +00:00
wtc%google.com
dc58ba1b0e Bug 588698: Add assertions to discover and enforce current locking order.
Based on patch contributed by Adam Langley of Google <agl@chromium.org>.
r=agl,nelson.
Modified Files:
	notes.txt ssl3con.c sslcon.c sslimpl.h sslsecur.c


git-svn-id: svn://10.0.0.236/trunk@261113 18797224-902f-48f8-a5cc-f745e15eee43
2010-08-28 00:56:10 +00:00
wtc%google.com
07feacb99e Bug 587234: Better error reporting for tiny DH keys in Server Key Exchange.
Add a new error code SSL_ERROR_WEAK_SERVER_KEY for the dh_p size check in
ssl3_HandleServerKeyExchange.  r=nelson.
Modified Files:
	cmd/lib/SSLerrs.h lib/ssl/ssl3con.c lib/ssl/sslerr.h


git-svn-id: svn://10.0.0.236/trunk@261049 18797224-902f-48f8-a5cc-f745e15eee43
2010-08-16 18:19:02 +00:00
wtc%google.com
2dc38c772f Bug 586470: Add 'const' to SEC_DerSignData and ssl3_UpdateHandshakeHashes.
Remove PK11_ImportPrivateKey.  r=emaldona.
Modified Files:
	cryptohi/cryptohi.h cryptohi/secsign.c pk11wrap/pk11pk12.c
	ssl/ssl3con.c


git-svn-id: svn://10.0.0.236/trunk@261027 18797224-902f-48f8-a5cc-f745e15eee43
2010-08-12 01:15:38 +00:00
wtc%google.com
1e99b8cb20 Bug 525092: Support TLS false start. The patch is contributed by Adam
Langley of Google <agl@chromium.org>.  r=wtc.
Modified Files:
	cmd/strsclnt/strsclnt.c cmd/tstclnt/tstclnt.c lib/ssl/ssl.h
	lib/ssl/ssl3con.c lib/ssl/ssl3gthr.c lib/ssl/sslimpl.h
	lib/ssl/sslsecur.c lib/ssl/sslsock.c tests/ssl/sslstress.txt


git-svn-id: svn://10.0.0.236/trunk@260919 18797224-902f-48f8-a5cc-f745e15eee43
2010-07-30 03:00:17 +00:00
wtc%google.com
2be3505cb6 Bug 571797: do not check block cipher padding if decryption failed. The
patch is contributed by Brian Smith <brian@briansmith.org>.  r=wtc.


git-svn-id: svn://10.0.0.236/trunk@260569 18797224-902f-48f8-a5cc-f745e15eee43
2010-06-24 19:53:20 +00:00
nelson%bolyard.com
4cf2a89eea Bug 571797: NSS should not send the decryption_failed alert
Patch contributed by Brian Smith <brian@briansmith.org>, r=nelson


git-svn-id: svn://10.0.0.236/trunk@260550 18797224-902f-48f8-a5cc-f745e15eee43
2010-06-24 09:24:18 +00:00
nelson%bolyard.com
536cc6effe Bug 571796: ssl3_HandleRecord should check all the padding bytes
Patch contributed by Brian Smith <brian@briansmith.org>, r=nelson


git-svn-id: svn://10.0.0.236/trunk@260486 18797224-902f-48f8-a5cc-f745e15eee43
2010-06-18 06:25:42 +00:00
nelson%bolyard.com
7286c7e079 Bug 562434: SSL_ForceHandshake returns SECSuccess if the peer sends us an application data record
git-svn-id: svn://10.0.0.236/trunk@260432 18797224-902f-48f8-a5cc-f745e15eee43
2010-06-06 22:30:02 +00:00
nelson%bolyard.com
a04e310e21 Bug 554354: SSL client doesn't validate ECDH params from server, r=rrelyea
git-svn-id: svn://10.0.0.236/trunk@260138 18797224-902f-48f8-a5cc-f745e15eee43
2010-04-03 19:06:23 +00:00
nelson%bolyard.com
c0b5e7332d Bug 415565: Use new NSPR atomic macros in NSS
Patch contributed by Steve Snyder <swsnyder@snydernet.net>, r=wtc


git-svn-id: svn://10.0.0.236/trunk@260135 18797224-902f-48f8-a5cc-f745e15eee43
2010-04-03 18:27:33 +00:00
wtc%google.com
88282f31d8 Bug 537356: Redefine SSL_RENEGOTIATE_CLIENT_ONLY as
SSL_RENEGOTIATE_TRANSITIONAL, changing its meaning for server sockets,  and
make it the default.  r=rrelyea.
Modified Files:
	ssl.h ssl3con.c sslsock.c


git-svn-id: svn://10.0.0.236/trunk@259722 18797224-902f-48f8-a5cc-f745e15eee43
2010-02-17 02:29:08 +00:00
wtc%google.com
7438b8e95a Bug 537356: Rename SCSV. In the final RFC, the symbolic name of the SCSV
changed to TLS_EMPTY_RENEGOTIATION_INFO_SCSV.  r=christophe,rrelyea.
Modified Files:
	cmd/ssltap/ssltap.c lib/ssl/ssl.h lib/ssl/ssl3con.c
	lib/ssl/sslproto.h


git-svn-id: svn://10.0.0.236/trunk@259715 18797224-902f-48f8-a5cc-f745e15eee43
2010-02-16 18:56:48 +00:00
wtc%google.com
13cd60e59c Bug 275744: Add new error code SSL_ERROR_RX_UNEXPECTED_UNCOMPRESSED_RECORD
when we detect missing compression.  The patch is contributed by Adam
Langley <agl@chromium.org>.  r=nelson,wtc.
Modified Files:
	cmd/lib/SSLerrs.h lib/ssl/ssl3con.c lib/ssl/sslerr.h


git-svn-id: svn://10.0.0.236/trunk@259586 18797224-902f-48f8-a5cc-f745e15eee43
2010-02-03 03:44:29 +00:00
wtc%google.com
d49496f0ff Bug 537356: Disable the ECC cipher suites for SSL 3.0 renegotiations
because we don't send the elliptic_curves and ec_point_format extensions
in SSL 3.0 client hello.  r=nelson.


git-svn-id: svn://10.0.0.236/trunk@259579 18797224-902f-48f8-a5cc-f745e15eee43
2010-02-03 02:46:15 +00:00
wtc%google.com
3f6d220c28 Bug 537356: server-side SSL_RENEGOTIATE_REQUIRES_XTN code should handle
both client-initiated (ss->ssl3.hs.ws == idle_handshake) and
server-initiated (ss->ssl3.hs.ws == wait_client_hello) renegotiations.
r=kaie,nelson.


git-svn-id: svn://10.0.0.236/trunk@259506 18797224-902f-48f8-a5cc-f745e15eee43
2010-01-29 19:55:42 +00:00
wtc%google.com
fa46d39e9d Bug 537356: In SSL 3.0, send and handle the renegotiation_info extension
but not any other extension.  r=rrelyea.
Modified Files:
	ssl3con.c ssl3ext.c


git-svn-id: svn://10.0.0.236/trunk@259505 18797224-902f-48f8-a5cc-f745e15eee43
2010-01-29 19:20:28 +00:00
kaie%kuix.de
bd4c4b9fa4 Bug 540304, Implement SSL_HandshakeNegotiatedExtension
r=nelson


git-svn-id: svn://10.0.0.236/trunk@259501 18797224-902f-48f8-a5cc-f745e15eee43
2010-01-28 16:14:25 +00:00
nelson%bolyard.com
0bc55de11a Bug 537356: Implement new safe SSL3 & TLS renegotiation, r=wtc
git-svn-id: svn://10.0.0.236/trunk@259500 18797224-902f-48f8-a5cc-f745e15eee43
2010-01-28 06:19:13 +00:00
wtc%google.com
66dfd7adba Bug 536474: Add support for logging pre-master secrets. The patch is
contributed by Adam Langley <agl@chromium.org>.  r=nelson,wtc.
Modified Files:
	ssl3con.c sslimpl.h sslsock.c


git-svn-id: svn://10.0.0.236/trunk@259455 18797224-902f-48f8-a5cc-f745e15eee43
2010-01-22 03:47:42 +00:00
alexei.volkov.bugs%sun.com
d26b36b737 360421 - Implement TLS Server Name Indication for servers. r=nelson
git-svn-id: svn://10.0.0.236/trunk@259396 18797224-902f-48f8-a5cc-f745e15eee43
2010-01-14 22:15:26 +00:00
wtc%google.com
0dd48d5002 Bug 531188: Don't call zlib with 0-length inputs (the avail_in is 0),
otherwise zlib returns Z_BUF_ERROR, which we consider fatal.  The patch is
contributed by Adam Langley of Google <agl@chromium.org>.  r=wtc,nelson.


git-svn-id: svn://10.0.0.236/trunk@259136 18797224-902f-48f8-a5cc-f745e15eee43
2009-12-01 17:59:46 +00:00
wtc%google.com
61c9cc167c Bug 531188: ssl3_DeflateDecompress should set an error code on failure.
Remove an extraneous ssl_ReleaseSpecReadLock(ss) call.  r=agl,nelson.


git-svn-id: svn://10.0.0.236/trunk@259090 18797224-902f-48f8-a5cc-f745e15eee43
2009-11-26 01:51:10 +00:00
wtc%google.com
a75ee146fb Bug 275744: Fix a compilation error on Windows. Need to declare variables
at the beginning of a block.  TBR=nelson.


git-svn-id: svn://10.0.0.236/trunk@259051 18797224-902f-48f8-a5cc-f745e15eee43
2009-11-21 05:31:16 +00:00
wtc%google.com
6159af71f0 Bug 275744: Back out the previous solution of undefining the compress macro
after including zlib.h.  Rename compress to compressor and decompress to
decompressor to avoid the compress macro that may be defined by zconf.h.
r=nelson.
Modified Files:
	ssl3con.c sslimpl.h


git-svn-id: svn://10.0.0.236/trunk@259050 18797224-902f-48f8-a5cc-f745e15eee43
2009-11-21 03:40:49 +00:00
wtc%google.com
7b748542b5 Bug 275744: Make sure the compression of resumed sessions is correct. The
patch is contributed by Adam Langley of Google <agl@chromium.org>.
r=wtc,nelson.


git-svn-id: svn://10.0.0.236/trunk@259049 18797224-902f-48f8-a5cc-f745e15eee43
2009-11-21 03:32:40 +00:00
wtc%google.com
0615d94be7 Bug 275744: Mozilla's zconf.h defines 'compress' as a macro. When NSS is
built as part of Mozilla, the 'compress' member of the ssl3CipherSpec
structure gets renamed.  Undefine 'compress' to avoid that.  r=nelson.


git-svn-id: svn://10.0.0.236/trunk@258982 18797224-902f-48f8-a5cc-f745e15eee43
2009-11-12 05:08:27 +00:00
wtc%google.com
c0ec7fbfae Bug 275744: In client hello, list compression methods in the order of
desirability so that servers that simply pick the first mutually supported
compression method will pick the best compression method.  Add compression
method info to the SSLChannelInfo structure.  Rename SSL3CompressionMethod
to SSLCompressionMethod and add the ssl_ prefix to the enum constants.
Remove an extra comma in strsclnt.c that breaks the concatenation of two
string literals.  r=agl,rrelyea,nelson.
Modified Files:
	cmd/selfserv/selfserv.c cmd/strsclnt/strsclnt.c
	cmd/tstclnt/tstclnt.c lib/ssl/ssl3con.c lib/ssl/ssl3ext.c
	lib/ssl/ssl3prot.h lib/ssl/sslimpl.h lib/ssl/sslinfo.c
	lib/ssl/sslsnce.c lib/ssl/sslt.h tests/ssl/sslstress.txt


git-svn-id: svn://10.0.0.236/trunk@258919 18797224-902f-48f8-a5cc-f745e15eee43
2009-11-07 18:23:06 +00:00
nelson%bolyard.com
089608da06 Bug 526689: (CVE-2009-3555) SSL3 & TLS Renegotiation Vulnerability
Disable SSL 3.x renegotiation by default.  Add new options to re-enable.
r=wtc,rrelyea


git-svn-id: svn://10.0.0.236/trunk@258888 18797224-902f-48f8-a5cc-f745e15eee43
2009-11-06 20:11:29 +00:00
wtc%google.com
7904b372b3 Bug 275744: Implement TLS compression RFC 3749. Add the SSL_ENABLE_DEFLATE
SSL option and the -z command-line option for tstclnt, strsclnt, and
selfserv for enabling the DEFLATE compression method.  The patch is
contributed by Adam Langley <agl@chromium.org> of Google.  r=nelson.
Modified Files:
	cmd/selfserv/selfserv.c cmd/strsclnt/strsclnt.c
	cmd/tstclnt/tstclnt.c lib/ssl/Makefile lib/ssl/ssl.h
	lib/ssl/ssl3con.c lib/ssl/ssl3prot.h lib/ssl/sslerr.h
	lib/ssl/sslimpl.h lib/ssl/sslsock.c


git-svn-id: svn://10.0.0.236/trunk@258862 18797224-902f-48f8-a5cc-f745e15eee43
2009-11-04 17:19:25 +00:00
wtc%google.com
e8653a0f01 Bug 522580: Add the NSS_SecureMemcmp function that performs a constant-time
compare of two memory regions, and use it in libSSL for comparing secret
data.  The patch is contributed by Adam Langley <agl@chromium.org> of
Google.  r=wtc,nelson.
Modified Files:
	lib/ssl/ssl3con.c lib/ssl/sslcon.c lib/ssl/sslgathr.c
	lib/util/nssutil.def lib/util/secport.c lib/util/secport.h


git-svn-id: svn://10.0.0.236/trunk@258699 18797224-902f-48f8-a5cc-f745e15eee43
2009-10-16 17:45:36 +00:00
nelson%bolyard.com
d049b9bded Bug 481216: Fix specific spelling errors in NSS & NSPR, r=timeless
git-svn-id: svn://10.0.0.236/trunk@256449 18797224-902f-48f8-a5cc-f745e15eee43
2009-03-04 21:57:28 +00:00
nelson%bolyard.com
d2e7f8c5a8 Bug 441321: Tolerate incorrect encoding of DSA signatures in SSL 3.0 handshakes
r=wtc.


git-svn-id: svn://10.0.0.236/trunk@256213 18797224-902f-48f8-a5cc-f745e15eee43
2009-02-13 05:23:23 +00:00
nelson%bolyard.com
3bf81e175c Bug 453234: Support for SEED Cipher Suites to TLS RFC 4010
patch by Yeonjung Kang <kang.yeonjung@gmail.com>, r=nelson


git-svn-id: svn://10.0.0.236/trunk@255547 18797224-902f-48f8-a5cc-f745e15eee43
2008-12-17 06:09:22 +00:00
wtc%google.com
997b195396 Bug 426266: Decoupled lib/softoken and lib/ssl. lib/softoken should not
include any headers from lib/ssl.  r=nelson.
Modified Files:
	softoken/pkcs11c.c ssl/manifest.mn ssl/ssl3con.c


git-svn-id: svn://10.0.0.236/trunk@254368 18797224-902f-48f8-a5cc-f745e15eee43
2008-09-22 23:47:00 +00:00