Actually we should call MHD_connection_update_event_loop_info() in any
case after processing of data.
I already applied some patches to master. Check whether it helps.
--
Best Wishes,
Evgeny Grin
On 15.03.2017 21:41, Evgeny Grin wrote:
> Yep, you are right, this patch is more correct.
> "suspended" should be set by user callbacks called in this function, so
> code must check for suspended before doing other thing.
>
> -- Best Wishes, Evgeny Grin On 15.03.2017 16:00, Vitaliy T wrote:
>> Another version, variant for MHD_connection_handle_idle().
>>
>> Short description:
>> Inside the function MHD_connection_handle_idle() between the loop and
>> later calling MHD_connection_update_event_loop_info() a lot things may
>> happen.
>>
>> diff --git a/src/microhttpd/connection.c b/src/microhttpd/connection.c
>> index f192a1bb..ab542913 100644
>> --- a/src/microhttpd/connection.c
>> +++ b/src/microhttpd/connection.c
>> @@ -3211,6 +3211,10 @@ MHD_connection_handle_idle (struct
>> MHD_Connection *connection)
>> }
>> break;
>> }
>> + if (connection->suspended) {
>> + connection->in_idle = false;
>> + return MHD_YES;
>> + }
>> timeout = connection->connection_timeout;
>> if ( (0 != timeout) &&
>> (timeout <= (MHD_monotonic_sec_counter() -
>> connection->last_activity)) )
>>
>>
>> Can't say which version is better, good, bad, either valid or invalid.
>> I hope the bug will be fixed somehow.
>> Thanks.
>>
