libmicrohttpd
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [libmicrohttpd] [PATCH]: MHD_connection_update_event_loop_info sends


From: silvioprog
Subject: Re: [libmicrohttpd] [PATCH]: MHD_connection_update_event_loop_info sends INTERNAL_ERROR for suspended connections
Date: Wed, 15 Mar 2017 18:43:20 -0300

Hello Vitaliy,

Sorry, but after applying that changes, could you share a small example showing how to use MHD with external epoll? If so, it could be very useful for the community, and MHD core could distribute it in the examples directory. :-)

On Wed, Mar 15, 2017 at 6:29 PM, Vitaliy T <address@hidden> wrote:
On 15 March 2017 at 23:54, Evgeny Grin <address@hidden> wrote:
> That's why I asked about user code example. I can't test what we trying
> to fix.

I understand you wish clearly. I will provide the code tomorrow.

> Did you test it or only review?

I am always do test the code, because there are might be regressions
or new bugs :)

> Seems that MHD read data from network after you suspended connection,
> which cause overflow buffer overflow and error response.
> You issue should be fixed by afb221c32aeeb2731e3b5362cc4ff8949d34edea.

I did not tested this commit yet.

> Disconnect by timeout for suspended connections should be fixed, so you
> can stop playing with timeout values.

Good. But I see that the root of the problem is not solved. I told
already about this check:

   timeout <= (MHD_monotonic_sec_counter() - connection->last_activity)

There is must be call inside the function MHD_resume_connection()
which will do next (or similar):

   connection->last_activity = MHD_monotonic_sec_counter();

Anyway, I will test latest patches and either raise this suggestion
again or do not.

> You mentioned that you had some problems in master with epoll and
> timeouts. Is it still actual?

epoll is not actual. I have solved most (see below) issues with epoll.

There are other issues with timeouts raised, I will explain them after
testing latest patches (or I will not if all is ok). At least, overall
work of MHD is much better than was before.

Thanks!

--
With Best Regards,
Vitaliy V. Tokarev

--
Silvio Clécio

reply via email to

[Prev in Thread] Current Thread [Next in Thread]