https://issues.apache.org/bugzilla/show_bug.cgi?id=52623
--- Comment #7 from Oden Eriksson <[email protected]> 2012-02-12 08:47:22 UTC --- (In reply to comment #2) [...] > or even better: > > Index: server/util_pcre.c > =================================================================== > --- server/util_pcre.c (revision 1241805) > +++ server/util_pcre.c (working copy) > @@ -139,7 +139,8 @@ > if (preg->re_pcre == NULL) > return AP_REG_INVARG; > > - preg->re_nsub = pcre_info((const pcre *)preg->re_pcre, NULL, NULL); > + pcre_fullinfo((const pcre *)preg->re_pcre, NULL, > + PCRE_INFO_CAPTURECOUNT, &(preg->re_nsub)); > return 0; > } > > Index: include/ap_regex.h > =================================================================== > --- include/ap_regex.h (revision 1241805) > +++ include/ap_regex.h (working copy) > @@ -88,7 +88,7 @@ > /* The structure representing a compiled regular expression. */ > typedef struct { > void *re_pcre; > - apr_size_t re_nsub; > + int re_nsub; > apr_size_t re_erroffset; > } ap_regex_t; > > > > but this requires a major bump since it changes a public API. Can you please > test the above patches? Thanks Ruediger for your quick responce here. I used the second version of the fix and applied it, but unfortunately I have not yet found time to test this much. -- Configure bugmail: https://issues.apache.org/bugzilla/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are the assignee for the bug. --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
