Bug 212866 - [patch] games/7kaa Segmentation fault on exit
Summary: [patch] games/7kaa Segmentation fault on exit
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Dmitry Marakasov
URL:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2016-09-20 20:29 UTC by kd-dev
Modified: 2016-10-12 09:01 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (amdmi3)

Attachments
Fixes the issue by calling audio.deinit before exiting. (2.16 KB, patch)
2016-09-20 20:29 UTC, kd-dev 		no flags Details | Diff
Fix from upstream. (3.95 KB, patch)
2016-10-11 19:37 UTC, kd-dev 		no flags Details | Diff
Show Obsolete (1) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description kd-dev 2016-09-20 20:29:58 UTC 
Created attachment 175017 [details]
Fixes the issue by calling audio.deinit before exiting.

A C++ deconstructor attempts to clean-up OpenAL resources at exit after
the library's deconstructor destroys the internal state.  The called
functions attempt to access the destroyed state causing the program to
exit abnormally.

To reproduce run the port then exit it.

The added patches moves the "deinit" calls out of the deconstructor and
into "SYS::deinit", which is called before exit.
Comment 1 Dmitry Marakasov freebsd_committer freebsd_triage 2016-09-21 13:22:23 UTC 
I'd prefer this to be reported to and reviewed by upstream.
Comment 2 kd-dev 2016-09-21 22:03:07 UTC 
(In reply to Dmitry Marakasov from comment #1)
It appears that this bug is FreeBSD specific because of the order
in which destructors are called.

FreeBSD:
C Binary         Libraries
Libraries        C++ Binary

Linux:
C Binary         C++ Binary
Libraries        Libraries
Comment 3 Dmitry Marakasov freebsd_committer freebsd_triage 2016-10-04 15:08:00 UTC 
(In reply to kdrakehp from comment #2)
> (In reply to Dmitry Marakasov from comment #1)
> It appears that this bug is FreeBSD specific because of the order
> in which destructors are called.
> 
> FreeBSD:
> C Binary         Libraries
> Libraries        C++ Binary
> 
> Linux:
> C Binary         C++ Binary
> Libraries        Libraries

It is not FreeBSD specific. Static data init/deinit order is not specified, so this is an upstream problem. Please report it upstream, it they are ok with the fix, we can add it to the port right away.
Comment 4 kd-dev 2016-10-11 19:37:59 UTC 
Created attachment 175636 [details]
Fix from upstream.

The issue was fixed upstream (https://github.com/the3dfxdude/7kaa/pull/70)
Comment 5 commit-hook freebsd_committer freebsd_triage 2016-10-12 09:01:29 UTC 
A commit references this bug:

Author: amdmi3
Date: Wed Oct 12 09:01:18 UTC 2016
New revision: 423836
URL: https://svnweb.freebsd.org/changeset/ports/423836

Log:
  - Fix segfault on exit [1]
  - Fix license

  PR:		212866 [1]
  Submitted by:	kdrakehp@zoho.com [1]

Changes:
  head/games/7kaa/Makefile
  head/games/7kaa/files/
  head/games/7kaa/files/patch-src_audio_openal_openal__audio.cpp
  head/games/7kaa/files/patch-src_client_OMUSIC.cpp
  head/games/7kaa/files/patch-src_client_OSYS.cpp
  head/games/7kaa/files/patch-src_video_sdl_vga__sdl.cpp