Closed Bug 420644 Opened 17 years ago Closed 17 years ago

Improve SSL tracing of key derivation

Categories

(NSS :: Libraries, enhancement, P3)

enhancement

Tracking

(Not tracked)

RESOLVED FIXED
3.12.1

People

(Reporter: nelson, Assigned: nelson)

Details

Attachments

(1 file)

When we converted NSS to use PKCS#11 exclusively for crypto, a lot of the original tracing capability of NSS, done in the context of libSSL, was lost. It was no longer possible to trace key values, because they were not seen outside of the PKCS#11 token. When I implemented SSL Bypass, it became possible (again) to trace the derived key values. I put some SSL tracing back into the bypass functions that derive keys, in lib/ssl/derive.c, but I did a minimal job of it. Now, there's been a request to restore that tracing back to its former glory. The attached patch is a start in that direction. With this patch, I was able to trace an SSL handshake, including key derivation with the following set of shell commands: SSLDEBUGFILE=/tmp/sslTrace.txt SSLBYPASS=1 SSLTRACE=127 SSLDEBUG=127 tstclnt -vvv -2B -h www.microsoft.com -f -c depruvxy < stdin.txt Where stdin.txt is a two-line file containing these lines: ----- two lines are below ------ GET / HTTP/1.0 ---- the two lines are above this one ----
Attachment #306958 - Flags: review?(wtc)
Comment on attachment 306958 [details] [diff] [review] enhance the labeling of SSL key derivation Julien, please review
Attachment #306958 - Attachment description: enhance the labelling of SSL key derivation → enhance the labeling of SSL key derivation
Attachment #306958 - Flags: review?(wtc) → review?(julien.pierre.boogz)
Status: NEW → ASSIGNED
Priority: -- → P3
Attachment #306958 - Flags: review?(julien.pierre.boogz) → review+
Thanks for the review. Checking in lib/ssl/derive.c; new revision: 1.10; previous revision: 1.9
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Target Milestone: 3.12 → 3.12.1
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: