|Summary:||devel/simgear version 2016.3.1 can cause flightgear to crash with assertion failure when changing radio frequencies|
|Product:||Ports & Packages||Reporter:||russo|
|Component:||Individual Port(s)||Assignee:||Ganael LAPLANCHE <martymac>|
|Severity:||Affects Only Me||CC:||russo|
Description russo 2016-10-20 17:18:56 UTC
Created attachment 175987 [details] Patch combining simgear git commits b9deeb and a97c14 A known bug in the release version of the simgear 2016.3.1 can lead to flightgear crashing with an assertion failure when the pilot changes a comm radio frequency or silences the radio. A forum thread describing the issue is at https://sourceforge.net/p/flightgear/mailman/message/35388214/ The bug has been fixed in the simgear project's git repo, but until the next release that leaves FreeBSD's port somewhat broken. To reproduce, one has to have flightgear set up so that it starts up with the radios turned on and receiving ATIS information. If you then turn the volume knob to zero or click the "swap frequency" button on the com1 radio, the code exits with the assertion failure. I have applied the changes made in simgear's git commits b9deeb and a97c14, and found that it fixes the problem with flightgear. These commits may be found at https://sourceforge.net/p/flightgear/simgear/ci/b9deebb59d872169eb164101b83b6ba957583ee2/ and https://sourceforge.net/p/flightgear/simgear/ci/a97c145f56233f2b4df633c13e5767d3a1b8617b/. The attached patch, if dropped into devel/simgear/files will fix the problem until the next release of simgear and flightgear, which won't have the bug.
Comment 1 russo 2016-10-20 17:20:36 UTC
I should note that because the simgear port produces only static libraries, after the patch is applied and the port rebuilt, one also has to rebuild and reinstall flightgear to get the fix.
Comment 2 commit-hook 2016-10-24 10:13:58 UTC
A commit references this bug: Author: martymac Date: Mon Oct 24 10:13:50 UTC 2016 New revision: 424558 URL: https://svnweb.freebsd.org/changeset/ports/424558 Log: Backport radio bug patch from upstream PR: 213650 Submitted by: firstname.lastname@example.org Changes: head/devel/simgear/Makefile head/devel/simgear/files/ head/devel/simgear/files/patch-simgear-sound-soundmgr_openal.cxx head/games/flightgear/Makefile
Comment 3 Ganael LAPLANCHE 2016-10-24 10:17:54 UTC
Hi, Thanks a lot for your patch, it has been committed with a few changes as the patch provided didn't match exactly the changes brought by b9deeb and a97c14: the return statement should be outside the #ifdef/#endif block. Regards, Ganael.
Comment 4 russo 2016-10-24 13:40:09 UTC
Egad. I saw that the return statement was supposed to be outside the ifdef/endif (it is the entire point of commit a97c14), and then goofed myself when applying the change by hand to produce my patch. I'm sorry it wasn't correct. But thanks for catching that and merging the change.
Comment 5 Ganael LAPLANCHE 2016-10-24 17:00:19 UTC
No pb! Thanks to you :)