cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] - master_buffer alloc only on pipelining set on

From: Robert Iakobashvili <coroberti_at_gmail.com>
Date: Tue, 24 Apr 2007 14:56:11 +0200

On 4/24/07, Daniel Stenberg <daniel_at_haxx.se> wrote:
> On Mon, 23 Apr 2007, Robert Iakobashvili wrote:
>
> > It has passed tests for me smoothly, including 530 several times.
> >
> > Could you, please, send to me the configuration options that you use?
>
> Actually, it surprises me that you could run this through test 530 as it
> failed for me 100%. Did you possibly change the patch after your last post to
> the list?
No, but it passed, don't no why.

>
> In CreateConnection() you did the free of the master buffer with the wrong
> input pointer (conn instead of old_conn)...

Yes, you are right.

>
> But, I fixed that and then the tests ran fine and then I committed this.
>
> Thanks!

Sometimes, some matters suddenly take a tough path.
Thank you for your patience and fix.

-- 
Sincerely,
Robert Iakobashvili,
coroberti %x40 gmail %x2e com
...................................................................
Navigare necesse est, vivere non est necesse
...................................................................
http://curl-loader.sourceforge.net
An open-source HTTP/S, FTP/S traffic
generating, and web testing tool.
Received on 2007-04-24