cURL / Mailing Lists / curl-library / Single Mail

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)

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.se
Received on 2009-05-04