Closed Bug 337345 Opened 18 years ago Closed 18 years ago

Leak in nsCrypto::GenerateCRMFRequest (security/manager/ssl/src/nsCrypto.cpp)

Categories

(Core Graveyard :: Security: UI, defect)

Product:
Core Graveyard
Component:
Security: UI
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: kherron+mozilla, Assigned: KaiE)

References

(
URL
)

Details

(Keywords: coverity, fixed1.8.1, Whiteboard: [good first bug])

Attachments

(1 file)

Patch v1
708 bytes, patch
rrelyea
: review+
darin.moz
: approval1.8.1+
Details | Diff | Splinter Review 
This is coverity issue 576. Please refer to the sample URL. At lines 1584-1586, the |keyids| array allocated at line 1552 is leaked.
Whiteboard: [good first bug]
Attached patch Patch v1 — — Splinter Review 

        Attachment #221503 -
        Flags: review?(rrelyea)

    
    

    
  
Comment on attachment 221503 [details] [diff] [review]
Patch v1

r+ = rrelyea

        Attachment #221503 -
        Flags: review?(rrelyea) → review+

    
    

    
  
fixed on trunk

Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED

    
    

    
  
Comment on attachment 221503 [details] [diff] [review]
Patch v1

should this leak fix land on the 1.8 branch?

        Attachment #221503 -
        Flags: approval1.8.1?

    
    

    
  
Comment on attachment 221503 [details] [diff] [review]
Patch v1

a=darin on behalf of drivers

        Attachment #221503 -
        Flags: approval1.8.1? → approval1.8.1+

    
    

    
  
fixed1.8.1
Keywords: fixed1.8.1
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: