Hello community, here is the log from the commit of package bomberclone checked in at Fri Mar 16 14:45:55 CET 2007. -------- --- bomberclone/bomberclone.changes 2006-09-29 10:21:08.000000000 +0200 +++ /mounts/work_src_done/STABLE/bomberclone/bomberclone.changes 2007-03-13 16:47:02.000000000 +0100 @@ -1,0 +2,6 @@ +Tue Mar 13 16:44:57 CET 2007 - pgajdos@suse.cz + +- fixed: 'warning: array subscript is above array bounds' (#242031) +- array_subscript.patch + +------------------------------------------------------------------- New: ---- bomberclone-0.11.7_array-subscript.patch ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Other differences: ------------------ ++++++ bomberclone.spec ++++++ --- /var/tmp/diff_new_pack.YY8740/_old 2007-03-16 14:45:16.000000000 +0100 +++ /var/tmp/diff_new_pack.YY8740/_new 2007-03-16 14:45:16.000000000 +0100 @@ -1,7 +1,7 @@ # # spec file for package bomberclone (Version 0.11.7) # -# Copyright (c) 2006 SUSE LINUX Products GmbH, Nuernberg, Germany. +# Copyright (c) 2007 SUSE LINUX Products GmbH, Nuernberg, Germany. # This file and all modifications and additions to the pristine # package are under the same license as the package itself. # @@ -14,12 +14,13 @@ BuildRequires: SDL_image-devel SDL_mixer-devel Summary: Clone of the game AtomicBomberMan Version: 0.11.7 -Release: 1 -License: GPL +Release: 38 +License: GNU General Public License (GPL) URL: http://www.bomberclone.de Group: Amusements/Games/Action/Other Source: %{name}-%{version}.tar.bz2 Patch: bomberclone-docpath.patch +Patch1: %{name}-%{version}_array-subscript.patch BuildRoot: %{_tmppath}/%{name}-%{version}-build Autoreqprov: on @@ -47,6 +48,7 @@ %prep %setup %patch +%patch1 %build aclocal @@ -71,7 +73,10 @@ %_datadir/games/bomberclone %_bindir/bomberclone -%changelog -n bomberclone +%changelog +* Tue Mar 13 2007 - pgajdos@suse.cz +- fixed: 'warning: array subscript is above array bounds' (#242031) +- array_subscript.patch * Fri Sep 29 2006 - hmacht@suse.de - update to 0.11.7 which fixes some security issues (novell bug 206913) ++++++ bomberclone-0.11.7_array-subscript.patch ++++++ --- src/player.c +++ src/player.c @@ -873,16 +873,16 @@ if (nr < 0 || nr >= MAX_BOMBS) // i out of range .. restart at bomb 0 nr -= MAX_BOMBS; - if (player->bombs[i].state == BS_off) { /* check if this bomb is free */ + if (player->bombs[nr].state == BS_off) { /* check if this bomb is free */ if (res == -1) - res = i; + res = nr; } else bombused++; // count number of used bombs } - if (res == -1 && i == player->bomb_lastex && player->bombs[i].state == BS_off) - res = i; + if (res == -1 && nr == player->bomb_lastex && player->bombs[nr].state == BS_off) + res = nr; if (bombused >= player->bombs_n) res = -1; /* all max number of bombs lay */ ++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++++ Remember to have fun... --------------------------------------------------------------------- To unsubscribe, e-mail: opensuse-commit+unsubscribe@opensuse.org For additional commands, e-mail: opensuse-commit+help@opensuse.org