#128: r6206 introduces a segfault and its fix isn't required anymore -> revert
suggested
---------------------------+------------------------------------------------
Reporter: Giel | Owner: Buginator
Type: defect | Status: new
Priority: blocker | Milestone: 2.1
Component: Engine: Sound | Version: 2.1
Keywords: | Operating_system: All
---------------------------+------------------------------------------------
r6206 is intended to fix problems caused by not unmounting /data/ due to
the music residing in there. r6215 addressed this problem in another
fashion: i.e. simply moving the music out of that directory.
Thus r6206 isn't required anymore, while at the same time it introduces a
new bug: when later on sound_DestroyStream is called (and that ''will''
happen) we'll try to close the filehandle (that by now has become NULL).
In addition we shouldn't close the filehandle ''before'' getting rid of
the decoder (by destroying it).
But since all of this only becomes a problem due to r6206, which in its
entirety isn't required anymore, I suggest we revert that revision (and
r6208 as well, which is a partial bugfix for r6206).
--
Ticket URL: <http://developer.wz2100.net/ticket/128>
Warzone 2100 Trac <http://wz2100.net/>
The Warzone 2100 Resurrection Project
_______________________________________________
Warzone-dev mailing list
[email protected]
https://mail.gna.org/listinfo/warzone-dev
