2019-12-09 05:54 CET

View Issue Details Jump to Notes ] Related Changesets ]
IDProjectCategoryView StatusLast Update
0005165Spring engineGeneralpublic2016-03-12 10:55
Reporterhokomoko 
Assigned To 
PrioritynormalSeverityminorReproducibilitysometimes
StatusresolvedResolutionfixed 
Product Version101.0+git 
Target VersionFixed in Version 
Summary0005165: Null pointer dereference in ModelParser
DescriptionWhen ModelParser is safe deleted, if the preload thread is still running it will fail on reaching https://github.com/spring/spring/blob/87e6ba1/rts/Rendering/Models/IModelParser.cpp#L134 since the pointer is nullptr.

My solution for that would've been putting the queue handling in the modelparser itself (as I originally did).

Kloot I have a suspicion that you won't like it as you split it out of the ModelParser in the first place.
Additional Informationhttp://buildbot.springrts.com/builders/validationtests/builds/5277/steps/validation%20test/logs/stdio
TagsNo tags attached.
Checked infolog.txt for Errors
Attached Files

-Relationships
+Relationships

-Notes

~0016047

Anonymous (viewer)

Fix 3a75f32292b097885575bb157d4d1ba65bc9b7ad committed to develop branch: fix 0005165
change modelParser from global pointer to function-static reference
s/modelParser/modelLoader to match the class name, repo: spring changeset id: 6820
+Notes

+Related Changesets

-Issue History
Date Modified Username Field Change
2016-03-12 07:45 hokomoko New Issue
2016-03-12 10:55 Changeset attached => spring develop 3a75f322
2016-03-12 10:55 Anonymous Note Added: 0016047
2016-03-12 10:55 Anonymous Status new => resolved
2016-03-12 10:55 Anonymous Resolution open => fixed
+Issue History