Bug 3682 - Maps with >1024 models cause a segfault
Status: RESOLVED FIXED
Alias: None
Product: ioquake3
Classification: Unclassified
Component: Misc
Version: GIT MASTER
Hardware: All All
: P3 minor
Assignee: Zachary J. Slater
QA Contact: ioquake3 bugzilla mailing list
URL:
Depends on:
Blocks:
 
Reported: 2008-06-12 15:48 EDT by Ben Noordhuis
Modified: 2008-07-05 19:52:03 EDT
0 users

See Also:

Attachments
Remove assert(), check return value and quit with an error if NULL (497 bytes, patch)
2008-06-12 15:49 EDT, Ben Noordhuis

Description Ben Noordhuis 2008-06-12 15:48:20 EDT 
R_LoadSubmodels() calls R_AllocModel() to allocate a model_t* handle but fails to check the return value. Actually, it does an assert(), but that does precisely nothing in a release build.
Comment 1 Ben Noordhuis 2008-06-12 15:49:58 EDT 
Created attachment 1783 [details]
Remove assert(), check return value and quit with an error if NULL
Comment 2 Tim Angus 2008-07-05 19:52:03 EDT 
Fixed in r1405.