-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/124561/
-----------------------------------------------------------
Review request for Marble.
Repository: marble
Description
-------
Qt's plugin loading mechanism is unable to load plugins from the assets virtual
file system they're bundled in (via the .apk). The current workaround in
PluginManager is to copy the files from assets:/ (the system plugin path) to
the local plugin path. A similar approach is done by Qt internally. Our app's
private storage is organized as follows:
apps/org.kde.marble.maps/
f/.local/share/marble/maps/earth/ // Downloaded map tiles etc
f/.marble/plugins // Marble plugins
r/qt-reserved-files/plugins // Qt plugins
r/qt-reserved-files/qml // QML plugins including QtQuick
Android controls
r/qt-reserved-files/android-style // Graphics resources
I can't tell how the r/ files are handled during package upgrades (can anyone
help out?), but the ones in f/ stay untouched. Therefore we can end up in the
following scenario:
- Marble Maps version n installs plugin foo
- Marble Maps version n+1 does not ship foo anymore, or has foo renamed bar
- User updates from n to n+1. f/ files stay intact, so foo stays around
- User starts n+1, foo is loaded
This can have all sorts of undesired effects, from having foo around although
we don't want it to crashes or random runtime behavior due to ABI changes.
The patch fixes this by generating a whitelist of plugins shipped in the
current version, then refusing to load any other plugins it encounters. Given
the path structure outlined above it should be safe to delete such orphane
plugins even (currently not implemented, just a comment in the code).
Diffs
-----
src/lib/marble/PluginManager.cpp 5cc47b2
Diff: https://git.reviewboard.kde.org/r/124561/diff/
Testing
-------
checking qDebug output on a Nexus 4 with adb logcat.
Thanks,
Dennis Nienhüser
_______________________________________________
Marble-devel mailing list
[email protected]
https://mail.kde.org/mailman/listinfo/marble-devel