Closed Bug 419096 Opened 16 years ago Closed 16 years ago

getPathSegAtLength() causes nsSVGPathSegMovetoAbs to leak

Categories

(Core :: SVG, defect)

Product:
Core
Component:
SVG
Type:
defect
Priority:
Not set
Severity:
minor

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla1.9beta4

People

(Reporter: jruderman, Assigned: longsonr)

References

Details

(Keywords: memory-leak, testcase)

Attachments

(2 files)

testcase
148 bytes, image/svg+xml
Details
patch
1.19 KB, patch
roc
: review+
roc
: superreview+
damons
: approval1.9+
Details | Diff | Splinter Review
Attached image testcase — 
Steps to reproduce:
1. Run a debug build of Firefox with XPCOM_MEM_LEAK_LOG=2.
2. Load the testcase.
3. Quit.

Result: trace-refcnt reports that a nsSVGPathSegMovetoAbs has leaked.
Attached patch patch — — Splinter Review 
Assignee: nobody → longsonr
Status: NEW → ASSIGNED

        Attachment #305182 -
        Flags: superreview?(roc)

        Attachment #305182 -
        Flags: review?(roc)

    
  
Assignee: longsonr → nobody
Status: ASSIGNED → NEW
Flags: blocking1.9?

        Attachment #305182 -
        Flags: superreview?(roc)

        Attachment #305182 -
        Flags: superreview+

        Attachment #305182 -
        Flags: review?(roc)

        Attachment #305182 -
        Flags: review+

        Attachment #305182 -
        Flags: approval1.9?

    
    

    
  
Comment on attachment 305182 [details] [diff] [review]
patch

a1.9+=damons

        Attachment #305182 -
        Flags: approval1.9? → approval1.9+

    
  
Assignee: nobody → longsonr
Keywords: checkin-needed

    
    

    
  
Checking in content/svg/content/src/nsSVGPathElement.cpp;
/cvsroot/mozilla/content/svg/content/src/nsSVGPathElement.cpp,v  <--  nsSVGPathElement.cpp
new revision: 1.42; previous revision: 1.41
done
Status: NEW → RESOLVED
Closed: 16 years ago
Keywords: checkin-needed
OS: Mac OS X → All
Hardware: PC → All
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9beta4

    
  
Flags: in-testsuite?
Flags: blocking1.9?

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






  

    
  







  

    

      
Attachment

      

      
      
      
      
    

    

      

        

          

            
General

            

              Creator: 



            

            
Created: 

            
Updated: 

            
Size: