linux-bluetooth.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH BlueZ] client/gatt: gatt_register_app: fix parsing of UUIDs
@ 2022-09-23 14:33 Christian Eggers
  2022-09-23 15:21 ` [BlueZ] " bluez.test.bot
  0 siblings, 1 reply; 2+ messages in thread
From: Christian Eggers @ 2022-09-23 14:33 UTC (permalink / raw)
  To: linux-bluetooth; +Cc: Christian Eggers

- argv[0] (shell command) must not be used as UUID argument
- fixed linked list iteration
---
 client/gatt.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/client/gatt.c b/client/gatt.c
index c3bdd4710897..e9f505172735 100644
--- a/client/gatt.c
+++ b/client/gatt.c
@@ -1212,7 +1212,7 @@ static gboolean get_uuids(const GDBusPropertyTable *property,
 	dbus_message_iter_open_container(iter, DBUS_TYPE_ARRAY,
 				DBUS_TYPE_STRING_AS_STRING, &entry);
 
-	for (uuid = uuids; uuid; uuid = g_list_next(uuid->next))
+	for (uuid = uuids; uuid; uuid = g_list_next(uuid))
 		dbus_message_iter_append_basic(&entry, DBUS_TYPE_STRING,
 							&uuid->data);
 
@@ -1238,7 +1238,7 @@ void gatt_register_app(DBusConnection *conn, GDBusProxy *proxy,
 		return bt_shell_noninteractive_quit(EXIT_FAILURE);
 	}
 
-	for (i = 0; i < argc; i++)
+	for (i = 1; i < argc; i++)
 		uuids = g_list_append(uuids, g_strdup(argv[i]));
 
 	if (uuids) {
-- 
2.35.3


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* RE: [BlueZ] client/gatt: gatt_register_app: fix parsing of UUIDs
  2022-09-23 14:33 [PATCH BlueZ] client/gatt: gatt_register_app: fix parsing of UUIDs Christian Eggers
@ 2022-09-23 15:21 ` bluez.test.bot
  0 siblings, 0 replies; 2+ messages in thread
From: bluez.test.bot @ 2022-09-23 15:21 UTC (permalink / raw)
  To: linux-bluetooth, ceggers

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

This is automated email and please do not reply to this email!

Dear submitter,

Thank you for submitting the patches to the linux bluetooth mailing list.
This is a CI test results with your patch series:
PW Link:https://patchwork.kernel.org/project/bluetooth/list/?series=679905

---Test result---

Test Summary:
CheckPatch                    PASS      0.69 seconds
GitLint                       PASS      0.50 seconds
Prep - Setup ELL              PASS      32.17 seconds
Build - Prep                  PASS      0.69 seconds
Build - Configure             PASS      10.18 seconds
Build - Make                  PASS      998.60 seconds
Make Check                    PASS      12.85 seconds
Make Check w/Valgrind         PASS      350.79 seconds
Make Distcheck                PASS      292.54 seconds
Build w/ext ELL - Configure   PASS      10.28 seconds
Build w/ext ELL - Make        PASS      102.02 seconds
Incremental Build w/ patches  PASS      0.00 seconds
Scan Build                    PASS      621.25 seconds



---
Regards,
Linux Bluetooth


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2022-09-23 15:22 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-09-23 14:33 [PATCH BlueZ] client/gatt: gatt_register_app: fix parsing of UUIDs Christian Eggers
2022-09-23 15:21 ` [BlueZ] " bluez.test.bot

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).