New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Possible memory leak in S_regclass #16293
Comments
From pipcet@gmail.comCreated by pipcet@gmail.comI believe the variable only_non_utf8_list in regcomp.c should be mortalized Inline Patchdiff --git a/regcomp.c b/regcomp.c
index 29af44c7a0..3ba091af8f 100644
--- a/regcomp.c
+++ b/regcomp.c
@@ -17721,6 +17721,8 @@ S_regclass(pTHX_ RExC_state_t *pRExC_state,
Perl Info
|
From pipcet@gmail.comperl-bug-008.diffdiff --git a/regcomp.c b/regcomp.c
index 29af44c7a0..3ba091af8f 100644
--- a/regcomp.c
+++ b/regcomp.c
@@ -17721,6 +17721,8 @@ S_regclass(pTHX_ RExC_state_t *pRExC_state, I32 *flagp, U32 depth,
if (_invlist_len(only_non_utf8_list) != 0) {
ANYOF_FLAGS(ret) |= ANYOF_SHARED_d_MATCHES_ALL_NON_UTF8_NON_ASCII_non_d_WARN_SUPER;
}
+
+ SvREFCNT_dec_NN(only_non_utf8_list);
}
else {
/* Here there were no complemented posix classes. That means
|
@khwilliamson - Status changed from 'new' to 'open' |
From @khwilliamsonThanks for reporting this. Fixed by 4ee748a |
@khwilliamson - Status changed from 'open' to 'pending release' |
From @khwilliamsonThank you for filing this report. You have helped make Perl better. With the release yesterday of Perl 5.28.0, this and 185 other issues have been Perl 5.28.0 may be downloaded via: If you find that the problem persists, feel free to reopen this ticket. |
@khwilliamson - Status changed from 'pending release' to 'resolved' |
Migrated from rt.perl.org#132548 (status was 'resolved')
Searchable as RT132548$
The text was updated successfully, but these errors were encountered: