summaryrefslogtreecommitdiff
path: root/scope.c
diff options
context:
space:
mode:
authorDavid Mitchell <davem@iabyn.com>2020-03-20 09:06:57 +0000
committerDavid Mitchell <davem@iabyn.com>2020-03-20 09:06:57 +0000
commit1943af6140f72047c5028b50f947b52f8998bebd (patch)
treea02ab1e3de087e1e3bc503d0da81238ad9e9c197 /scope.c
parentb33ac5557cb0255702946345f30e9a5ef0e8d88f (diff)
downloadperl-1943af6140f72047c5028b50f947b52f8998bebd.tar.gz
fixup to free_and_set_cop_warnings()
v5.31.9-156-g94c8b9c1f0 introduced the free_and_set_cop_warnings() macro. It's first argument expects a COP rather than a COP*. Its usage in S_restore_cop_warnings(() is to modify PL_cucop, but in order to pass a COP rather than a COP*, the original commit made a local copy of PL_curcop and ended up inadvertently updating the copy instead. This commit changes the maco so it expects a COP*, and updates the bulk of its callers to use &PL_compiling rather than PL_compiling, and fixes up S_restore_cop_warnings(). The symptoms were ASAN failures in a few test scripts including uni/parser.t and ext/XS-APItest/t/handy0*.t. (The S_restore_cop_warnings() function was only used by Perl__force_out_malformed_utf8_message(), so didn't cause many issues outside of test scripts which forced such "malformed "errors).
Diffstat (limited to 'scope.c')
-rw-r--r--scope.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/scope.c b/scope.c
index ad8f8e764b..a948a7b4f2 100644
--- a/scope.c
+++ b/scope.c
@@ -1448,7 +1448,7 @@ Perl_leave_scope(pTHX_ I32 base)
case SAVEt_COMPILE_WARNINGS:
a0 = ap[0];
- free_and_set_cop_warnings(PL_compiling, (STRLEN*) a0.any_ptr);
+ free_and_set_cop_warnings(&PL_compiling, (STRLEN*) a0.any_ptr);
break;
case SAVEt_PARSER: