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)
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

index a5bd172..a59b7f2 100644 (file)
@@ -71,7 +71,7 @@ sub writemain{
     my ($tail1,$tail2,$tail3) = ( $tail =~ /\A(.*{\s*\n)(.*\n)(\s*\}.*)\Z/s );
 
     print $tail1;
-    print "\tconst char file[] = __FILE__;\n";
+    print "\tstatic const char file[] = __FILE__;\n";
     print "\tdXSUB_SYS;\n" if $] > 5.002;
     print $tail2;