This is a live mirror of the Perl 5 development currently hosted at https://github.com/perl/perl5
split was leaving PL_sv_undef in unused ary slots
authorDavid Mitchell <davem@iabyn.com>
Wed, 30 Nov 2016 08:59:01 +0000 (08:59 +0000)
committerDavid Mitchell <davem@iabyn.com>
Wed, 30 Nov 2016 09:11:25 +0000 (09:11 +0000)
This:

    @a = split(/-/,"-");
    $a[1] = undef;
    $a[0] = 0;

was giving

    Modification of a read-only value attempted at foo line 3.

This is because:

1) unused slots in AvARRAY between AvFILL and AvMAX should always be
null; av_clear(), av_extend() etc do this; while av_store(), if storing
to a slot N somewhere between AvFILL and AvMAX, doesn't bother to clear
between (AvFILL+1)..(N-1) on the assumption that everyone else plays
nicely.

2) pp_split() when splitting directly to an array, sometimes over-splits
and has to null out the excess elements;

3) Since perl 5.19.4, unused AV slots are now marked with NULL rather than
&PL_sv_undef;

4) pp_split was still using &PL_sv_undef;

The fault was with (4), and is easily fixed.

pp.c
t/op/split.t

diff --git a/pp.c b/pp.c
index d406ee1..b198b47 100644 (file)
--- a/pp.c
+++ b/pp.c
@@ -6095,7 +6095,7 @@ PP(pp_split)
            while (iters > 0 && (!TOPs || !SvANY(TOPs) || SvCUR(TOPs) == 0)) {
                if (TOPs && !make_mortal)
                    sv_2mortal(TOPs);
            while (iters > 0 && (!TOPs || !SvANY(TOPs) || SvCUR(TOPs) == 0)) {
                if (TOPs && !make_mortal)
                    sv_2mortal(TOPs);
-               *SP-- = &PL_sv_undef;
+               *SP-- = NULL;
                iters--;
            }
        }
                iters--;
            }
        }
index 037aa2e..ceaea00 100644 (file)
@@ -7,7 +7,7 @@ BEGIN {
     set_up_inc('../lib');
 }
 
     set_up_inc('../lib');
 }
 
-plan tests => 159;
+plan tests => 161;
 
 $FS = ':';
 
 
 $FS = ':';
 
@@ -610,3 +610,14 @@ is "@a", '1 2 3', 'assignment to split-to-array (stacked)';
 
 
 }
 
 
 }
+
+# splitting directly to an array wasn't filling unused AvARRAY slots with
+# NULL
+
+{
+    my @a;
+    @a = split(/-/,"-");
+    $a[1] = 'b';
+    ok eval { $a[0] = 'a'; 1; }, "array split filling AvARRAY: assign 0";
+    is "@a", "a b", "array split filling AvARRAY: result";
+}