Re: [Qemu-devel] [PATCH v6 00/12] curl: fix curl read

[Richard W.M. Jones]

[I chopped the CC list down]

On Tue, May 28, 2013 at 07:14:01PM +0800, Fam Zheng wrote:
> There seems no easy way to me to inject sleep to io reqs in curl.c.
> Another option might be configure & compile qemu with
> CFLAGS=-DDEBUG_CURL and grab the stdout, so that we can see how requests
> are processed.

No problems.

I started with v6 patch.  I did:

$ CFLAGS=-DDEBUG_CURL ./configure --enable-werror --target-list="i386-softmmu 
x86_64-softmmu"
$ make

Note there is a bug!

block/curl.c: In function ‘curl_complete_io’:
block/curl.c:211:5: error: too many arguments for format 
[-Werror=format-extra-args]
cc1: all warnings being treated as errors
make: *** [block/curl.o] Error 1

Please add the attached patch to your patch set.

After fixing that and continuing the build, I then ran:

$ /tmp/test.sh >& /tmp/curl-debug.log

This shows the error and produces an absolutely vast amount of
debugging (attached).  Note that I killed it after a while but there
should be enough to show the error.

Rich.

-- 
Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones
Fedora Windows cross-compiler. Compile Windows programs, test, and
build Windows installers. Over 100 libraries supported.
http://fedoraproject.org/wiki/MinGW

0001-curl-Fix-format-string-in-debugging-statement.patch
Description: Text document

curl-debug.log.xz
Description: application/xz