On Thu, Feb 11, 2016 at 3:14 PM, Yann Ylavic <ylavic....@gmail.com> wrote:
> On Thu, Feb 11, 2016 at 3:10 PM, Ruediger Pluem <rpl...@apache.org> wrote:
>>
>> conn->ssl_hostname might be allocated from conn->scpool.
>> So the pointer might be invalid after socket_cleanup(conn). So you need to 
>> apr_pstrdup it first.
>> No real good idea which pool to use here.
>
> Oh, correct!
>
>> Probably create a temp subpool of conn->pool,
>> strdup, after socket_cleanup dup with conn->scpool and destroy tmp pool.
>
> Yes, possibly overkill, I have to find something else...

Maybe r1729847 is fine?

>
> Regards,
> Yann.

Reply via email to