On Wed, Oct 7, 2015 at 8:45 PM, Ruediger Pluem <rpl...@apache.org> wrote: > > > On 10/07/2015 12:23 AM, yla...@apache.org wrote: >> >> static void eor_bucket_destroy(void *data) >> { >> - request_rec *r = (request_rec *)data; >> + apr_bucket *h = data; > > IMHO this should be > > apr_bucket_refcount *h;
Good catch! Actually, we even need: typedef struct { apr_bucket_refcount refcount; request_rec *data; } apr_bucket_eor; and use this type everywhere for h. Thanks, Yann.