Hi Daniel, On Sun, Feb 21, 2010, Daniel Örstadius wrote: > > Please try to at least do a compile check before you submit patches. > > This one gives the following error: > > audio/avdtp.c: In function ‘handle_unanswered_req’: > > audio/avdtp.c:908: error: comparison between pointer and integer > > > > What you probably want is session->req->signal_id == AVDTP_ABORT. > > > > Sorry. The attached patch compiles without warnings on "./configure && make". > (new dependency to a capabilities lib prevents ./bootstrap-configure atm) You can avoid that with "./bootstrap-configure --disable-capng" > The pending request might be freed twice when receiving an Abort > response, in handle_unanswered_req and session_cb. Avoid freeing > it in handle_unanswered_req. > --- > audio/avdtp.c | 7 +++++++ > 1 files changed, 7 insertions(+), 0 deletions(-) Thanks, the patch is now upstream. 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
