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

TextureCache: Fix crash for invalid textures. #3268

Merged
merged 1 commit into from Nov 18, 2015

Conversation

degasus
Copy link
Member

@degasus degasus commented Nov 17, 2015

No description provided.

@degasus degasus force-pushed the texturecrash branch 2 times, most recently from 179ddcf to ee2223c Compare November 17, 2015 22:28
@delroth
Copy link
Member

delroth commented Nov 17, 2015

Please update the description to make it more clear the crash only happens when recording fifo logs. Otherwise, LGTM.

@JMC47
Copy link
Contributor

JMC47 commented Nov 17, 2015

Fixes our longstanding medabots infinity crash.

@phire
Copy link
Member

phire commented Nov 17, 2015

Does it only crash when recording fifologs?

Perhaps we should be adding more error checking to FIfoRecorder.

@JMC47
Copy link
Contributor

JMC47 commented Nov 17, 2015

No, it crashes in game normally in D3D and OpenGL (but not SW renderer.) The original fix that @degasus posted only fixed the video-backend part, not the fifoplayer part, which is why he had to add a second fix to the fifoplayer.

@delroth
Copy link
Member

delroth commented Nov 18, 2015

OK, I misread that. LGTM then.

delroth added a commit that referenced this pull request Nov 18, 2015
TextureCache: Fix crash for invalid textures.
@delroth delroth merged commit 2a7c039 into dolphin-emu:master Nov 18, 2015
@degasus
Copy link
Member Author

degasus commented Nov 18, 2015

Oh, no, this fixes both, the videocommon part and the fiforecorder, as are broken in the same way here. But my bigger issue with this crash is that it does not happen on videosw. Sounds like the game configures an invalid texture, but don't sample from it. I'm going to verify it if @JMC47 provides me a fifolog.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants