From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from eggs.gnu.org ([2001:4830:134:3::10]:40433) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1Um54X-0006sI-FZ for qemu-devel@nongnu.org; Mon, 10 Jun 2013 12:35:59 -0400 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1Um54V-000245-MV for qemu-devel@nongnu.org; Mon, 10 Jun 2013 12:35:57 -0400 Date: Mon, 10 Jun 2013 17:35:50 +0100 From: "Richard W.M. Jones" Message-ID: <20130610163550.GD4515@redhat.com> References: <1370870347-22080-1-git-send-email-rjones@redhat.com> <51B5F21C.9020700@msgid.tls.msk.ru> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <51B5F21C.9020700@msgid.tls.msk.ru> Subject: Re: [Qemu-devel] [Qemu-trivial] [PATCH] curl: Don't set curl options on the handle just before it's going to be deleted. List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , To: Michael Tokarev Cc: qemu-trivial@nongnu.org, kwolf@redhat.com, qemu-devel@nongnu.org, stefanha@redhat.com On Mon, Jun 10, 2013 at 07:34:52PM +0400, Michael Tokarev wrote: > 10.06.2013 17:19, Richard W.M. Jones wrote: > > From: "Richard W.M. Jones" > > > > (Found by Kamil Dudka) > > > > Signed-off-by: Richard W.M. Jones > > --- > > block/curl.c | 1 - > > 1 file changed, 1 deletion(-) > > > > diff --git a/block/curl.c b/block/curl.c > > index b634ccf..bf31efe 100644 > > --- a/block/curl.c > > +++ b/block/curl.c > > @@ -453,7 +453,6 @@ static int curl_open(BlockDriverState *bs, QDict *options, int flags) > > goto out; > > curl_easy_getinfo(state->curl, CURLINFO_CONTENT_LENGTH_DOWNLOAD, &d); > > curl_easy_setopt(state->curl, CURLOPT_WRITEFUNCTION, (void *)curl_read_cb); > > - curl_easy_setopt(state->curl, CURLOPT_NOBODY, 0); > > if (d) > > s->len = (size_t)d; > > else if(!s->len) > > Adding a bit more context: > > > curl_easy_setopt(state->curl, CURLOPT_NOBODY, 1); > curl_easy_setopt(state->curl, CURLOPT_WRITEFUNCTION, (void *)curl_size_cb); > if (curl_easy_perform(state->curl)) > goto out; > curl_easy_getinfo(state->curl, CURLINFO_CONTENT_LENGTH_DOWNLOAD, &d); > curl_easy_setopt(state->curl, CURLOPT_WRITEFUNCTION, (void *)curl_read_cb); > =>> curl_easy_setopt(state->curl, CURLOPT_NOBODY, 0); > if (d) > s->len = (size_t)d; > else if(!s->len) > goto out; > DPRINTF("CURL: Size = %zd\n", s->len); > > curl_clean_state(state); > curl_easy_cleanup(state->curl); > state->curl = NULL; > > So indeed, the curl object is being removed here a few lines down the code. > > But it looks like the previous call to curl_easy_setopt, to set WRITEFUNCTION, > is also not very useful. Should we probably want to remove that too, or don't > remove either of these? Yes you're right. For some reason I "saw" another call to curl_easy_perform which obviously isn't there in reality. Will send a v2 patch shortly after I've done a bit of testing. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones virt-top is 'top' for virtual machines. Tiny program with many powerful monitoring features, net stats, disk stats, logging, etc. http://people.redhat.com/~rjones/virt-top