View Single Post
Old 29 July 2018, 13:45   #29
hexaae
Registered User
hexaae's Avatar
 
Join Date: Jul 2006
Location: Italy
Age: 42
Posts: 1,118
Quote:
Originally Posted by PeterK View Post
I'm not sure if it works reliable, but I found two AllocVec() instructions for 80 bytes and changed them to allocate 127 bytes now. I didn't check what the code is really doing with these buffers, but since you said the crashes start with 84 characters these buffer could be guilty. AllocVec() needs 4 additional bytes for storing the size and AllocMem() would round up this request to 88 bytes, which would give exactly 84 usable bytes (83 chars + Null) before a buffer overflow occurs. It seems to work now on my system, at least in a first quick test.

Mmmh, today I tried to patch only one instance of the two AllocVec() and works fine as well with >83 filenames. Better minimal changes to avoid potential mem-leaks?
hexaae is offline  
 
Page generated in 0.03921 seconds with 11 queries