Bug 108724 – Xbomber-101 Segfaults on selection options

• Product: Gentoo Linux
• Status: RESOLVED FIXED

[Description David Pyke 2005-10-10 06:27:06 UTC]

When I select single player options, the game crashes with "Segmentation Fault",
Additionally, selecting remap keys has no effect (nothing happens)

Reproducible: Always
Steps to Reproduce:
1.start game
2.select single player options
3.boom!

Actual Results:  
Crashed to desktop with segmentation fault on command line

Expected Results:  
Shown the options page?

Portage 2.0.51.22-r3 (default-linux/amd64/2005.1, gcc-3.4.4, glibc-2.3.5-r2,
2.6.12-gentoo-r10 x86_64)
=================================================================
System uname: 2.6.12-gentoo-r10 x86_64 AMD Opteron(tm) Processor 142
Gentoo Base System version 1.6.13
ccache version 2.4 [enabled]
dev-lang/python:     2.3.5-r2
sys-apps/sandbox:    1.2.12
sys-devel/autoconf:  2.13, 2.59-r6
sys-devel/automake:  1.4_p6, 1.5, 1.6.3, 1.7.9-r1, 1.8.5-r3, 1.9.6-r1
sys-devel/binutils:  2.15.92.0.2-r10
sys-devel/libtool:   1.5.18-r1
virtual/os-headers:  2.6.11-r2
ACCEPT_KEYWORDS="amd64"
AUTOCLEAN="yes"
CBUILD="x86_64-pc-linux-gnu"
CFLAGS="-O2 -march=opteron -pipe"
CHOST="x86_64-pc-linux-gnu"
CONFIG_PROTECT="/etc /usr/kde/2/share/config /usr/kde/3/share/config
/usr/lib/X11/xkb /usr/lib64/mozilla/defaults/pref /usr/share/config
/var/qmail/control"
CONFIG_PROTECT_MASK="/etc/gconf /etc/terminfo /etc/env.d"
CXXFLAGS="-O2 -march=opteron -pipe"
DISTDIR="/usr/portage/distfiles"
FEATURES="autoconfig ccache distlocks sandbox sfperms strict"
GENTOO_MIRRORS="http://gentoo.osuosl.org/ http://mirrors.acm.cs.rpi.edu/gentoo/"
LDFLAGS="-Wl,-O1"
MAKEOPTS="-j2"
PKGDIR="/usr/portage/packages"
PORTAGE_TMPDIR="/var/tmp"
PORTDIR="/usr/portage"
PORTDIR_OVERLAY="/usr/local/portage"
SYNC="rsync://rsync.gentoo.org/gentoo-portage"
USE="amd64 X aac alsa apache2 audiofile avi bdf berkdb bidi bitmap-fonts bmp
bonobo browserplugin bzip2 cdparanoia cdr crypt css cups curl dvd dvdr dvdread
eds emboss encode esd evo fbcon flac foomaticdb fortran ftp gd gif gnome gpm
gstreamer gtk gtk2 gtkhtml guile hal icq imagemagick imlib ipv6 java jpeg lcms
libwww lm_sensors lzw lzw-tiff mad maildir mbox mime mozilla moznoirc mozsvg mp3
mpeg mplayer msn nas ncurses nls nptl nsplugin nvidia offensive ogg oggvorbis
openal opengl oscar pam pcre pda pdflib perl php plotutils png ppds python qt
quicktime readline samba sdl sndfile snmp soap sockets sox speex spell ssl tcltk
tcpd theora tidy tiff truetype truetype-fonts type1-fonts usb userlocales v4l
videos vorbis wifi wxwindows xine xml xml2 xpm xv xvid zlib userland_GNU
kernel_linux elibc_glibc"
Unset:  ASFLAGS, CTARGET, LANG, LC_ALL, LINGUAS

[Comment 1 SpanKY 2005-10-10 07:59:18 UTC]

dont waste our time filing bugs for packages which arent even KEYWORDed on your
architecture

[Comment 2 David Pyke 2005-10-10 11:42:26 UTC]

10 Oct 2005; Luis Medinas (metalgod) xbomber-101.ebuild: Added ~amd64 keyword. 

We're not supposed to report bugs on things keyworded for testing?

[Comment 3 Chris Gianelloni (RETIRED) 2005-10-10 12:09:41 UTC]

...

[Comment 4 Chris Gianelloni (RETIRED) 2005-10-10 12:11:46 UTC]

amd64 team... can you verify that this package actually works for you, as you
added the keywords...

[Comment 5 SpanKY 2005-10-10 13:00:44 UTC]

sorry, my tree was out of date, it didnt have an amd64 KEYWORD ;)

[Comment 6 Mike Cvet 2005-10-10 15:16:50 UTC]

Created attachment 70324 [details, diff]
Fixes vsprintf statement

Sorry, I apologize, I asked for marking ~amd64 testing just the game and not
the settings - this is a patch that fixes the crash.

[Comment 7 Mike Cvet 2005-10-10 15:22:42 UTC]

Comment on attachment 70324 [details, diff]
Fixes vsprintf statement

*sigh* Fixes the crash for options menu, but won't save config info... so
nevermind =/

[Comment 8 Danny van Dyk (RETIRED) 2005-11-06 08:14:22 UTC]

Ok, here are two things to do:

a) Fix the segfault. This is easily done by using correct arguments for all
vsprintf calls in the source. Patch already in Portage.

b) Fix the "Remap Movement Keys" thing. Undoable, as it isn't coded yet (it
seems). Have a look at the menu code from domode0() function in bomber.c:

                additem("EXIT GAME");
                additem("START SINGLE PLAYER GAME");
                additem("SINGLE PLAYER OPTIONS");
                additem("REMAP MOVEMENT KEYS");
                additem("START NETWORK GAME");
                additem("JOIN NETWORK GAME");
                sel=domenu(0);
                if(!sel) {exitflag=1;break;}
                if(sel==1) {gamemode=1;break;}
                if(sel==2) {gamemode=2;break;}
                if(sel==4) {gamemode=3;break;}
                if(sel==5) {gamemode=4;break;}

'sel' corresponds to the selection, and there is no item for sel==3, which would
be the remap item.

Games: Is there any active upstream for this? If yes, can you guys relay the patch?