Hi Soenke, > On the reception of packets with a packet type that is not processed by > the virtbt driver, the memory of the allocated socket buffers is never > freed. Add a default case that frees these to avoid a memory leak. > > Signed-off-by: Soenke Huster <soenke.huster@xxxxxxxxx> > --- > drivers/bluetooth/virtio_bt.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/bluetooth/virtio_bt.c b/drivers/bluetooth/virtio_bt.c > index 57908ce4fae8..2de0f54ada73 100644 > --- a/drivers/bluetooth/virtio_bt.c > +++ b/drivers/bluetooth/virtio_bt.c > @@ -202,6 +202,8 @@ static void virtbt_rx_handle(struct virtio_bluetooth *vbt, struct sk_buff *skb) > hci_skb_pkt_type(skb) = pkt_type; > hci_recv_frame(vbt->hdev, skb); > break; > + default: > + kfree_skb(skb); I prefer that even default cases have a break;. Regards Marcel
