Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix crash in projectorganizer's crash-fix #559

Merged
merged 2 commits into from
Apr 10, 2017

Conversation

techee
Copy link
Member

@techee techee commented Apr 10, 2017

Well, yeah, there was a NULL check missing in the previous pull request. No make it clear what has changed I kept the previous patch and added another patch which fixes the crash when no project is open.

techee added 2 commits April 5, 2017 18:04
…he tree

Since expand_on_idle() is called as an idle function, it can happen it's
executed after project is closed. Before trying to expand paths it now
first checks if the idle call corresponds to the project for which
the idle call was created.

Should fix crash reported in geany#549.
The previous code only checked this for paths shown in the sidebar but
since the plugins also tries to perform the expansion for the current
document when this option is enabled, it needs to be done for this case
as well.

Just leave the function when prj_org == NULL.
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on Ubuntu 16.04 x64
This patch solves #549

@frlan frlan merged commit fd94a04 into geany:master Apr 10, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants