httpd-bugs mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From bugzi...@apache.org
Subject DO NOT REPLY [Bug 52623] does not build with pcre-8.30
Date Sun, 12 Feb 2012 08:47:22 GMT
https://issues.apache.org/bugzilla/show_bug.cgi?id=52623

--- Comment #7 from Oden Eriksson <oeriksson@mandriva.com> 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: bugs-unsubscribe@httpd.apache.org
For additional commands, e-mail: bugs-help@httpd.apache.org


Mime
View raw message