Old SCI0 Template Save Bug

[troflip]

I think the thread took a turn and stopped being about the save bug 

[lskovlun]

I found the bug! It is a buffer overflow in the script code.
We have this line:

Code: [Select]

(method (doit strDescription &tmp temp0 temp1 temp2 temp3 [temp4 360] [temp365 20] [temp386 20])
Look closely at the temp4 variable (which contains the description strings and comes just before the index numbers). It is sized as 360 (20*18 words), but this is not quite enough. Eric's authentic file has this define:

Code: [Select]

(define BUFFERSIZE 361) ;(define BUFFERSIZE (+ (* MAXGAMES COMMENTBUFF) 1))If you change the size of that temp4 array to be just a bit larger (361 words), you'll get the intended result.

[Doan Sephim]

Fantastic! I did the suggested change and it seems to work perfectly!

Awesome catch! I wouldn't have seen that in a million years.

Quick question, is there any potential unintended consequences or is this a fairly clean fix. I get that it was an overflow issue, but my comprehension of how and why is not so good.

[Kawa]

It's a clean fix and holy shit that's a sneaky one haha

[lskovlun]

It's a clean fix and holy shit that's a sneaky one haha

It's sneaky and goes right back to SCI Studio. That means this is handwritten, not decompiled code. It was rewritten from an ASM block.

[troflip]

Wow! You should feel good about finding that one! 

[EricOakford]

It's a clean fix and holy shit that's a sneaky one haha

It's sneaky and goes right back to SCI Studio. That means this is handwritten, not decompiled code. It was rewritten from an ASM block.

Naturally, when you only have disassembled code for reference, mistakes can be made. With original source for reference, these can be discovered and fixed.

[Doan Sephim]

You guys are awesome! Thanks for looking into this for me. I'm looking forward to using SCI01 moving forward, so I'm sure these problems will be a thing of the past.

[lskovlun]

I have to thank Phil for coming up with a different way to test this. It was really the discrepancy between the two approaches that led me on the right track.

[MusicallyInspired]

Awesome!!