[nrf fromtree] Bluetooth: Host: Clean up buffer user data in case of error

`bt_l2cap_chan_send_cb()` overwrote the buffer user data for internal
use. In the case where sending fails, this would be visible for the
caller. If the caller relied on the buffer user data to be unchanged,
this could cause unexpected behavior.

L2CAP tx metadata was also not freed in the error case.

Signed-off-by: Herman Berget <[email protected]>
(cherry picked from commit e65fdd4)
Signed-off-by: Herman Berget <[email protected]>

Author: hermabe

subsys/bluetooth/host/l2cap.c

@@ -2952,6 +2952,7 @@ int bt_l2cap_chan_send_cb(struct bt_l2cap_chan *chan, struct net_buf *buf, bt_co
 {
 	struct bt_l2cap_le_chan *le_chan = BT_L2CAP_LE_CHAN(chan);
 	struct l2cap_tx_meta_data *data;
+	void *old_user_data = l2cap_tx_meta_data(buf);
 	int err;
 
 	if (!buf) {
@@ -3003,7 +3004,11 @@ int bt_l2cap_chan_send_cb(struct bt_l2cap_chan *chan, struct net_buf *buf, bt_co
 			net_buf_put(&le_chan->tx_queue, buf);
 			return l2cap_tx_meta_data(buf)->sent;
 		}
+
 		BT_ERR("failed to send message %d", err);
+
+		l2cap_tx_meta_data(buf) = old_user_data;
+		free_tx_meta_data(data);
 	}
 
 	return err;