All of lore.kernel.org
 help / color / mirror / Atom feed
* [Qemu-devel] [PATCH v2] curl: Don't set curl options on the handle just before it's going to be deleted.
@ 2013-06-10 16:40 Richard W.M. Jones
  2013-06-11  7:50 ` Stefan Hajnoczi
  2013-06-11  8:22 ` Kevin Wolf
  0 siblings, 2 replies; 3+ messages in thread
From: Richard W.M. Jones @ 2013-06-10 16:40 UTC (permalink / raw)
  To: qemu-devel; +Cc: kwolf, stefanha

From: "Richard W.M. Jones" <rjones@redhat.com>

(Found by Kamil Dudka)

Signed-off-by: Richard W.M. Jones <rjones@redhat.com>
Cc: Michael Tokarev <mjt@tls.msk.ru>
---
 block/curl.c | 2 --
 1 file changed, 2 deletions(-)

diff --git a/block/curl.c b/block/curl.c
index b634ccf..8d0d45d 100644
--- a/block/curl.c
+++ b/block/curl.c
@@ -452,8 +452,6 @@ static int curl_open(BlockDriverState *bs, QDict *options, int flags)
     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)
-- 
1.8.2.1

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [PATCH v2] curl: Don't set curl options on the handle just before it's going to be deleted.
  2013-06-10 16:40 [Qemu-devel] [PATCH v2] curl: Don't set curl options on the handle just before it's going to be deleted Richard W.M. Jones
@ 2013-06-11  7:50 ` Stefan Hajnoczi
  2013-06-11  8:22 ` Kevin Wolf
  1 sibling, 0 replies; 3+ messages in thread
From: Stefan Hajnoczi @ 2013-06-11  7:50 UTC (permalink / raw)
  To: Richard W.M. Jones; +Cc: kwolf, Fam Zheng, qemu-devel, stefanha

On Mon, Jun 10, 2013 at 05:40:15PM +0100, Richard W.M. Jones wrote:
> From: "Richard W.M. Jones" <rjones@redhat.com>
> 
> (Found by Kamil Dudka)
> 
> Signed-off-by: Richard W.M. Jones <rjones@redhat.com>
> Cc: Michael Tokarev <mjt@tls.msk.ru>
> ---
>  block/curl.c | 2 --
>  1 file changed, 2 deletions(-)

Acked-by: Stefan Hajnoczi <stefanha@redhat.com>

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Qemu-devel] [PATCH v2] curl: Don't set curl options on the handle just before it's going to be deleted.
  2013-06-10 16:40 [Qemu-devel] [PATCH v2] curl: Don't set curl options on the handle just before it's going to be deleted Richard W.M. Jones
  2013-06-11  7:50 ` Stefan Hajnoczi
@ 2013-06-11  8:22 ` Kevin Wolf
  1 sibling, 0 replies; 3+ messages in thread
From: Kevin Wolf @ 2013-06-11  8:22 UTC (permalink / raw)
  To: Richard W.M. Jones; +Cc: qemu-devel, stefanha

Am 10.06.2013 um 18:40 hat Richard W.M. Jones geschrieben:
> From: "Richard W.M. Jones" <rjones@redhat.com>
> 
> (Found by Kamil Dudka)
> 
> Signed-off-by: Richard W.M. Jones <rjones@redhat.com>
> Cc: Michael Tokarev <mjt@tls.msk.ru>

Thanks, applied to the block branch.

Kevin

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2013-06-11  8:22 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2013-06-10 16:40 [Qemu-devel] [PATCH v2] curl: Don't set curl options on the handle just before it's going to be deleted Richard W.M. Jones
2013-06-11  7:50 ` Stefan Hajnoczi
2013-06-11  8:22 ` Kevin Wolf

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.