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

possibly memory corruption #1793

Closed
mrexodia opened this issue Nov 3, 2017 · 2 comments
Closed

possibly memory corruption #1793

mrexodia opened this issue Nov 3, 2017 · 2 comments
Labels
bug

Comments

@mrexodia
Copy link
Member

@mrexodia mrexodia commented Nov 3, 2017

Yesterday I was debugging some programs and after restarting I saw that the status label stayed stuck on Initializing. At first it didn't seem to impact anything, but pretty soon after that other things started breaking as well.

My steps to reproduce (yesterday, today it all works fine obviously):

  • Load some debuggee
  • Hold step for some time
  • Press restart

There are two things that could happen:

  1. The label stays stuck on initializing
  2. The label stays stuck on paused (much more rare)

The first case means that TaskThread.WakeUp doesn't wake up the thread (ReleaseSemaphore fails with an invalid handle error)

The second case means that WaitForSingleObject in TaskThread.Loop always instantly returns and keeps executing the relevant function (GuiSetDebugState) with whatever the last value was (paused).

I tried running under appverifier, but that just hung the process so nothing useful came out of that for me.

@mrexodia
Copy link
Member Author

@mrexodia mrexodia commented Nov 4, 2017

Some progress:

Duncan (mrexodia), [04.11.17 19:25]
MOTHERFUCK

Duncan (mrexodia), [04.11.17 19:25]
@Nukem9 you on?

Duncan (mrexodia), [04.11.17 19:26]
because this shit is crazy

Duncan (mrexodia), [04.11.17 19:26]
the TaskThread didn't get memory corrupted, but somehow CloseHandle is called

Duncan (mrexodia), [04.11.17 19:26]
at first I had a named semaphore with handle 0x624

Duncan (mrexodia), [04.11.17 19:26]
then reloading a bunch of times to trigger the bug

Duncan (mrexodia), [04.11.17 19:27]
https://i.imgur.com/7Z20nM3.png

Duncan (mrexodia), [04.11.17 19:27]
I just smell kernel bugs
@mrexodia mrexodia changed the title possibly memory corruption (please help!) possibly memory corruption Nov 4, 2017
@mrexodia mrexodia closed this Nov 4, 2017
@mrexodia
Copy link
Member Author

@mrexodia mrexodia commented Nov 4, 2017

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
1 participant
You can’t perform that action at this time.