savegame.c: minor cleanup and crash prevention in doloadplayer2

if loading directly from file to state.

git-svn-id: https://svn.eduke32.com/eduke32@2190 1a8010ca-5511-0410-912e-c29ae57300e0
This commit is contained in:
helixhorned 2011-12-21 18:42:12 +00:00
parent eee507741d
commit a7d1177d23

View file

@ -652,10 +652,8 @@ int32_t G_LoadPlayer(int32_t spot)
g_showShareware = 0; g_showShareware = 0;
everyothertime = 0; everyothertime = 0;
// clearbufbyte(playerquitflag,MAXPLAYERS,0x01010101);
for (i=0; i<MAXPLAYERS; i++) for (i=0; i<MAXPLAYERS; i++)
clearbufbyte(&g_player[i].playerquitflag,1,0x01010101); g_player[i].playerquitflag = 1;
Net_ResetPrediction(); Net_ResetPrediction();
@ -1114,7 +1112,7 @@ static int32_t readspecdata(const dataspec_t *spec, int32_t fil, uint8_t **dumpv
j=kread(fil, cmpstrbuf, i); j=kread(fil, cmpstrbuf, i);
if (j!=i || Bmemcmp(sp->ptr, cmpstrbuf, i)) if (j!=i || Bmemcmp(sp->ptr, cmpstrbuf, i))
{ {
OSD_Printf("rds: spec=%p, sp=%p ", spec, sp); OSD_Printf("rsd: spec=%p, sp=%p ", spec, sp);
if (j!=i) if (j!=i)
OSD_Printf(" kread returned %d, expected %d.\n", j, i); OSD_Printf(" kread returned %d, expected %d.\n", j, i);
else else
@ -1143,13 +1141,13 @@ static int32_t readspecdata(const dataspec_t *spec, int32_t fil, uint8_t **dumpv
} }
if (i!=j) if (i!=j)
{ {
OSD_Printf("rsd: spec=%p, sp=%p, mem=%p ", spec, sp, mem); OSD_Printf("rsd: spec=%p, sp=%p, mem=%p\n", spec, sp, mem);
OSD_Printf("rsd: %s: read %d, expected %d!\n", OSD_Printf(" %s: read %d, expected %d!\n",
((sp->flags&DS_CNTMASK)==0 && sp->size*cnt<=savegame_comprthres)? ((sp->flags&DS_CNTMASK)==0 && sp->size*cnt<=savegame_comprthres)?
"UNCOMP":"COMPR", i, j); "UNCOMP":"COMPR", i, j);
if (i==-1) if (i==-1)
perror("read"); OSD_Printf(" read: %s\n", strerror(errno));
return -1; return -1;
} }
} }
@ -1686,7 +1684,7 @@ static int32_t doallocsnap(int32_t allocinit)
svinitsnap = Bmalloc(svsnapsiz); svinitsnap = Bmalloc(svsnapsiz);
svdiffsiz = svsnapsiz; // theoretically it's less than could be needed in the worst case, but practically it's overkill svdiffsiz = svsnapsiz; // theoretically it's less than could be needed in the worst case, but practically it's overkill
svdiff = Bmalloc(svdiffsiz); svdiff = Bmalloc(svdiffsiz);
if (svsnapshot==NULL || (allocinit && svinitsnap==0) || svdiff==NULL) if (svsnapshot==NULL || (allocinit && svinitsnap==NULL) || svdiff==NULL)
{ {
sv_freemem(); sv_freemem();
return 1; return 1;
@ -2222,6 +2220,7 @@ static int32_t doloadplayer2(int32_t spot, int32_t fil, uint8_t **memptr)
if (Gv_ReadSave(fil, 1)) return -7; if (Gv_ReadSave(fil, 1)) return -7;
sv_makevarspec(); sv_makevarspec();
if (mem)
for (i=1; svgm_vars[i].flags!=DS_END; i++) for (i=1; svgm_vars[i].flags!=DS_END; i++)
{ {
Bmemcpy(mem, svgm_vars[i].ptr, svgm_vars[i].size*svgm_vars[i].cnt); // careful! works because there are no DS_DYNAMIC's! Bmemcpy(mem, svgm_vars[i].ptr, svgm_vars[i].size*svgm_vars[i].cnt); // careful! works because there are no DS_DYNAMIC's!
@ -2305,9 +2304,9 @@ static void postloadplayer1()
//6 //6
g_showShareware = 0; g_showShareware = 0;
// everyothertime = 0; // everyothertime = 0;
// clearbufbyte(playerquitflag,MAXPLAYERS,0x01010101);
for (i=0; i<MAXPLAYERS; i++) for (i=0; i<MAXPLAYERS; i++)
clearbufbyte(&g_player[i].playerquitflag,1,0x01010101); g_player[i].playerquitflag = 1;
} }
static void postloadplayer2() static void postloadplayer2()