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