Projects
Factory:RISC-V:Gnome
glib2
_service:tar_scm:backport-regex-Compute-the-off...
Sign Up
Log In
Username
Password
Overview
Repositories
Revisions
Requests
Users
Attributes
Meta
File _service:tar_scm:backport-regex-Compute-the-offsets-size-based-on-match-results.patch of Package glib2
From e8628a7ed59e54b5a5e498de0375f101a4e76e64 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Marco=20Trevisan=20=28Trevi=C3=B1o=29?= <mail@3v1n0.net> Date: Tue, 6 Sep 2022 19:05:24 +0200 Subject: [PATCH] regex: Compute the offsets size based on match results While the ovector count would include all the allocated space, we only care about the actual match values, so avoid wasting allocations and just use the ones we need to hold the offsets. --- glib/gregex.c | 16 +++++++++++++--- 1 file changed, 13 insertions(+), 3 deletions(-) diff --git a/glib/gregex.c b/glib/gregex.c index cf86f0fe0d..8a3be9076b 100644 --- a/glib/gregex.c +++ b/glib/gregex.c @@ -832,10 +832,20 @@ recalc_match_offsets (GMatchInfo *match_info, GError **error) { PCRE2_SIZE *ovector; + uint32_t ovector_size = 0; uint32_t pre_n_offset; uint32_t i; - if (pcre2_get_ovector_count (match_info->match_data) > G_MAXUINT32 / 2) + g_assert (!IS_PCRE2_ERROR (match_info->matches)); + + if (match_info->matches == PCRE2_ERROR_PARTIAL) + ovector_size = 1; + else if (match_info->matches > 0) + ovector_size = match_info->matches; + + g_assert (ovector_size != 0); + + if (pcre2_get_ovector_count (match_info->match_data) < ovector_size) { g_set_error (error, G_REGEX_ERROR, G_REGEX_ERROR_MATCH, _("Error while matching regular expression %s: %s"), @@ -844,7 +854,7 @@ recalc_match_offsets (GMatchInfo *match_info, } pre_n_offset = match_info->n_offsets; - match_info->n_offsets = pcre2_get_ovector_count (match_info->match_data) * 2; + match_info->n_offsets = ovector_size * 2; ovector = pcre2_get_ovector_pointer (match_info->match_data); if (match_info->n_offsets != pre_n_offset) @@ -2387,7 +2397,7 @@ g_regex_match_all_full (const GRegex *regex, _("Error while matching regular expression %s: %s"), regex->pattern, match_error (info->matches)); } - else if (info->matches > 0) + else if (info->matches != PCRE2_ERROR_NOMATCH) { if (!recalc_match_offsets (info, error)) info->matches = PCRE2_ERROR_NOMATCH; -- GitLab
Locations
Projects
Search
Status Monitor
Help
Open Build Service
OBS Manuals
API Documentation
OBS Portal
Reporting a Bug
Contact
Mailing List
Forums
Chat (IRC)
Twitter
Open Build Service (OBS)
is an
openSUSE project
.
浙ICP备2022010568号-2