[LLVMbugs] Fix for heap corruption bug

Argiris Kirtzidis akyrtzi at gmail.com
Sat Feb 23 02:44:49 PST 2008


Hi,

The attached patch fixes a very small bug:

---------------------------------------
  if (!cstr)
    cstr = new char[len + (isNullTerm ? 1 : 0)];
 
  assert (cstr != NULL);
 
  for (unsigned i = 0; i < len; ++i)
    cstr[i] = (char) ReadInt();
 
  if (isNullTerm)
    cstr[len+1] = '\0';
---------------------------------------

"cstr[len+1] = '\0';" writes one byte after the reserved block of 
memory; msvcrt was kind enough to report it with a "heap corruption" 
error ;)


-------------- next part --------------
A non-text attachment was scrubbed...
Name: heap-fix.zip
Type: application/octet-stream
Size: 318 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20080223/f8109674/attachment.obj>
-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: heap-fix.patch
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20080223/f8109674/attachment.ksh>


More information about the llvm-bugs mailing list