[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read
From: |
Fam Zheng |
Subject: |
Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read |
Date: |
Tue, 21 May 2013 09:54:15 +0800 |
User-agent: |
Mutt/1.5.21 (2010-09-15) |
On Mon, 05/20 09:49, Richard W.M. Jones wrote:
> On Mon, May 20, 2013 at 09:41:06AM +0100, Richard W.M. Jones wrote:
> > On Mon, May 20, 2013 at 03:03:34PM +0800, Fam Zheng wrote:
> > > CURL library API has changed, the current curl driver is not working.
> > > This patch rewrites the use of API as well as the structure of internal
> > > states.
> >
> > I tried this, but it segfaults:
> >
> > Program terminated with signal 11, Segmentation fault.
>
> That stack trace was wrong. I was testing against the version of
> libcurl in Fedora which is known to be broken.
>
> Here is the stack trace, this time really running against
> curl-7_30_0-147-gae26ee3:
>
> Program terminated with signal 11, Segmentation fault.
> #0 curl_read_cb (ptr=<optimized out>, size=<optimized out>,
> nmemb=<optimized out>, opaque=0x7f63d48ba340) at block/curl.c:240
> 240 size_t aio_base = acb->sector_num * SECTOR_SIZE;
Looks like a memory corrupt (QLIST head is invalid pointer). But I can't
reproduce here with your steps. Can you try qemu-io?
$LD_LIBRARY_PATH=~/d/curl/lib/.libs ~/d/qemu/qemu-io
http://192.168.0.249/scratch/winxp.img -c 'read 0 512'
Thanks.
--
Fam
- [Qemu-devel] [PATCH v3 03/10] curl: change curl_multi_do to curl_fd_handler, (continued)
- [Qemu-devel] [PATCH v3 03/10] curl: change curl_multi_do to curl_fd_handler, Fam Zheng, 2013/05/20
- [Qemu-devel] [PATCH v3 04/10] curl: fix curl_open, Fam Zheng, 2013/05/20
- [Qemu-devel] [PATCH v3 05/10] curl: add timer to BDRVCURLState, Fam Zheng, 2013/05/20
- [Qemu-devel] [PATCH v3 06/10] curl: introduce CURLDataCache, Fam Zheng, 2013/05/20
- [Qemu-devel] [PATCH v3 07/10] curl: make use of CURLDataCache., Fam Zheng, 2013/05/20
- [Qemu-devel] [PATCH v3 08/10] curl: use list to store CURLState, Fam Zheng, 2013/05/20
- [Qemu-devel] [PATCH v3 09/10] curl: add cache quota., Fam Zheng, 2013/05/20
- [Qemu-devel] [PATCH v3 10/10] curl: introduce ssl_no_cert runtime option., Fam Zheng, 2013/05/20
- Re: [Qemu-devel] [PATCH v3 00/10] curl: fix curl read, Richard W.M. Jones, 2013/05/20