[Bluez-devel] hci_send_req and EVT_CMD_STATUS

[Johan Hedberg <johan.hedberg@nokia.com>]

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

Hi Marcel,

I see that you changed hci_send_req so that it will never return
cmd_status events with non-zero status (instead the function fails with
EIO). If this was intentional (the fix I sent would have allowed getting
the cmd status event in all circumstanses), please apply the attached
patch since checking for rp.status after hci_send_req has successfully
returned is unecessary.

Johan

[-- Attachment #2: cmd-status.patch --]
[-- Type: text/plain, Size: 1111 bytes --]

Index: dbus.c
===================================================================
RCS file: /cvsroot/bluez/utils/hcid/dbus.c,v
retrieving revision 1.31
diff -u -r1.31 dbus.c
--- dbus.c	22 Oct 2005 13:03:41 -0000	1.31
+++ dbus.c	23 Oct 2005 19:55:39 -0000
@@ -1374,12 +1374,6 @@
 		goto failed;
 	}
 
-	if (rp.status) {
-		syslog(LOG_ERR, "Inquiry command failed with status 0x%02X", rp.status);
-		reply = bluez_new_failure_msg(msg, BLUEZ_ESYSTEM_OFFSET + EIO);
-		goto failed;
-	}
-
 	reply = dbus_message_new_method_return(msg);
 
 failed:
@@ -1496,12 +1490,6 @@
 		goto failed;
 	}
 
-	if (rp.status) {
-		syslog(LOG_ERR, "Remote name command failed with status 0x%02X", rp.status);
-		reply = bluez_new_failure_msg(msg, BLUEZ_ESYSTEM_OFFSET + EIO);
-		goto failed;
-	}
-
 	reply = dbus_message_new_method_return(msg);
 
 failed:
@@ -1655,12 +1643,6 @@
 		goto failed;
 	}
 
-	if (rp.status) {
-		syslog(LOG_ERR, "Authentication command failed with status 0x%02X", rp.status);
-		reply = bluez_new_failure_msg(msg, BLUEZ_ESYSTEM_OFFSET + EIO);
-		goto failed;
-	}
-
 failed:
 	if (sock >= 0)
 		close(sock);