xs_init() must pass a static char* when creating &DynaLoader::boot_DynaLoader.
authorNicholas Clark <nick@ccl4.org>
Mon, 25 Mar 2013 12:06:09 +0000 (13:06 +0100)
committerNicholas Clark <nick@ccl4.org>
Mon, 25 Mar 2013 20:19:36 +0000 (21:19 +0100)
commit1b77350017006d9e581d4eda49c59b68e408f51b
tree410aa761dad94946e7a50009b517f51860623888
parent6bf48f47bbdbe4838b70d67830ead4ebc3d318d4
xs_init() must pass a static char* when creating &DynaLoader::boot_DynaLoader.

newXS() assumes that the passed pointer to the filename is in static storage,
or otherwise will outlive the PVCV that it is about to create, and hence that
it's safe to copy the pointer, not the value, to CvFILE.  Hence xs_init()
must not use an auto array to "store" the filename, as that will be on the
stack, and becomes invalid as soon as xs_init() returns.  The analogous bug
fix was made in universal.c by commit 157e3fc8c802010d in Feb 2006.

Spotted by compiling for ithreads with gcc 4.8.0's ASAN and running
dist/B-Deparse/t/deparse.t
minimod.pl