[PATCH BlueZ v1] client/player: Fix transport.send

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



From: Luiz Augusto von Dentz <luiz.von.dentz@xxxxxxxxx>

This fixes the usage of getpeername, introduced by 04153538aaf4
("client/player: Fix using unicast QoS for broadcast"), without
initializing optlen which causes the following problem:

Unable to send: Operation not permitted (1)
---
 client/player.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/client/player.c b/client/player.c
index df5cb892428a..7a7598c397af 100644
--- a/client/player.c
+++ b/client/player.c
@@ -5140,7 +5140,7 @@ static void cmd_send_transport(int argc, char *argv[])
 			err = transport_send(transport, fd, NULL);
 		} else {
 			struct sockaddr_iso addr;
-			socklen_t optlen;
+			socklen_t optlen = sizeof(addr);
 
 			err = getpeername(transport->sk, &addr, &optlen);
 			if (!err) {
@@ -5154,7 +5154,7 @@ static void cmd_send_transport(int argc, char *argv[])
 		}
 
 		if (err < 0) {
-			bt_shell_printf("Unable to send: %s (%d)",
+			bt_shell_printf("Unable to send: %s (%d)\n",
 						strerror(-err), -err);
 			close(fd);
 			return bt_shell_noninteractive_quit(EXIT_FAILURE);
-- 
2.45.2





[Index of Archives]     [Bluez Devel]     [Linux Wireless Networking]     [Linux Wireless Personal Area Networking]     [Linux ATH6KL]     [Linux USB Devel]     [Linux Media Drivers]     [Linux Audio Users]     [Linux Kernel]     [Linux SCSI]     [Big List of Linux Books]

  Powered by Linux