Brian Pane <[EMAIL PROTECTED]> writes:
> [EMAIL PROTECTED] wrote:
>
> > - if (b->pattern ? apr_strmatch(b->pattern, val, val_len) :
> > + if ((b->pattern && apr_strmatch(b->pattern, val, val_len)) ||
> > !ap_regexec(b->preg, val, 0, NULL, 0)) {
> >
>
> Those two expressions aren't equivalent. If b->pattern is nonzero
> but the apr_strmatch fails, the "||" version will fall through into
> the ap_regexec--which will segfault because the regex is going to be
> null if b->pattern is non-null.
thanks!
--
Jeff Trawick | [EMAIL PROTECTED]
Born in Roswell... married an alien...
