Closed Bug 191417 Opened 22 years ago Closed 20 years ago

clean up TLS intolerant "restart" code

Categories

(Core Graveyard :: Security: UI, defect)

Product:
Core Graveyard
Component:
Security: UI
Version:
Other Branch
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
psm2.4

People

(Reporter: darin.moz, Assigned: darin.moz)

Details

Attachments

(1 file)

v1 patch
1.68 KB, patch
KaiE
: review+
bryner
: superreview+
Details | Diff | Splinter Review 
If a site is found to be TLS intolerant, nsNSSIOLayer will signal necko to
restart the connection.  If detected on PR_Read, nsNSSIOLayer returns 0;
however, if detected on PR_Write, nsNSSIOLayer returns -1 with
PR_CONNECT_RESET_ERROR.  it should really set PR_CONNECT_RESET_ERROR in both cases.
Attached patch v1 patch — — Splinter Review 

    
  

        Attachment #113177 -
        Flags: review?(kaie)

    
  

        Attachment #113177 -
        Flags: superreview?(bryner)

    
    

    
  
Comment on attachment 113177 [details] [diff] [review]
v1 patch

r=kaie

        Attachment #113177 -
        Flags: review?(kaie) → review+

        Attachment #113177 -
        Flags: superreview?(bryner) → superreview+

    
  
Status: NEW → ASSIGNED
Target Milestone: --- → 2.4

    
    

    
  
Fix checked in.

Status: ASSIGNED → RESOLVED
Closed: 20 years ago
Resolution: --- → FIXED

    
    

    
  
thank you jgmyers!  i completely forgot about this patch.  my normal bug queries
only cover mozilla milestones :-/

    
  
Product: PSM → Core
Product: Core → Core Graveyard

          You need to log in
          before you can comment on or make changes to this bug.
        






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: