Re: [PATCH obexd v2] Fix handling asynchronous plugin reads

[Johan Hedberg <johan.hedberg@gmail.com>]

Hi Slawek,

On Fri, Aug 12, 2011, Slawomir Bochenski wrote:
> Calling OBEX_ResumeRequest() from handle_async_io() may result in direct
> calling obex_event_cb() (this happens when obex_write_stream() will
> deliver not enough bytes to fully fill OpenOBEX TX packet). In this case
> set_io_watch will fail if handle_async_io() is called from
> obex_object_set_io_flags(), because the watch is already installed.
> Originally when code returns from OBEX_ResumeRequest(), handle_async_io()
> returns FALSE which makes obex_object_set_io_flags() remove this watch.
> 
> This patch adds variable for tracking whether subsequent calls suspended
> get request, causing obex_object_set_io_flags() remove the watch only
> when the request is not suspended.
> 
> Additionaly member 'streaming' of obex_session has been renamed to
> 'stream_open'.
> ---
>  src/obex-priv.h |    3 ++-
>  src/obex.c      |   14 +++++++++-----
>  2 files changed, 11 insertions(+), 6 deletions(-)

Applied. Thanks.

Johan