cybersphinx uploaded file warzone2100.gdmp-p6I6Uy (28.3 KiB)

cybersphinx uploaded file warzone2100.gdmp-ouIPbh (26.7 KiB)

Giel commented

Strange..., especially considering that all those values at netplay.c:664 have already been evaluated previously on line 650, except for fds, but that's a local variable.

Thus I'm not at all sure what's going on there. Perhaps a bugged FD_SET (shudders)?

Per uploaded file strangefunc.diff (0.5 KiB)

That function looks rather odd. This patch makes it less so.

cybersphinx commented

I tried to reproduce this now, and it didn't fail even once... maybe it was some wrong object file compiled in (though that shouldn't happen), this was after a distclean. Or could it be some server-side problem?

Giel commented

@per: feel free to commit (both trunk and 2.3)

Replying to Warzone2100/old-trac-import#1326 (comment:2):

I tried to reproduce this now, and it didn't fail even once... maybe it was some wrong object file compiled in (though that shouldn't happen), this was after a distclean. Or could it be some server-side problem?

I don't care if it's a server side problem: the client shouldn't crash as a result...

cybersphinx commented

Replying to Warzone2100/old-trac-import#1326 (comment:3):

I don't care if it's a server side problem: the client shouldn't crash as a result...
The question was: Can it be a server side problem? If not, miscompilation gets more likely.

Zarel commented

Considering many people using self-compiled versions of Warzone on several different operating systems have gotten this exact crash, I doubt it's miscompilation.

Per's patch, while a good idea to commit, couldn't possibly have anything to do with this crash, either: In any situation where Per's patch would make a difference, the game would freeze in the while loop before it could crash where it does.

Buginator changed status from new to pending

Buginator commented

All these patches have died in the HD crash...

Buginator commented

(and the crash dumps)

trac-robot changed status from pending to closed

trac-robot commented

This ticket was closed automatically by the system. It was previously set to a Pending status and hasn't been updated within 7 days.