cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: curl_easy_duphandle issue

From: Daniel Stenberg <daniel_at_haxx.se>
Date: Mon, 9 Mar 2009 09:32:35 +0100 (CET)

On Mon, 9 Mar 2009, Frank Hempel wrote:

> I thought the patch would do this already. Say one curl_slist_append() call
> fails and results NULL. But the input list to curl_slist_append(), which is
> outcurl->change.cookielist, remains as it is and is not overwritten with
> NULL (cl_src), due to the break before. So it will be freed with
> curl_slist_free_all() later (second part of the patch; maybe you just
> overlooked that? Or maybe I'm just too blind to see...).

Gah, oops. Sorry! You're right, I didn't read it through properly. I'll
officially widthdraw the nit.

-- 
  / daniel.haxx.se
Received on 2009-03-09