curl-library
Re: Curl_getoff_all_pipelines SIGSEGV
From: Daniel Stenberg <daniel_at_haxx.se>
Date: Mon, 4 May 2009 10:26:55 +0200 (CEST)
Date: Mon, 4 May 2009 10:26:55 +0200 (CEST)
On Fri, 1 May 2009, Lénaïc Lefever wrote:
> In the Curl_done method in url.c (curl-7.19.4), the
> Curl_getoff_all_pipelines call happens before the conn->bits.done check.
>
> Wouldn't it be safer to check for conn->bits.done before calling
> Curl_getoff_all_pipelines to avoid possible SIGSEGVs?
Yes I think it would. Did you run the test suite fine with that change?
In general I don't like precautions like that in the code, but I figure fixing
the real flaw is a bit too much work for anyone at this point in time.
-- / daniel.haxx.seReceived on 2009-05-04