Closed Bug 336466 Opened 18 years ago Closed 18 years ago

Coverity oom crash [@ CERT_DecodeAuthKeyID]

Categories

(NSS :: Libraries, defect, P2)

Product:
NSS
Component:
Libraries
Version:
3.11
Platform:
All
Linux
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED
Milestone:
3.11.2

People

(Reporter: timeless, Assigned: alvolkov.bgs)

References

(
URL
)

Details

(Keywords: coverity, crash, Whiteboard: [CID 304 710])

Crash Data

Attachments

(1 file, 1 obsolete file)

same, but less space->tab conversions
1.16 KB, patch
nelson
: review+
wtc
: review+
Details | Diff | Splinter Review 
Description: Pointer "value" dereferenced before NULL check
Attached patch reorder null check (obsolete) — — Splinter Review 
Assignee: nobody → timeless
Status: NEW → ASSIGNED

        Attachment #221174 -
        Flags: review?(nelson)

    
    

    
  
Comment on attachment 221174 [details] [diff] [review]
reorder null check

Sorry, file changes that include lots of gratuitous tab->space conversion will not be acceptable.

        Attachment #221174 -
        Flags: review?(nelson) → review-
Hardware: PC → All
Target Milestone: --- → 3.11.2

    
    

    
  
that's fine. it's not my job to fix your bugs. i provide patches as a courtesy
Assignee: timeless → alexei.volkov.bugs
Status: ASSIGNED → NEW
Priority: -- → P2

    
    

    
  


  

        Attachment #221174 -
        Attachment is obsolete: true

        Attachment #221852 -
        Flags: review?(nelson)

    
    

    
  
Comment on attachment 221852 [details] [diff] [review]
same, but less space->tab conversions

r=nelson

        Attachment #221852 -
        Flags: review?(nelson) → review+

    
    

    
  
truck
/cvsroot/mozilla/security/nss/lib/certdb/xauthkid.c,v  <--  xauthkid.c
new revision: 1.5; previous revision: 1.4
3.11 branch
/cvsroot/mozilla/security/nss/lib/certdb/xauthkid.c,v  <--  xauthkid.c
new revision: 1.4.28.1; previous revision: 1.4

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

    
  

        Attachment #221852 -
        Flags: review+

    
    

    
  
CID 710
Whiteboard: CID 710

    
    

    
  
Also CID 304
Whiteboard: CID 710 → [CID 304 710]

    
  
Summary: oom crash [@ CERT_DecodeAuthKeyID] → Coverity oom crash [@ CERT_DecodeAuthKeyID]
Crash Signature: [@ CERT_DecodeAuthKeyID]

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: