Fatal Error: "Out of range write" in bitStream.cc

Quote:We need to get under the hood! In order to load more than forty or so sequences at a time, we have to change the way Torque loads dsq files.

This will be handled in:
engine/ts/TSShapeConstruct.h
engine/ts/TSShapeConstruct.cc

The problem is that the engine loads all these names with their complete path names and stores them as a big string array, which it has to send as a network packet (even in a single player game). That's fine for the thirty sequences provided, but it means that if you try to include something like, say 72 additional sequences, all those characters are quickly going to flood the buffer beyond the 1500 (or technically 2000) byte maximum size.

So, I had to write new versions of the TSShapeConstruct functions that deal with sequence names: packData and unpackData. In my versions I stripped off the path and other easily reproduced information, and got it down to storing only the sequence name, which in my application I then reduced to a naming convention of three characters each, as in "100.dsq", "101.dsq", etc.

(For those who like to know the limits, these changes have been tested successfully up to about 456 sequences.)

Quote:So, the first thing we need to do is open engine/ts/tsShapeConstruct.h. Go to line 27 or so, and find:
"NumSequenceBits = 7,"

3. Change that to seven to a nine:
"NumSequenceBits = 9," That changes one of the caps, from 128 to 512.

Next, go to tsShapeConstruct.cc.

Right at the top, just after the #include statements and before the "IMPLEMENT_CO_DATABLOCK" statement, around line 11,

4. Insert the following line:
#define TS_IS_RELATIVE_SEQUENCES 1

This is a toggle for what we're about to do to packData and unpackData.

void TSShapeConstructor::packData(BitStream* stream)
{

   Parent::packData(stream);
   stream->writeString(mShape);

   S32 count = 0;
   for (S32 b=0; b<MaxSequences; b++)
      if (mSequence[b])
         count++;
   stream->writeInt(count,NumSequenceBits);

   
   for (S32 i=0; i<MaxSequences; i++)
     if (mSequence[i]) {
       if (TS_IS_RELATIVE_SEQUENCES) {
	 const char *subtext = dStrrchr(mSequence[i],'/');
	 stream->writeString(subtext);
       } else {
	 stream->writeString(mSequence[i]);
       }
     }
}

void TSShapeConstructor::unpackData(BitStream* stream)
{
   char shapePath[90],pathedSeq[90];
   Parent::unpackData(stream);
   mShape = stream->readSTString();
   
   if (TS_IS_RELATIVE_SEQUENCES) {
     dsize_t strLen = (int)dStrlen(mShape);
     const char *subtext = dStrrchr(mShape,'/');

     dsize_t endLen = (int)dStrlen(subtext);
     dsize_t pathLen = (strLen - endLen) ;
	
     dStrncpy(shapePath,mShape,pathLen);
     shapePath[pathLen] = '[[60c1da72daaa7]]';
   }

   S32 i = 0, count = stream->readInt(NumSequenceBits);
   for (; i<count; i++) {
     mSequence[i] = stream->readSTString();
     if (TS_IS_RELATIVE_SEQUENCES) {
       dSprintf(pathedSeq,90,"%s%s",shapePath,mSequence[i]);
       mSequence[i] = StringTable->insert(pathedSeq);
     }
   }
   while (i<MaxSequences)
      mSequence[i++] = NULL;
}