From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: MIME-version: 1.0 Content-type: text/plain; charset=UTF-8 Message-id: <5278A066.5020409@samsung.com> Date: Tue, 05 Nov 2013 16:38:14 +0900 From: Seung-Woo Kim Reply-to: sw0312.kim@samsung.com To: Marcel Holtmann Cc: "linux-bluetooth@vger.kernel.org development" , "Gustavo F. Padovan" , Johan Hedberg , Syam Sidhardhan , Seung-Woo Kim Subject: Re: [PATCH] Bluetooth: Fix RFCOMM bind fail for L2CAP sock References: <1383634944-4141-1-git-send-email-sw0312.kim@samsung.com> In-reply-to: List-ID: Hello Marcel Holtmann, On 2013년 11월 05일 16:26, Marcel Holtmann wrote: > Hi Seung-Woo, > >> L2CAP socket bind checks its bdaddr type but RFCOMM kernel thread >> does not assign proper bdaddr type for L2CAP sock. This can cause >> that RFCOMM failure. >> >> Signed-off-by: Seung-Woo Kim >> --- >> The issue is came from following commit, but it is only in bluetooth-next tree, >> so this patch is not needed for stable tree. >> >> commit 80c1a2e76dbb36588facb265ab364b1302f7ef8c >> Author: Johan Hedberg >> Date: Mon Oct 14 21:17:52 2013 +0300 >> >> Bluetooth: Reject invalid bdaddr types for sockets >> >> This patch is based on commit cca34b785a21d220176038c45071ffe842e99fd3 of >> bluetooth-next tree. >> --- >> net/bluetooth/rfcomm/core.c | 2 ++ >> 1 files changed, 2 insertions(+), 0 deletions(-) > > good catch. Patch has been applied to bluetooth-next tree. > > You might also want to send a patch that adds the l2_bdaddr_type setting before the connect() call. Case of kernel_connect() in rfcomm_session_create(), it reuses l2_bdaddr_type used for kernel_bind(), so it works fine for me: I tested both incoming connection and outgoing connection cases of rfcomm. But if you prefer explicit setting for connect(), please let me know. Best Regards, - Seung-Woo Kim > > Regards > > Marcel > > -- Seung-Woo Kim Samsung Software R&D Center --