On 2/14/07, Troman <[EMAIL PROTECTED]> wrote:
----- Original Message ----- From: "Ari Johnson" <[EMAIL PROTECTED]> To: "Development list" <firstname.lastname@example.org> Sent: Wednesday, February 14, 2007 10:43 PM Subject: Re: [Warzone-dev] Current problems > On 2/14/07, Troman <[EMAIL PROTECTED]> wrote: >> >> ----- Original Message ----- >> From: "Ari Johnson" <[EMAIL PROTECTED]> >> To: "Development list" <email@example.com> >> Sent: Wednesday, February 14, 2007 10:18 PM >> Subject: Re: [Warzone-dev] Current problems >> >> >> > On 2/14/07, Troman <[EMAIL PROTECTED]> wrote: >> >> > The question is how the strings are stored - do we malloc up space >> >> > for >> >> > them when we read in the file? >> >> >> >> When save games are loaded? I think loaded strings should be just >> >> copied >> >> into the space allocated for string variables (union's .sval pointer). >> > >> > That won't work, if I am reading you correctly. The .sval pointer is >> > just that: a pointer. You need to point it to some memory to store >> > the actual contents of the string. The way to solve this is to find >> > out where .sval pointers come from in the normal running of the game. >> > Are they malloc()ed and later free()d, are they stored in a central >> > store with garbage collection for the scripting engine, or are they >> > stored in static buffers somewhere? Once we know this, we can go >> > forward with fixing how they are stored in evntsave.c. >> >> What I wanted to say is that at that point .sval already points to a >> valid >> memory address, so it should be safe to copy the string. > > I don't believe that it is. If you read eventLoadContextHashed, up > through line 485, where does sval get initialized? In eventNewContext(), line 415.
Except that it's the "data" variable which is set, whereas eventNewContext is only called with reference to psCode, release, and psCCont. Maybe I am missing how those relate. _______________________________________________ Warzone-dev mailing list Warzonefirstname.lastname@example.org https://mail.gna.org/listinfo/warzone-dev