WebKit Bugzilla
New
Browse
Log In
×
Sign in with GitHub
or
Remember my login
Create Account
·
Forgot Password
Forgotten password account recovery
RESOLVED WONTFIX
54955
Theoretical resource memory leaks in AnimationBase.cpp
https://bugs.webkit.org/show_bug.cgi?id=54955
Summary
Theoretical resource memory leaks in AnimationBase.cpp
sangeetha.sugavanam
Reported
2011-02-22 06:41:49 PST
In method ensurePropertyMap() in AnimationBase.cpp, memory is allocated for a Vector of type PropertyWrapperBase. Unfortunately, memory is never deallocated for this. A suggestion in the code says to use a refptr and toss it once we are done using the Vector. Unfortunately, this function is being called by many other methods and because of that destroying using in a refptr in the function itself may lead to crashes. This bug needs to be further investigated.
Attachments
Add attachment
proposed patch, testcase, etc.
Alexey Proskuryakov
Comment 1
2011-02-22 10:42:02 PST
What's the observable effect of this? There is a FIXME in the code, but it doesn't explain why not destroying the wrappers in a problem at all.
Simon Fraser (smfr)
Comment 2
2011-02-22 11:10:32 PST
This would only count as a leak if you wanted to unload the WebCore library and expect all associated memory to go away. I don't believe that WebCore supports being unloaded on any platform.
Simon Fraser (smfr)
Comment 3
2011-02-24 22:04:19 PST
We have many of these in WebCore. We're not going to change this strategy any time soon.
Note
You need to
log in
before you can comment on or make changes to this bug.
Top of Page
Format For Printing
XML
Clone This Bug