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

Access violation crash #83

Closed
tmewett opened this issue May 10, 2020 · 3 comments
Closed

Access violation crash #83

tmewett opened this issue May 10, 2020 · 3 comments
Labels
bug Something isn't working

Comments

@tmewett
Copy link
Owner

tmewett commented May 10, 2020

From tinyrodent:

Just had the game crash, first time in a while. CE 1.8.2, Windows 7 64-bit. Just before this happened, I detonated a large cloud of explosive gas - unsure if that was related.

Problem signature:
 Problem Event Name:   APPCRASH
 Application Name: brogue.exe
 Application Version:  0.0.0.0
 Application Timestamp:    5e88a453
 Fault Module Name:    msvcrt.dll
 Fault Module Version: 7.0.7601.17744
 Fault Module Timestamp:   4eeb033f
 Exception Code:   c0000005
 Exception Offset: 000000000000716e
 OS Version:   6.1.7601.2.1.0.768.3
 Locale ID:    1033
 Additional Information 1: e402
 Additional Information 2: e40203178aaf3f352719c02485be6bce
 Additional Information 3: 233d
 Additional Information 4: 233dae01d946a591b7f3271c3753b5eb

Exception code 0xc0000005 is apparently access violation/invalid memory access. The error occurs at PC 0x000000000000716e into msvcrt.dll. I don't really know how to find out what function this is in.

@tmewett tmewett added the bug Something isn't working label May 10, 2020
@ghost
Copy link

ghost commented Sep 29, 2020

@tmewett This could be due to out-of-bounds errors in PowerTables.c. I was debugging with the sanitizer and got this error during some intense in-game action:

src/brogue/PowerTables.c:117:12: runtime error: index 204 out of bounds for type 'fixpt [200]'

The functions in this file are not all correctly guarded, e.g POW_REGEN[bonus] with bonus = -10 ...

@ghost
Copy link

ghost commented Sep 29, 2020

@tmewett I'm working on a bunch of bugfixes, including PowerTables.c, here.

@tmewett
Copy link
Owner Author

tmewett commented Jan 28, 2021

Going to assume this was the same as #221, now fixed.

@tmewett tmewett closed this as completed Jan 28, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant