Re: [BUG] KASAN: slab-use-after-free Write in sk_skb_reason_drop

[Eulgyu Kim <eulgyukim@snu.ac.kr>]

>
> The fix could be this:
>
> diff --git a/drivers/net/tun.c b/drivers/net/tun.c
> index b183189f1853..6a0bbd4def76 100644
> --- a/drivers/net/tun.c
> +++ b/drivers/net/tun.c
> @@ -289,8 +289,12 @@ static void tun_napi_disable(struct tun_file *tfile)
>
>   static void tun_napi_del(struct tun_file *tfile)
>   {
> -       if (tfile->napi_enabled)
> -               netif_napi_del(&tfile->napi);
> +       if (!tfile->napi_enabled)
> +               return;
> +
> +       mutex_lock(&tfile->napi_mutex);
> +       netif_napi_del(&tfile->napi);
> +       mutex_unlock(&tfile->napi_mutex);
>   }
>
>   static bool tun_napi_frags_enabled(const struct tun_file *tfile)
> @@ -1783,6 +1787,12 @@ static ssize_t tun_get_user(struct tun_struct
> *tun, struct tun_file *tfile,
>
>                  if (frags) {
>                          mutex_lock(&tfile->napi_mutex);
> +                       if (unlikely(tfile->detached ||
> + rcu_access_pointer(tfile->tun) != tun)) {
> +                               err = -EBUSY;
> +  mutex_unlock(&tfile->napi_mutex);
> +                               goto out;
> +                       }
>                          skb = tun_napi_alloc_frags(tfile, copylen, from);
>                          /* tun_napi_alloc_frags() enforces a layout for
> the skb.
>                           * If zerocopy is enabled, then this layout will be
> @@ -1981,6 +1991,7 @@ static ssize_t tun_get_user(struct tun_struct
> *tun, struct tun_file *tfile,
>                  mutex_unlock(&tfile->napi_mutex);
>          }
>
> +out:
>          return err ?: total_len;
>   }
>

Hello,

We have tested the proposed patch, and our reproducer did not trigger any issue.

Thanks!

Best Regards,
Eulgyu Kim