Bluetooth: BAP: Fix bad cast to void * instead of void for memcpy

Thalley · kartben · commit 2dbc10ab6b8e · 2025-04-04T18:16:54.000+02:00
The result of memcpy should be cast to (void) and not (void *).

Signed-off-by: Emil Gydesen &lt;emil.gydesen@nordicsemi.no&gt;
diff --git a/subsys/bluetooth/audio/bap_scan_delegator.c b/subsys/bluetooth/audio/bap_scan_delegator.c
@@ -912,8 +912,8 @@ static int scan_delegator_mod_src(struct bt_conn *conn,
 	}
 
 	/* Store requested_bis_sync after everything has been validated */
-	(void *)memcpy(internal_state->requested_bis_sync, requested_bis_sync,
-		       sizeof(requested_bis_sync));
+	(void)memcpy(internal_state->requested_bis_sync, requested_bis_sync,
+		     sizeof(requested_bis_sync));
 	atomic_clear_bit(internal_state->flags, BASS_RECV_STATE_INTERNAL_FLAG_NOTIFY_PEND);
 
 	/* Notify if changed */

Original file line number	Diff line number	Diff line change
`@@ -912,8 +912,8 @@ static int scan_delegator_mod_src(struct bt_conn *conn,`
`912`	`912`	`}`
`913`	`913`
`914`	`914`	`/* Store requested_bis_sync after everything has been validated */`
`915`		`- (void *)memcpy(internal_state->requested_bis_sync, requested_bis_sync,`
`916`		`- sizeof(requested_bis_sync));`
	`915`	`+ (void)memcpy(internal_state->requested_bis_sync, requested_bis_sync,`
	`916`	`+ sizeof(requested_bis_sync));`
`917`	`917`	`atomic_clear_bit(internal_state->flags, BASS_RECV_STATE_INTERNAL_FLAG_NOTIFY_PEND);`
`918`	`918`
`919`	`919`	`/* Notify if changed */`