It may happen (and happens on OS X) that plugin activation using
plugin_new() triggers some action which causes the tree view to
update. However, as the old plugin was freed before, the tree view
contains an invalid pointer to p which causes invalid memory access.
After freeing the old pointer, set the tree view value to NULL - the
plugin pointer is checked at other places for NULL value so it
doesn't crash.
You can view, comment on, or merge this pull request online at:
https://github.com/geany/geany/pull/732
-- Commit Summary --
* Avoid possible invalid memory access when activating plugin
-- File Changes --
M src/plugins.c (3)
-- Patch Links --
https://github.com/geany/geany/pull/732.patchhttps://github.com/geany/geany/pull/732.diff
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/732
About compat, how hard/sensible would it be to try the old location if not found in the new one? e.g. "merge" the directories when reading them.
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/pull/485#issuecomment-152851387
Please consider scrolling the <b>Compiler</b> tab to the line holding the error message for the error highlighted in the edit area. Thanks!
---
Reply to this email directly or view it on GitHub:
https://github.com/geany/geany/issues/726