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

Reply via email to