Re: [PATCH net-next v5 05/10] bridge: cfm: Kernel space implementation of CFM. CCM frame TX added.

From: Jakub Kicinski
Date: Wed Oct 14 2020 - 19:00:03 EST


On Mon, 12 Oct 2020 14:04:23 +0000 Henrik Bjoernlund wrote:
> + skb = dev_alloc_skb(CFM_CCM_MAX_FRAME_LENGTH);
> + if (!skb)
> + return NULL;
> +
> + rcu_read_lock();
> + b_port = rcu_dereference(mep->b_port);
> + if (!b_port) {
> + rcu_read_unlock();
> + return NULL;
> + }

At a quick scan I noticed you appear to be leaking the skb here.
So let me point out some more nit picks.