| 00:03.00 | Constitution | sounds like an elegant design |
| 00:03.40 | blast007 | checking 'if (display != NULL)' inside dying() takes care of that, but then there's some other issue (that may be the same on Windows and Linux) |
| 00:06.04 | BulletCatcher | See if reverting fa71748d (which reverts cef9ee02 itself) helps. |
| 00:07.10 | blast007 | BulletCatcher: this is a client crash |
| 00:07.22 | BulletCatcher | Never mind, then. :-) |
| 00:07.26 | blast007 | :) |
| 00:09.42 | blast007 | and on Linux it's one of those fun ones where it doesn't do it in a debug build... |
| 00:13.06 | blast007 | http://static.bzexcess.com/exception.txt <-- that's the call stack when I break on the exception |
| 00:13.38 | blast007 | First-chance exception at 0x54c0ad7a (msvcp100d.dll) in bzflag.exe: 0xC0000005: Access violation reading location 0x3f800004. |
| 00:13.51 | the_map | oh wow, the call stack looks like fun |
| 00:14.16 | blast007 | definitely |
| 00:14.24 | blast007 | which is why I kept putting this off ;) |
| 00:14.33 | blast007 | been an issue for a long while |
| 00:14.50 | blast007 | but I thought it was just a 'windows debug build' thing |
| 01:56.28 | Constitution | fwiw, it doesn't crash on mac, release or debug |
| 01:57.03 | blast007 | and exits with a 0 exit code? |
| 02:37.18 | Constitution | correct |
| 02:38.39 | *** join/#bzflag BZnotify (~BZnotify@192.30.252.41) |
| 02:38.42 | BZnotify | [13bzflag] 15macsforme pushed 2 new commits to 062.4: 02http://git.io/vB16B |
| 02:38.42 | BZnotify | 13bzflag/062.4 14dd1b06f 15Joshua Bodine: Removed README.XMINGW file. |
| 02:38.42 | BZnotify | 13bzflag/062.4 14ed93a75 15Joshua Bodine: Updated README.SDL file. |
| 02:38.42 | *** part/#bzflag BZnotify (~BZnotify@192.30.252.41) |
| 02:58.56 | *** join/#bzflag BZnotify (~BZnotify@192.30.252.34) |
| 02:58.56 | BZnotify | [13bzflag] 15macsforme pushed 1 new commit to 062.4: 02http://git.io/vB1iA |
| 02:58.56 | BZnotify | 13bzflag/062.4 146cbfa5f 15Joshua Bodine: Updated README.Linux file. |
| 02:58.56 | *** part/#bzflag BZnotify (~BZnotify@192.30.252.34) |
| 08:24.39 | *** join/#bzflag zuii (~akos@BC06BF93.catv.pool.telekom.hu) |
| 11:55.00 | *** join/#bzflag EvilJStoker (jstoker@unaffiliated/jstoker) |
| 14:08.57 | *** join/#bzflag trepan (~trepan@s72-38-252-50.static.datacom.cgocable.net) |
| 15:25.48 | blast007 | this is the commit where both Windows debug and Linux non-debug started to crash on exit: https://github.com/BZFlag-Dev/bzflag/commit/b885cc524731368e6b910e654c62d141daa94e87 |
| 15:34.36 | blast007 | think I found the issue. There was a loop in Team::updateShotColors() that used <= instead of <, so the there was an array boundary overrun. |
| 15:36.47 | BulletCatcher | That would be a problem. |
| 15:36.51 | BulletCatcher | Another oddity about that commit is that addBrightness() is a class method instead of simply a static function, but that shouldn't cause a crash. |
| 15:38.08 | blast007 | checking if Cppcheck would have caught that bounds issue |
| 15:42.08 | blast007 | nope.. nor flawfinder. Guess that's not a job for static analysis tools. |
| 15:58.42 | *** join/#bzflag BZnotify (~BZnotify@192.30.252.40) |
| 15:58.42 | BZnotify | [13bzflag] 15blast007 pushed 1 new commit to 062.4: 02http://git.io/vBDwg |
| 15:58.42 | BZnotify | 13bzflag/062.4 143983258 15Scott Wichser: Fix an array boundary overrun that was causing a crash on Windows and Linux when exiting the client. |
| 15:58.42 | *** part/#bzflag BZnotify (~BZnotify@192.30.252.40) |
| 15:59.20 | BulletCatcher | ~blast007++ |
| 16:00.28 | blast007 | has anyone else been able to reproduce issue #29? |
| 16:02.03 | blast007 | the issue doesn't seem plausable, really.. might have seen like a possible issue if the player's tank was just invisible, but they also could not shoot the other player |
| 16:08.27 | alezakos | I couldn't reproduce it, even when I joined the Clay Hills server with 2.4.4 as an observer some time after the bug was reported - that specific player was visible to me |
| 16:11.21 | the_map | I could always see that player (and all the others) on Clay Hills or any other server; I'm on bzflag 2.4.4 RC2 |
| 16:13.46 | BulletCatcher | It might help to know the player slot indexes at the time of the incident. |
| 16:16.00 | BulletCatcher | Some other buffer overrun (or possibly even the one fixed just now, although I don't see how) could have set a tank color's alpha to zero. |
| 16:28.37 | *** join/#bzflag Constitution (~josh@bzflag/developer/Constitution) |
| 16:29.45 | *** join/#bzflag contempt (contempt@unaffiliated/contempt) |
| 16:52.20 | Constitution | blast007: you fixed it? |
| 16:52.31 | blast007 | yeah |
| 16:52.37 | Constitution | ~blast007++ |
| 16:52.51 | blast007 | git bisect is sweet |
| 18:38.00 | *** join/#bzflag BZnotify (~BZnotify@192.30.252.40) |
| 18:38.00 | BZnotify | [13bzflag] 15macsforme pushed 1 new commit to 062.4: 02http://git.io/vBysJ |
| 18:38.00 | BZnotify | 13bzflag/062.4 145334f75 15Joshua Bodine: Do not clear the plugin/event handler map on WorldEventManager destruction, because we don't own it and it might have already been freed before our destructor was called.... |
| 18:38.00 | *** part/#bzflag BZnotify (~BZnotify@192.30.252.40) |
| 18:42.49 | Constitution | so would an RC3 would be appropriate prior to release, since we've committed a few semi-significant changes? |
| 18:43.52 | Constitution | it won't hurt us if the RC3 release is the same as the final release |
| 18:44.27 | Constitution | maybe once we finish the documentation update |
| 19:25.11 | the_map | feature request: capes on super bullets |
| 19:32.33 | *** join/#bzflag zuii (~akos@BC06BF93.catv.pool.telekom.hu) |
| 19:36.36 | blast007 | yeah, I think we'll do an RC3 soon-ish |
| 19:36.54 | blast007 | give it a couple more weeks at minimum for any other odities to pop up |
| 19:45.21 | *** join/#bzflag Constitution (b8653225@bzflag/developer/Constitution) |
| 20:06.13 | Constitution | make distcheck is failing for me on Ubuntu 14.04... http://pastebin.com/Wg5BAhkr |
| 20:12.56 | blast007 | Constitution: are you using cpputest from their github, or from the Ubuntu package? |
| 20:15.55 | blast007 | (or probably, you don't have cpputest installed at all) |
| 20:24.12 | Constitution | that would probably do it, huh |
| 20:24.49 | *** join/#bzflag jcp (~quassel@bzflag/contributor/javawizard2539) |
| 20:25.09 | Constitution | should we add that to the list of needed packages in the readme? |
| 20:26.13 | blast007 | it's not necessary for building the game, only for development purposes |
| 20:26.15 | *** join/#bzflag DTRemenak (~DTRemenak@about/essy/CrazyCoder/DTRemenak) |
| 20:26.15 | *** mode/#bzflag [+v DTRemenak] by ChanServ |
| 20:29.51 | blast007 | I'm not entirely sure which version we need at minimum. |
| 20:46.52 | *** join/#bzflag jcp (~quassel@bzflag/contributor/javawizard2539) |
| 21:12.03 | *** join/#bzflag EvilJStoker (jstoker@unaffiliated/jstoker) |
| 21:12.03 | *** join/#bzflag DTRemenak (~DTRemenak@about/essy/CrazyCoder/DTRemenak) |
| 21:12.03 | *** mode/#bzflag [+v DTRemenak] by ChanServ |
| 21:12.12 | *** join/#bzflag AAA_awright (~aaa@2607:5300:60:7d4::1) |
| 21:14.53 | *** join/#bzflag EvilJStoker (jstoker@unaffiliated/jstoker) |
| 21:18.40 | *** join/#bzflag alezakos (~kongr45gp@bzflag/developer/alezakos) |
| 21:19.29 | *** join/#bzflag Kaelten (Kaelten@WoWUIDev/WoWAce/WoWIFA/CurseStaff/kaelten) |
| 21:20.18 | *** join/#bzflag alpha1-2 (~alpha1@host179.181-1-180.telecom.net.ar) |
| 21:25.53 | *** join/#bzflag joevano (~joevano@bzflag/developer/JoeVano) |
| 21:25.53 | *** mode/#bzflag [+v joevano] by ChanServ |
| 21:33.03 | *** join/#bzflag blast007 (~blast007@bzflag/developer/Blast) |
| 21:46.18 | alpha1-2 | is there a way to make a search about a specific version number of a server? For example 2.4.0. I am trying http://my.bzflag.org/db/ but it seems not to work. |
| 21:51.53 | blast007 | no |
| 21:52.39 | blast007 | the 'version' there is the protocol version |
| 21:53.29 | blast007 | BZFS0026 would show you three servers, the various 2.0 servers |
| 21:53.29 | blast007 | BZFS0221 would show you all the 2.4 servers |
| 21:54.07 | alpha1-2 | I searched for a name but it didn't work |
| 21:54.07 | alpha1-2 | is it buggy maybe? |
| 21:54.25 | alpha1-2 | (title) |
| 21:55.31 | blast007 | from that page? there is no search there |
| 21:55.40 | blast007 | title is only for ADD or REMOVE actions |
| 21:57.32 | alpha1-2 | ah okay |
| 21:57.58 | *** mode/#bzflag [+o blast007] by ChanServ |
| 21:58.10 | *** mode/#bzflag [-b *!*alpha1@181.90.27.*] by blast007 |
| 21:58.10 | *** mode/#bzflag [-o blast007] by blast007 |
| 21:58.28 | alpha1-2 | ah it works with those versions, yes, not buggy |
| 22:00.20 | blast007 | the searching from the server list in the game is all done inside the game code, not the list |