Re: $whoami calculation in Sys::Syslog.pm should not be greedy

[p5sagit/p5-mst-13.2.git] / av.c

diff --git a/av.c b/av.c
index cad6eae..9e94805 100644 (file)
--- a/av.c
+++ b/av.c
@@ -1,6 +1,6 @@
 /*    av.c
  *
- *    Copyright (c) 1991-1994, Larry Wall
+ *    Copyright (c) 1991-1997, Larry Wall
  *
  *    You may distribute under the terms of either the GNU General Public
  *    License or the Artistic License, as specified in the README file.
@@ -33,6 +33,9 @@ AV* av;
        if (sv != &sv_undef)
            (void)SvREFCNT_inc(sv);
     }
+    key = AvARRAY(av) - AvALLOC(av);
+    while (key)
+       AvALLOC(av)[--key] = &sv_undef;
     AvREAL_on(av);
 }
 
@@ -179,10 +182,13 @@ SV *val;
 
     if (!av)
        return 0;
+    if (!val)
+       val = &sv_undef;
 
     if (SvRMAGICAL(av)) {
        if (mg_find((SV*)av,'P')) {
-           mg_copy((SV*)av, val, 0, key);
+           if (val != &sv_undef)
+               mg_copy((SV*)av, val, 0, key);
            return 0;
        }
     }
@@ -192,14 +198,12 @@ SV *val;
        if (key < 0)
            return 0;
     }
-    if (!val)
-       val = &sv_undef;
-
+    if (SvREADONLY(av) && key >= AvFILL(av))
+       croak(no_modify);
+    if (!AvREAL(av) && AvREIFY(av))
+       av_reify(av);
     if (key > AvMAX(av))
        av_extend(av,key);
-    if (AvREIFY(av))
-       av_reify(av);
-
     ary = AvARRAY(av);
     if (AvFILL(av) < key) {
        if (!AvREAL(av)) {
@@ -362,6 +366,8 @@ register AV *av;
 
     if (!av || AvFILL(av) < 0)
        return &sv_undef;
+    if (SvREADONLY(av))
+       croak(no_modify);
     retval = AvARRAY(av)[AvFILL(av)];
     AvARRAY(av)[AvFILL(av)--] = &sv_undef;
     if (SvSMAGICAL(av))
@@ -379,12 +385,10 @@ register I32 num;
 
     if (!av || num <= 0)
        return;
-    if (!AvREAL(av)) {
-       if (AvREIFY(av))
-           av_reify(av);
-       else
-           croak("Can't unshift");
-    }
+    if (SvREADONLY(av))
+       croak(no_modify);
+    if (!AvREAL(av) && AvREIFY(av))
+       av_reify(av);
     i = AvARRAY(av) - AvALLOC(av);
     if (i) {
        if (i > num)
@@ -422,6 +426,8 @@ register AV *av;
 
     if (!av || AvFILL(av) < 0)
        return &sv_undef;
+    if (SvREADONLY(av))
+       croak(no_modify);
     retval = *AvARRAY(av);
     if (AvREAL(av))
        *AvARRAY(av) = &sv_undef;