[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 00/10] curl: fix curl read
From: |
Stefan Hajnoczi |
Subject: |
Re: [Qemu-devel] [PATCH v4 00/10] curl: fix curl read |
Date: |
Wed, 22 May 2013 16:39:45 +0200 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Wed, May 22, 2013 at 12:12:24PM +0100, Richard W.M. Jones wrote:
> On Wed, May 22, 2013 at 01:04:51PM +0200, Paolo Bonzini wrote:
> > Something is trying to write, but there's no write operation defined for
> > CURL.
> >
> > I guess curl (and other backends too) should reject being opened for
> > write. Alternatively, block.c could do that for them.
>
> Yes, I'd just got to that conclusion as well :-)
>
> The attached patch fixes the crash for me.
Please post a top-level thread so that your patch gets picked up by
scripts and noticed by humans too :).
Alternatively Fam can include it in the next revision of this series.
Stefan
- [Qemu-devel] [PATCH v4 05/10] curl: add timer to BDRVCURLState, (continued)
- [Qemu-devel] [PATCH v4 05/10] curl: add timer to BDRVCURLState, Fam Zheng, 2013/05/21
- [Qemu-devel] [PATCH v4 06/10] curl: introduce CURLDataCache, Fam Zheng, 2013/05/21
- [Qemu-devel] [PATCH v4 07/10] curl: make use of CURLDataCache., Fam Zheng, 2013/05/21
- [Qemu-devel] [PATCH v4 08/10] curl: use list to store CURLState, Fam Zheng, 2013/05/21
- [Qemu-devel] [PATCH v4 09/10] curl: add cache quota., Fam Zheng, 2013/05/21
- [Qemu-devel] [PATCH v4 10/10] curl: introduce ssl_no_cert runtime option., Fam Zheng, 2013/05/21
- Re: [Qemu-devel] [PATCH v4 00/10] curl: fix curl read, Fam Zheng, 2013/05/22
- Re: [Qemu-devel] [PATCH v4 00/10] curl: fix curl read, Richard W.M. Jones, 2013/05/22