Bug 266235 - GeoNode objects leak
Summary: GeoNode objects leak
Status: CONFIRMED
Alias: None
Product: marble
Classification: Applications
Component: general (show other bugs)
Version: unspecified
Platform: Compiled Sources Linux
: NOR normal
Target Milestone: some future version
Assignee: marble-bugs
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-02-13 22:27 UTC by Michael G. Hansen
Modified: 2021-03-09 22:51 UTC (History)
1 user (show)

See Also:
Latest Commit:
Version Fixed In:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael G. Hansen 2011-02-13 22:27:59 UTC
Version:           unspecified (using Devel) 
OS:                Linux

When running Marble through valgrind, I found that Marble is leaking many GeoNode objects, for example:

==16890== 23,278 (40 direct, 23,238 indirect) bytes in 1 blocks are definitely lost in loss record 4,309 of 4,315
==16890==    at 0x4C24DFA: operator new(unsigned long) (vg_replace_malloc.c:261)
==16890==    by 0x54039A8: Marble::dgml::DgmlPropertyTagHandler::parse(Marble::GeoParser&) const (DgmlPropertyTagHandler.cpp:51)
==16890==    by 0x53E0B19: Marble::GeoParser::parseDocument() (GeoParser.cpp:145)
==16890==    by 0x53E0B93: Marble::GeoParser::parseDocument() (GeoParser.cpp:172)
==16890==    by 0x53E0B93: Marble::GeoParser::parseDocument() (GeoParser.cpp:172)
==16890==    by 0x53E0B93: Marble::GeoParser::parseDocument() (GeoParser.cpp:172)
==16890==    by 0x53E0CCD: Marble::GeoParser::read(QIODevice*) (GeoParser.cpp:94)
==16890==    by 0x547E931: Marble::MapThemeManager::loadMapThemeFile(QString const&) (MapThemeManager.cpp:136)
==16890==    by 0x547EA37: Marble::MapThemeManager::loadMapTheme(QString const&) (MapThemeManager.cpp:119)
==16890==    by 0x547F87F: Marble::MapThemeManager::mapThemes() const (MapThemeManager.cpp:86)
==16890==    by 0x5451EF4: Marble::TileLoader::updateTextureLayers() (TileLoader.cpp:177)
==16890==    by 0x5493D70: Marble::TextureLayer::Private::Private(Marble::MapThemeManager*, Marble::HttpDownloadManager*, Marble::SunLocator*, Marble::TextureLayer*) (TextureLayer.cpp:70)

When I set DUMP_GEONODE_LEAKS to 1 I get:

Found 196895 GeoNode object LEAKS!
marble(16936)/kparts KParts::Part::~Part: deleting widget Marble::ControlView(0x1ef30d0) ""
Found -19 GeoNode object LEAKS!
Found -112 GeoNode object LEAKS!
Found -31 GeoNode object LEAKS!
Found -42 GeoNode object LEAKS!
Found -429 GeoNode object LEAKS!
Found -123 GeoNode object LEAKS!
Found -2301 GeoNode object LEAKS!
Found -193614 GeoNode object LEAKS!
Found -6 GeoNode object LEAKS!
Found -6 GeoNode object LEAKS!

Michael


Reproducible: Always
Comment 1 Michael G. Hansen 2011-02-13 22:29:05 UTC
Somehow the version information went missing, but I get these leaks in both 4.6 and in trunk.

Michael
Comment 2 Justin Zobel 2021-03-09 22:51:29 UTC
Thank you for the bug report.

As this report hasn't seen any changes in 5 years or more, we ask if you can please confirm that the issue still persists.

If this bug is no longer persisting or relevant please change the status to resolved.