apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Tom Donovan <donov...@bellatlantic.net>
Subject Re: [Fwd: connobject.c broke with apr 1.3.2]
Date Mon, 30 Jun 2008 18:28:34 GMT
William A. Rowe, Jr. wrote:
> Guys, please take a look, looks like one of the 1.3.x generation patches
> has broken our API compatibility rules.
It looks like this has always been wrong in mod_python.

APR_BRIGADE_SENTINEL takes a brigade argument, not a bucket.


I think no error displayed pre-1.3.2 because, unfortunately, both buckets and brigades have
a member 
named 'list':

    For buckets - 'list' is a pointer to the allocator function.
    For brigades, 'list' is the ring of buckets.

The condition was probably never true when he was checking the wrong kind of 'list' for a
sentinel.  Well, hopefully it was never true!


View raw message