Closed Bug 376014 Opened 17 years ago Closed 17 years ago

Possible stack overflow in nsDocNavStartProgressListener

Categories

(Toolkit :: Safe Browsing, defect)

Product:
Toolkit
Component:
Safe Browsing
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
Firefox 3 alpha4

People

(Reporter: sciguyryan, Assigned: sciguyryan)

Details

Attachments

(1 file)

Patch v1.0
2.25 KB, patch
tony
: review+
Details | Diff | Splinter Review 
This is a spin-off from bug 371473 comment 17. Thanks to Andrew Miller for reporting this because we could have a stack overflow occur in |nsDocNavStartProgressListener::IsSpurious| in the same way as specified in the above bug.

A possible patch is coming up.
Attached patch Patch v1.0 — — Splinter Review 

        Attachment #260151 -
        Flags: superreview?(tony)

        Attachment #260151 -
        Flags: review?(tony)

    
    

    
  
Comment on attachment 260151 [details] [diff] [review]
Patch v1.0

Thanks for fixing this.

        Attachment #260151 -
        Flags: superreview?(tony)

        Attachment #260151 -
        Flags: superreview+

        Attachment #260151 -
        Flags: review?(tony)

        Attachment #260151 -
        Flags: review+

    
  
Whiteboard: [checkin needed]

    
    

    
  
mozilla/browser/components/safebrowsing/src/nsDocNavStartProgressListener.cpp 1.18

Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: [checkin needed]

    
  
Target Milestone: --- → Firefox 3 alpha4

    
  
Flags: in-testsuite?
Product: Firefox → Toolkit

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: