Created attachment 55069 [details] Valgrind log of konqueror --sync http://www.abc.net.au/news/ Version: unspecified (using KDE 4.5.85) OS: Linux For KDE 4.5.x (and probably before) and now in the beta's of KDE 4.6 I've been seeing a browser window left open on the Australian ABC News website (http://www.abc.net.au/news/) grow larger and larger in terms of memory used. Running Konqueror under Valgrind with the command line: valgrind --log-file=konqueror-memleak.txt --leak-check=full -v konqueror --sync http://abc.net.au/news/ reveals several memory leaks, the second largest of which is in the KJS/DOM code: ==23441== 205,816 (34,176 direct, 171,640 indirect) bytes in 4,272 blocks are definitely lost in loss record 2,904 of 2,905 ==23441== at 0x4C28973: operator new(unsigned long) (vg_replace_malloc.c:261) ==23441== by 0x1A33FC14: cssyyparse(void*) (parser.cpp:2464) ==23441== by 0x1A31CEEF: DOM::CSSParser::runParser() (cssparser.cpp:151) ==23441== by 0x1A323331: DOM::CSSParser::parseSelectorList(DOM::DocumentImpl*, DOM::DOMString const&) (cssparser.cpp:324) ==23441== by 0x1A22695D: khtml::SelectorQuery::querySelectorImp(bool, DOM::NodeImpl*, DOM::DOMString const&, int&) (wa_selectors.cpp:41) ==23441== by 0x1A226EE3: khtml::SelectorQuery::querySelectorAll(DOM::NodeImpl*, DOM::DOMString const&, int&) (wa_selectors.cpp:106) ==23441== by 0x1A1E8824: DOM::NodeImpl::querySelectorAll(DOM::DOMString const&, int&) (dom_nodeimpl.cpp:1362) ==23441== by 0x1A39847E: DOMDocumentProtoFunc::callAsFunction(KJS::ExecState*, KJS::JSObject*, KJS::List const&) (kjs_dom.cpp:1202) ==23441== by 0x1AE2C857: KJS::Machine::runBlock(KJS::ExecState*, WTF::Vector<unsigned char, 0ul> const&, KJS::ExecState*) (object.h:626) ==23441== by 0x1AE0ADB3: KJS::FunctionImp::callAsFunction(KJS::ExecState*, KJS::JSObject*, KJS::List const&) (function.cpp:172) ==23441== by 0x1AE2C857: KJS::Machine::runBlock(KJS::ExecState*, WTF::Vector<unsigned char, 0ul> const&, KJS::ExecState*) (object.h:626) ==23441== by 0x1AE0ADB3: KJS::FunctionImp::callAsFunction(KJS::ExecState*, KJS::JSObject*, KJS::List const&) (function.cpp:172) I'll attach the full Valgrind log for your info. Reproducible: Always Steps to Reproduce: konqueror http://abc.net.au/news/ and wait, monitor memory usage with the system load viewer plasma widget.. Actual Results: Memory usage increases over time to several hundred MB. Expected Results: Memory usage should be stable.
Forgot to mention (sigh) that KDE bug #225843 (memory leak on http://www.boerse-frankfurt.de/DE/index.aspx?pageID=44&NewsID=4551) could be related as it too mentions the JavaScript code as a potential issue.
Dear Bug Submitter, This bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? I am setting the status to NEEDSINFO pending your response, please change the Status back to REPORTED when you respond. Thank you for helping us make KDE software even better for everyone!
Dear Bug Submitter, This is a reminder that this bug has been stagnant for a long time. Could you help us out and re-test if the bug is valid in the latest version? This bug will be moved back to REPORTED Status for manual review later, which may take a while. If you are able to, please lend us a hand. Thank you for helping us make KDE software even better for everyone!
(In reply to Andrew Crouthamel from comment #3) > Dear Bug Submitter, > > This is a reminder that this bug has been stagnant for a long time. Could > you help us out and re-test if the bug is valid in the latest version? This > bug will be moved back to REPORTED Status for manual review later, which may > take a while. If you are able to, please lend us a hand. > > Thank you for helping us make KDE software even better for everyone! Will do, have just opened the site now, will keep an eye on it. All the best, Chris
(In reply to Chris Samuel from comment #4) > Will do, have just opened the site now, will keep an eye on it. I think this is going to be pretty impossible to debug now. Konqueror has removed the --sync option to prevent it going into the background and the ABC News site now says it doesn't support this browser and gives a very basic HTML view instead. Konqueror has also lost the ability to spoof the browser version. As there's no longer a way to debug it, and I no longer use Konqueror for my everyday browsing I'm going to close this as fixed as there's no closer resolution (it's not invalid because it was a problem, but there's no way to tell if it's still doing it).