On Fri, Sep 12, 2014 at 4:57 PM, Yann Ylavic <ylavic....@gmail.com> wrote: > AP_DECLARE(apr_status_t) ap_filter_setaside_brigade(ap_filter_t *f, > ap_filter_aside_t **aside, > apr_bucket_brigade *bb) > { > if (!APR_BRIGADE_EMPTY(bb)) { > [...] > } > else if ((*aside)->pool) {
We'd probably better check *aside != NULL before here too. > [...] > } > return APR_SUCCESS; > } I think that's all :-p