Hi Jaganath,
On Thu, Mar 08, 2012, Jaganath Kanakkassery wrote:
> @@ -1335,6 +1335,12 @@ void g_obex_unref(GObex *obex)
>
> g_slist_free_full(obex->req_handlers, g_free);
>
> + do {
> + ret = write_data(obex->io, G_IO_OUT, obex);
> + if (obex->pending_req && obex->pending_req->cancelled)
> + break;
> + } while(ret);
This is not ok since we should only attempt writing to the transport if
G_IO_OUT is *really* set and not just fake it. Otherwise the call might
block which is not acceptable for the way gobex is designed (to be used
with a single async mainloop).
Johan
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
