2013/7/18 Dave Reisner <[email protected]>:
> On Thu, Jul 18, 2013 at 04:43:23PM +0400, Evgeny Turnaev wrote:
>> 2013/7/18 Daniel Stenberg <[email protected]>:
>> > On Wed, 17 Jul 2013, Evgeny Turnaev wrote:
>> >
>> >> Diff in attach.
>> >
>> >
>> > Thanks, I edited it very slightly and pushed!
>>
>> Uhh. Sorry. I didnt tested it with real curl requests until now.
>> There is major flaw in patch.
>> Fix in attachment.
>> Sorry.
>>
>
> This seems like it's still wrong. You're clobbering the value of 'i',
> which might be used to store the return from Curl_poll, and possibly
> assigned to *ret. You need a separate variable to iterate through
> extra_fds.revents[].

You are right Dave.
Another fix in attachment.

>
>
>> >
>> > --
>> >
>> >  / daniel.haxx.se
>> >
>> > -------------------------------------------------------------------
>> > List admin: http://cool.haxx.se/list/listinfo/curl-library
>> > Etiquette:  http://curl.haxx.se/mail/etiquette.html
>
>
>> -------------------------------------------------------------------
>> List admin: http://cool.haxx.se/list/listinfo/curl-library
>> Etiquette:  http://curl.haxx.se/mail/etiquette.html
>
> -------------------------------------------------------------------
> List admin: http://cool.haxx.se/list/listinfo/curl-library
> Etiquette:  http://curl.haxx.se/mail/etiquette.html

Attachment: multi.c.diff.2
Description: Binary data

-------------------------------------------------------------------
List admin: http://cool.haxx.se/list/listinfo/curl-library
Etiquette:  http://curl.haxx.se/mail/etiquette.html

Reply via email to