0000641: Crash when changing true color mode

Forum rules
Please don't bump threads here if you have a problem - it will often be forgotten about if you do. Instead, make a new thread here.

Post a reply

Smilies
:D :) :( :o :shock: :? 8-) :lol: :x :P :oops: :cry: :evil: :twisted: :roll: :wink: :geek: :ugeek: :!: :?: :idea: :arrow: :| :mrgreen: :3: :wub: >:( :blergh:
View more smilies

BBCode is OFF
Smilies are ON

Topic review
   

Expand view Topic review: 0000641: Crash when changing true color mode

Re: 0000641: Crash when changing true color mode

by Rachael » Sun Apr 30, 2017 2:38 pm

Technically, this bug is actually very old, it's just that having a menu on the vid_renderer and an option that forces a video reset right next to it just brought it to light. You could reproduce it in ancient versions of GZDoom simply by changing vid_renderer in the console and then selecting a new video mode.

Enjay wrote:For what it's worth, I've just made myself a build with the fix and changed the options a dozen or more times: no crash for me.
Thank you! :) - I suppose if no one else reports any problems with this, you should be clear to do a release, Graf. I'd wait for the CI's to finish building, first, though, just to be sure.

Re: 0000641: Crash when changing true color mode

by Enjay » Sun Apr 30, 2017 2:25 pm

Rachael wrote:I would advise against that. This fix definitely needs to be tested.
For what it's worth, I've just made myself a build with the fix and changed the options a dozen or more times: no crash for me.

Re: 0000641: Crash when changing true color mode

by leodoom85 » Sun Apr 30, 2017 2:20 pm

You can check this one better...at the end of the video, the crash happens. This one's the 32-bit version

Re: 0000641: Crash when changing true color mode

by Rachael » Sun Apr 30, 2017 2:09 pm

I would advise against that. This fix definitely needs to be tested.

I basically took out any reference to "vid_renderer" that was not directly in the hardware code. So far it seems to be working.

Addressed in this commit.

Re: 0000641: Crash when changing true color mode

by Graf Zahl » Sun Apr 30, 2017 12:49 pm

Once this is fixed I'll do a new release.

Re: 0000641: Crash when changing true color mode

by Rachael » Sun Apr 30, 2017 12:39 pm

Reopened this one, then - I will address it when I have some extra time. Ultimately, maybe it might be better to compare "vid_renderer" and "currentrenderer" and latch vid_renderer before changing the mode.

Re: 0000641: Crash when changing true color mode

by leodoom85 » Sun Apr 30, 2017 12:37 pm

Also, that bug works in the 32-bit version but probably it was fixed too :)

Re: 0000641: Crash when changing true color mode

by Enjay » Sun Apr 30, 2017 12:36 pm

Rachael wrote:Figures someone would find this bug, and of all people you would be the one. :)
Federal Network Newsreel announcer (Starship troopers) wrote:Every day, Federal scientists are looking for new ways to kill bugs.
I'm doing my part! ;)

Re: 0000641: Crash when changing true color mode

by Chris » Sun Apr 30, 2017 12:35 pm

The crash still occurs even with this commit. There's problems when you go the other way too, BTW. If you start with software rendering, change to opengl, then toggle the truecolor option, the window seems to reinitialize using opengl rendering, but if you try to start a game it's all messed up with only the HUD being drawn.

gzdoom-crash.7z (7,341 bytes)

Re: 0000641: Crash when changing true color mode

by Rachael » Sun Apr 30, 2017 12:34 pm

Re: 0000641: Crash when changing true color mode

by Rachael » Sun Apr 30, 2017 12:34 pm

Figures someone would find this bug, and of all people you would be the one. :)

0000641: Crash when changing true color mode

by Enjay » Sun Apr 30, 2017 12:34 pm

https://mantis.zdoom.org/view.php?id=641

Code: Select all

Summary	0000641: Crash when changing true color mode
Description	I can reliably get a crash in 3.0.0 64 bit when changing to software output and switching true color mode off.
Steps To Reproduce	Start game in OpenGL mode (you can start an actual game or just do it from the title screen)
Go to the change rendering output menu
Change hardware acceleration to Doom software
Change Software true color to off
Crash
https://mantis.zdoom.org/file_download. ... 5&type=bug

Top