View Issue Details [ Jump to Notes ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
---|---|---|---|---|---|---|---|---|---|
0005479 | Spring engine | General | public | 2017-02-14 14:10 | 2017-02-14 17:29 | ||||
Reporter | Google_Frog | ||||||||
Assigned To | hokomoko | ||||||||
Priority | normal | Severity | crash | Reproducibility | have not tried | ||||
Status | closed | Resolution | duplicate | ||||||
Product Version | 103.0 +git | ||||||||
Target Version | Fixed in Version | ||||||||
Summary | 0005479: 103.0.1-638-g6b1d325 synced vector::_M_range_check crash | ||||||||
Description | These two crashes indicate that both users in a 1v1 crashed with a fatal vector::_M_range_check error: https://github.com/ZeroK-RTS/CrashReports/issues/119 https://github.com/ZeroK-RTS/CrashReports/issues/120 Reports came in about as soon as I set the engine on the server. I was not able to reproduce it locally though. | ||||||||
Tags | No tags attached. | ||||||||
Checked infolog.txt for Errors | |||||||||
Attached Files |
|
![]() |
|
Kloot (developer) 2017-02-14 14:17 |
no traces, but my bet is these originated in GuiHandler like 0005451 and 0005466#c17301. so far I have no idea how to reproduce it. |
hokomoko (developer) 2017-02-14 14:22 Last edited: 2017-02-14 14:23 |
my spring debug build doesn't even launch. I suspect same issue EDIT: wait, no it's FPE |
hokomoko (developer) 2017-02-14 14:50 Last edited: 2017-02-14 14:51 |
bisected to 6cae248c33b353372d01eba1bc97c4a54960dd83 When is LuaParser used in multiple threads? |
Kloot (developer) 2017-02-14 14:56 |
are you talking about the range_check or the FPE (which I don't get)? as for LuaParser: https://github.com/spring/spring/blob/develop/rts/Lua/LuaParser.cpp#L209 |
hokomoko (developer) 2017-02-14 15:04 |
I am talking about the FPE (it happens during engine startup - not game load). The SIGFPE is in ArchiveScanner's MT'd LuaParsers apparently. My suspicion is that a LuaParser constructed on one thread is then required from a different one. I think it may be best to just lock LuaParsers from start to finish. If you don't object I'll implement this. |
hokomoko (developer) 2017-02-14 15:23 |
ok, FPE is because the ScopedDisableFPUExceptions was changed to not contain CheckTableForNaNs |
Kloot (developer) 2017-02-14 15:42 |
hmz, my bad. back to GuiHandler debugging I guess. |
hokomoko (developer) 2017-02-14 15:45 |
I still think the LuaParser currentParser bit should be changed since MT is a bitch |
Kloot (developer) 2017-02-14 16:01 |
seems overkill for now imo, but definitely something to keep in mind. |
hokomoko (developer) 2017-02-14 16:02 |
I kinda already did it, just didn't want to push without asking ^^ |
hokomoko (developer) 2017-02-14 17:29 |
confirmed to be guihandler by local test |
![]() |
|||
Date Modified | Username | Field | Change |
---|---|---|---|
2017-02-14 14:10 | Google_Frog | New Issue | |
2017-02-14 14:17 | Kloot | Note Added: 0017319 | |
2017-02-14 14:22 | hokomoko | Note Added: 0017320 | |
2017-02-14 14:23 | hokomoko | Note Edited: 0017320 | View Revisions |
2017-02-14 14:50 | hokomoko | Note Added: 0017321 | |
2017-02-14 14:51 | hokomoko | Note Edited: 0017321 | View Revisions |
2017-02-14 14:56 | Kloot | Note Added: 0017322 | |
2017-02-14 15:04 | hokomoko | Note Added: 0017323 | |
2017-02-14 15:23 | hokomoko | Note Added: 0017324 | |
2017-02-14 15:42 | Kloot | Note Added: 0017325 | |
2017-02-14 15:45 | hokomoko | Note Added: 0017326 | |
2017-02-14 16:01 | Kloot | Note Added: 0017327 | |
2017-02-14 16:02 | hokomoko | Note Added: 0017328 | |
2017-02-14 17:29 | hokomoko | Note Added: 0017329 | |
2017-02-14 17:29 | hokomoko | Relationship added | duplicate of 0005466 |
2017-02-14 17:29 | hokomoko | Assigned To | => hokomoko |
2017-02-14 17:29 | hokomoko | Status | new => closed |
2017-02-14 17:29 | hokomoko | Resolution | open => duplicate |