In G_SaveMapState, fix 1024-byte write beyond array bound on 64-bit platforms.
This was caused by mismatched for types mapstate_t's animateptr[] and the global one (int32_t vs. int32_t *). Bump BYTEVERSION.
File di log originale · 20131028-4125.txt
------------------------------------------------------------------------ r4125 | helixhorned | 2013-10-28 14:26:32 -0700 (Mon, 28 Oct 2013) | 6 lines In G_SaveMapState, fix 1024-byte write beyond array bound on 64-bit platforms. This was caused by mismatched for types mapstate_t's animateptr[] and the global one (int32_t vs. int32_t *). Bump BYTEVERSION. ------------------------------------------------------------------------ r4124 | helixhorned | 2013-10-28 14:26:30 -0700 (Mon, 28 Oct 2013) | 7 lines m32script: when failing to load file, don't append '.m32' if already there. So that there are no error messages like "M32 file `a.m32.m32' not found." Also, in build.c's 'save as' code, add a bound check that would probably always pass in practice, but looks a bit safer and may fail in very cornerly cases. ------------------------------------------------------------------------ r4123 | helixhorned | 2013-10-28 14:26:28 -0700 (Mon, 28 Oct 2013) | 1 line Mapster32: save r_usetileshades and r_pr_artmapping with mapster32.cfg. ------------------------------------------------------------------------ r4122 | helixhorned | 2013-10-28 14:26:27 -0700 (Mon, 28 Oct 2013) | 1 line m32script: when printing error line, replace tabs with spaces. ------------------------------------------------------------------------ r4121 | helixhorned | 2013-10-28 14:26:25 -0700 (Mon, 28 Oct 2013) | 9 lines Mapster32: take initial map file name from ExtPreInit()->G_CheckCommandLine(). Instead of directly in build.c:app_main(). Previously, it was possible that an argument to an option, e.g. "WGR2" with "-game_dir WGR2", would get interpreted as the initial map to load. This in turn would attempt to execute WGR2.cfg as sequence of OSD commands -- where that file is an EDuke32 config file instead! Now: maps are passed without any options, e.g. "mapster32 [opts...] debug.map". The map file name is not added to the "additional parameters" for map testing. ------------------------------------------------------------------------
when failing to load file, don't append '.m32' if already there.
So that there are no error messages like "M32 file `a.m32.m32' not found." Also, in build.c's 'save as' code, add a bound check that would probably always pass in practice, but looks a bit safer and may fail in very cornerly cases.
save r_usetileshades and r_pr_artmapping with mapster32.cfg.
when printing error line, replace tabs with spaces.
take initial map file name from ExtPreInit()->G_CheckCommandLine().
Instead of directly in build.c:app_main(). Previously, it was possible that an argument to an option, e.g. "WGR2" with "-game_dir WGR2", would get interpreted as the initial map to load. This in turn would attempt to execute WGR2.cfg as sequence of OSD commands -- where that file is an EDuke32 config file instead! Now: maps are passed without any options, e.g. "mapster32 [opts...] debug.map". The map file name is not added to the "additional parameters" for map testing.