bzfs crash on exit

[matelich]

Love the tails on the bullets. Just jumped from May to latest code, lots of fun stuff.

I don't know if my configuration specifically was causing the crash, but this fixes it (erase returns the iter after listIt, so effectively already incremented for you).

I assume this is fine for such a small change, do you use the sf.net patch tool otherwise?

Code: Select all

c:\Users\tom\code\bz\bzflag\bzflag>svn diff src
Index: src/bzfs/WorldEventManager.cxx
===================================================================
--- src/bzfs/WorldEventManager.cxx      (revision 20236)
+++ src/bzfs/WorldEventManager.cxx      (working copy)
@@ -83,7 +83,8 @@
         listIt = evList.erase(listIt);
         foundOne = true;
       }
-      listIt++;
+      else
+         listIt++;
     }
   }

[trepan]

Thank you for the patch.

Code: Select all

r20250 | trepan | 2009-07-21 03:20:08 -0300 (Tue, 21 Jul 2009) | 3 lines
* patch by 'matelich' to fix a bzfs exit crash, forum post:
  http://my.bzflag.org/bb/viewtopic.php?f=32&t=14505&p=140451