From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Subject: Re: [PATCH] Fix double free on AVDTP Abort response From: Marcel Holtmann To: Daniel =?ISO-8859-1?Q?=D6rstadius?= Cc: linux-bluetooth@vger.kernel.org In-Reply-To: References: <20100219173257.GA12855@jh-x301> Content-Type: text/plain; charset="UTF-8" Date: Sun, 21 Feb 2010 14:04:09 +0100 Message-ID: <1266757449.18491.17.camel@violet> Mime-Version: 1.0 Sender: linux-bluetooth-owner@vger.kernel.org List-ID: Hi Daniel, > > 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 use ./bootstrap-configure --disable-capng. Regards Marcel