|
 |
9eebde9 |
From 3f8dbf40138bd2bcb569b23c88888a41ede9c355 Mon Sep 17 00:00:00 2001
|
|
|
9eebde9 |
From: Tony Cook <tony@develop-help.com>
|
|
|
9eebde9 |
Date: Mon, 5 Aug 2019 15:23:45 +1000
|
|
|
9eebde9 |
Subject: [PATCH] (perl #134266) make sure $@ is writable when we write to it
|
|
|
9eebde9 |
MIME-Version: 1.0
|
|
|
9eebde9 |
Content-Type: text/plain; charset=UTF-8
|
|
|
9eebde9 |
Content-Transfer-Encoding: 8bit
|
|
|
9eebde9 |
|
|
|
9eebde9 |
when unwinding.
|
|
|
9eebde9 |
|
|
|
9eebde9 |
Since except_sv might be ERRSV we try to preserve it's value,
|
|
|
9eebde9 |
if not the actual SV (which we have an extra refcount on if it is
|
|
|
9eebde9 |
except_sv).
|
|
|
9eebde9 |
|
|
|
9eebde9 |
Petr Písař: Ported to 5.30.0 from
|
|
|
9eebde9 |
933e3e630076d4fdbe32a101eeb5f12e37ec4ac2.
|
|
|
9eebde9 |
|
|
|
9eebde9 |
Signed-off-by: Petr Písař <ppisar@redhat.com>
|
|
|
9eebde9 |
---
|
|
|
9eebde9 |
perl.h | 17 +++++++++++++++++
|
|
|
9eebde9 |
pp_ctl.c | 10 ++++++++--
|
|
|
9eebde9 |
t/lib/croak/pp_ctl | 8 ++++++++
|
|
|
9eebde9 |
3 files changed, 33 insertions(+), 2 deletions(-)
|
|
|
9eebde9 |
|
|
|
9eebde9 |
diff --git a/perl.h b/perl.h
|
|
|
9eebde9 |
index e5a5585..383487c 100644
|
|
|
9eebde9 |
--- a/perl.h
|
|
|
9eebde9 |
+++ b/perl.h
|
|
|
9eebde9 |
@@ -1357,6 +1357,23 @@ EXTERN_C char *crypt(const char *, const char *);
|
|
|
9eebde9 |
} \
|
|
|
9eebde9 |
} STMT_END
|
|
|
9eebde9 |
|
|
|
9eebde9 |
+/* contains inlined gv_add_by_type */
|
|
|
9eebde9 |
+#define SANE_ERRSV() STMT_START { \
|
|
|
9eebde9 |
+ SV ** const svp = &GvSV(PL_errgv); \
|
|
|
9eebde9 |
+ if (!*svp) { \
|
|
|
9eebde9 |
+ *svp = newSVpvs(""); \
|
|
|
9eebde9 |
+ } else if (SvREADONLY(*svp)) { \
|
|
|
9eebde9 |
+ SV *dupsv = newSVsv(*svp); \
|
|
|
9eebde9 |
+ SvREFCNT_dec_NN(*svp); \
|
|
|
9eebde9 |
+ *svp = dupsv; \
|
|
|
9eebde9 |
+ } else { \
|
|
|
9eebde9 |
+ SV *const errsv = *svp; \
|
|
|
9eebde9 |
+ if (SvMAGICAL(errsv)) { \
|
|
|
9eebde9 |
+ mg_free(errsv); \
|
|
|
9eebde9 |
+ } \
|
|
|
9eebde9 |
+ } \
|
|
|
9eebde9 |
+ } STMT_END
|
|
|
9eebde9 |
+
|
|
|
9eebde9 |
|
|
|
9eebde9 |
#ifdef PERL_CORE
|
|
|
9eebde9 |
# define DEFSV (0 + GvSVn(PL_defgv))
|
|
|
9eebde9 |
diff --git a/pp_ctl.c b/pp_ctl.c
|
|
|
9eebde9 |
index a38b9c1..1f2d812 100644
|
|
|
9eebde9 |
--- a/pp_ctl.c
|
|
|
9eebde9 |
+++ b/pp_ctl.c
|
|
|
9eebde9 |
@@ -1720,9 +1720,13 @@ Perl_die_unwind(pTHX_ SV *msv)
|
|
|
9eebde9 |
* perls 5.13.{1..7} which had late setting of $@ without this
|
|
|
9eebde9 |
* early-setting hack.
|
|
|
9eebde9 |
*/
|
|
|
9eebde9 |
- if (!(in_eval & EVAL_KEEPERR))
|
|
|
9eebde9 |
+ if (!(in_eval & EVAL_KEEPERR)) {
|
|
|
9eebde9 |
+ /* remove any read-only/magic from the SV, so we don't
|
|
|
9eebde9 |
+ get infinite recursion when setting ERRSV */
|
|
|
9eebde9 |
+ SANE_ERRSV();
|
|
|
9eebde9 |
sv_setsv_flags(ERRSV, exceptsv,
|
|
|
9eebde9 |
(SV_GMAGIC|SV_DO_COW_SVSETSV|SV_NOSTEAL));
|
|
|
9eebde9 |
+ }
|
|
|
9eebde9 |
|
|
|
9eebde9 |
if (in_eval & EVAL_KEEPERR) {
|
|
|
9eebde9 |
Perl_ck_warner(aTHX_ packWARN(WARN_MISC), "\t(in cleanup) %" SVf,
|
|
|
9eebde9 |
@@ -1784,8 +1788,10 @@ Perl_die_unwind(pTHX_ SV *msv)
|
|
|
9eebde9 |
*/
|
|
|
9eebde9 |
S_pop_eval_context_maybe_croak(aTHX_ cx, exceptsv, 2);
|
|
|
9eebde9 |
|
|
|
9eebde9 |
- if (!(in_eval & EVAL_KEEPERR))
|
|
|
9eebde9 |
+ if (!(in_eval & EVAL_KEEPERR)) {
|
|
|
9eebde9 |
+ SANE_ERRSV();
|
|
|
9eebde9 |
sv_setsv(ERRSV, exceptsv);
|
|
|
9eebde9 |
+ }
|
|
|
9eebde9 |
PL_restartjmpenv = restartjmpenv;
|
|
|
9eebde9 |
PL_restartop = restartop;
|
|
|
9eebde9 |
JMPENV_JUMP(3);
|
|
|
9eebde9 |
diff --git a/t/lib/croak/pp_ctl b/t/lib/croak/pp_ctl
|
|
|
9eebde9 |
index b1e754c..de0221b 100644
|
|
|
9eebde9 |
--- a/t/lib/croak/pp_ctl
|
|
|
9eebde9 |
+++ b/t/lib/croak/pp_ctl
|
|
|
9eebde9 |
@@ -51,3 +51,11 @@ use 5.01;
|
|
|
9eebde9 |
default{}
|
|
|
9eebde9 |
EXPECT
|
|
|
9eebde9 |
Can't "default" outside a topicalizer at - line 2.
|
|
|
9eebde9 |
+########
|
|
|
9eebde9 |
+# NAME croak with read only $@
|
|
|
9eebde9 |
+eval '"a" =~ /${*@=\_})/';
|
|
|
9eebde9 |
+die;
|
|
|
9eebde9 |
+# this would previously recurse infinitely in the eval
|
|
|
9eebde9 |
+EXPECT
|
|
|
9eebde9 |
+Unmatched ) in regex; marked by <-- HERE in m/_) <-- HERE / at (eval 1) line 1.
|
|
|
9eebde9 |
+ ...propagated at - line 2.
|
|
|
9eebde9 |
--
|
|
|
9eebde9 |
2.21.0
|
|
|
9eebde9 |
|