[patch, fortran] Fix off-by-one array size in trans-intrinsic.c

FX fxcoudert@gmail.com
Wed Sep 1 10:29:00 GMT 2010


Many thanks to Juha for the bug report, and sorry for not replying earlier.  Once I had valgrind installed on my machine, it became clear your issue was reproducible. In fact, I simply made an array that's too small... The attached patch fixes it.

It also removes an unused assignment to real16_decl for complex intrinsics (it was used in a preliminary version of my initial patch, but is not needed anymore).

Bootstrapped and regtested both with TFmode enabled and without, on x86_64-linux. Tested a few testcases under valgrind. OK to commit?

FX


-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix.ChangeLog
Type: application/octet-stream
Size: 176 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/fortran/attachments/20100901/4d8adcd8/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fix.diff
Type: application/octet-stream
Size: 1067 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/fortran/attachments/20100901/4d8adcd8/attachment-0001.obj>


More information about the Fortran mailing list