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

Null Pointer Crash #1019

Closed
BentleyJOakes opened this issue Jan 12, 2017 · 1 comment
Closed

Null Pointer Crash #1019

BentleyJOakes opened this issue Jan 12, 2017 · 1 comment

Comments

@BentleyJOakes
Copy link
Contributor

@BentleyJOakes BentleyJOakes commented Jan 12, 2017

make sure these boxed are checked before submitting your issue - thank you!

  • Check OS
    • Windows
    • Mac OSX
    • Linux
  • Check MMEX version
  • 1.3.x
  • 1.2.x
  • 1.1 or older

Issue:
Thread 1 "mmex" received signal SIGSEGV, Segmentation fault.
0x000000000074e2a0 in mmDateRange::destroy() ()
(gdb) bt
#0 0x000000000074e2a0 in mmDateRange::destroy() ()
#1 0x0000000000639220 in mmHomePagePanel::getData() (this=0x1128a10) at ../src/mmhomepagepanel.cpp:559
#2 0x0000000000638878 in mmHomePagePanel::createHTML() (this=0x1128a10) at ../src/mmhomepagepanel.cpp:519

Problem Location:
src/mmhomepagepanel.cpp Line 559
date_range_->destroy();

Suggestion:
Add if (date_range_) before this line. This prevents the crash.

Note:
The crash may require having accounts from a previous version of MoneyManagerEx.

BentleyJOakes added a commit to BentleyJOakes/moneymanagerex that referenced this issue Jan 12, 2017
stef145g added a commit that referenced this issue Jan 13, 2017
stef145g added a commit that referenced this issue Jan 13, 2017
@stef145g
Copy link
Contributor

@stef145g stef145g commented Jan 13, 2017

Thanks for the patch.
This has also been added to the v1.3.x branch

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants