Re: [Bluez-devel] SPP connections

[Mayank BATRA <mayank.batra@st.com>]

[-- Attachment #1: Type: text/plain, Size: 820 bytes --]

Hi Mark,

> I have a program and listen for connections from a device that uses
> SPP.  My program just opens RFCOMM socket and waits for connection to
> come in.  The device that connects into my program has a connection LED
> indicating a successful connections.
> 
> I have noticed that if close my RFCOMM socket, even if I stop my
> listener process, the connection LED on the device remains lit.  The
> only thing I get it to not lit (and thus indicates that it is giving up
> on that connection) is to remove my USB bluetooth dongle.

I also faced the same problem sometime ago and Marcel gave me the
attached patch which fixed the problem.
By the way, which kernel are you using? This problem is there only in
older kernel versions.

Best Regards,

Mayank

--
Random Quote Follows:
Recursive, adj.; see Recursive

[-- Attachment #2: patch-rfcomm-incoming-session-refcount.txt --]
[-- Type: text/plain, Size: 1079 bytes --]

# This is a BitKeeper generated diff -Nru style patch.
#
# ChangeSet
#   2005/03/18 13:25:42+01:00 marcel@holtmann.org 
#   [Bluetooth] Fix session reference counting for RFCOMM
#   
#   When an incoming connection terminates, the signal DLC is never
#   closed and thus the underlaying L2CAP connection stays open. This
#   problem doesn't show up often, because most times the other side
#   takes care of terminating the signal DLC.
#   
#   Signed-off-by: Marcel Holtmann <marcel@holtmann.org>
# 
# net/bluetooth/rfcomm/core.c
#   2005/03/18 13:24:00+01:00 marcel@holtmann.org +4 -0
#   Fix session reference counting for RFCOMM
# 
diff -Nru a/net/bluetooth/rfcomm/core.c b/net/bluetooth/rfcomm/core.c
--- a/net/bluetooth/rfcomm/core.c	2005-03-18 13:30:07 +01:00
+++ b/net/bluetooth/rfcomm/core.c	2005-03-18 13:30:07 +01:00
@@ -389,6 +389,8 @@
 		rfcomm_dlc_unlock(d);
 
 		skb_queue_purge(&d->tx_queue);
+		rfcomm_session_put(s);
+
 		rfcomm_dlc_unlink(d);
 	}
 
@@ -597,6 +599,8 @@
 		*err = -ENOMEM;
 		goto failed;
 	}
+
+	rfcomm_session_hold(s);
 
 	s->initiator = 1;
 

[-- Attachment #3: Type: text/plain, Size: 347 bytes --]

-------------------------------------------------------------------------
Take Surveys. Earn Cash. Influence the Future of IT
Join SourceForge.net's Techsay panel and you'll get the chance to share your
opinions on IT & business topics through brief surveys - and earn cash
http://www.techsay.com/default.php?page=join.php&p=sourceforge&CID=DEVDEV

[-- Attachment #4: Type: text/plain, Size: 164 bytes --]

_______________________________________________
Bluez-devel mailing list
Bluez-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/bluez-devel