Textures.cpp: improved td_pulObjects/td_ulObject fix

icculus / Serious-Engine

An open source version of a game engine developed by Croteam for the classic Serious Sam games.

GNU General Public License v2.0

165 stars 23 forks source link

Closed DanielGibson closed 8 years ago

DanielGibson commented 8 years ago

td_pulObjects is explicitly set to NULL in the constructor - before only td_ulObject was set to NONE (0), so on 64bit half of td_pulObjects bytes would remain garbage
only check td_ulObject for NONE if td_ctFrames <= 1 (until now it would frequently check it for NONE even if td_ctFrames > 1, if td_pulObjects was != NULL)

This should be a bit more robust, I hope it fixes the crash @rohit-n mentioned in #26

rohit-n commented 8 years ago

This fixed the startup segfault on my end, thanks.

DanielGibson commented 8 years ago

cool, thanks for testing!