diff mbox series

[BlueZ,v2,2/2] bap: Use util_iov_new when setting bcode

Message ID 20240619095119.167652-3-vlad.pruteanu@nxp.com (mailing list archive)
State Accepted
Commit 5ec05848a9d441603a1d34ca184e2c470f0bfd62
Headers show
Series bap: Allocate memory for broadcast code | expand

Checks

Context Check Description
tedd_an/pre-ci_am success Success
tedd_an/CheckPatch success CheckPatch PASS
tedd_an/GitLint success Gitlint PASS
tedd_an/IncrementalBuild success Incremental Build PASS

Commit Message

Vlad Pruteanu June 19, 2024, 9:51 a.m. UTC 
Currently the broadcast code is set without memory being allocated
for it resulting in segmentation fault. This fixes the issue by using
the newly created util_iov_new that allocates the memory before
setting it.
---
 profiles/audio/bap.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)
diff mbox series

Patch

diff --git a/profiles/audio/bap.c b/profiles/audio/bap.c
index 53e7b3e34..e82a25382 100644
--- a/profiles/audio/bap.c
+++ b/profiles/audio/bap.c
@@ -1042,9 +1042,8 @@  static void create_stream_for_bis(struct bap_data *bap_data,
 	setup->qos.bcast.framing = qos->bcast.framing;
 	setup->qos.bcast.encryption = qos->bcast.encryption;
 	if (setup->qos.bcast.encryption)
-		util_iov_append(setup->qos.bcast.bcode,
-				qos->bcast.bcode,
-				sizeof(qos->bcast.bcode));
+		setup->qos.bcast.bcode = util_iov_new(qos->bcast.bcode,
+						sizeof(qos->bcast.bcode));
 	setup->qos.bcast.options = qos->bcast.options;
 	setup->qos.bcast.skip = qos->bcast.skip;
 	setup->qos.bcast.sync_timeout = qos->bcast.sync_timeout;