strsclnt doesn't support TLS_RSA_WITH_NULL_SHA

RESOLVED FIXED in 3.11.7

Status

P2
normal
RESOLVED FIXED
12 years ago
12 years ago

People

(Reporter: nelson, Assigned: nelson)

Tracking

3.11.4
3.11.7

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

When I added support for TLS_RSA_WITH_NULL_SHA to NSS (bug 161529),
I somehow overlooked strsclnt.  I added the "z" cipher letter to 
tstclnt, selfserv and vfyserv, but not to strsclnt.  That omission
needs to be corrected.

This omission is one of the causes of bug 367037 and bug 367133.
Those bugs should be fixed first, before this one is fixed, so I 
am marking this bug as depending on those two.  

The fix for this bug will be the addition of one line to the array
ssl3CipherSuites in strsclnt.c
(Assignee)

Updated

12 years ago
No longer depends on: 367133
(Assignee)

Comment 1

12 years ago
Created attachment 258491 [details] [diff] [review]
one line patch

This should do it.
Assignee: nobody → nelson
Status: NEW → ASSIGNED
Attachment #258491 - Flags: superreview?(christophe.ravel.bugs)
Attachment #258491 - Flags: review?(slavomir.katuscak)

Updated

12 years ago
Attachment #258491 - Flags: review?(slavomir.katuscak) → review+
(Assignee)

Updated

12 years ago
Attachment #258491 - Flags: superreview?(christophe.ravel.bugs) → superreview?(julien.pierre.boogz)

Updated

12 years ago
Attachment #258491 - Flags: superreview?(julien.pierre.boogz) → superreview+
(Assignee)

Comment 2

12 years ago
Checking in strsclnt.c; new revision: 1.56; previous revision: 1.55
Checking in strsclnt.c; new revision: 1.43.2.9; previous revision: 1.43.2.8
Status: ASSIGNED → RESOLVED
Last Resolved: 12 years ago
Priority: -- → P2
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.