From: Craig Gallek <kraigatg...@gmail.com>
Date: Tue, 22 Dec 2015 15:05:07 -0500

> +     for (i = 0; i < reuse->num_socks; i++) {
> +             if (reuse->socks[i] == sk) {
> +                     reuse->socks[i] = reuse->socks[reuse->num_socks - 1];
> +                     reuse->num_socks--;
> +                     if (reuse->num_socks == 0)
> +                             kfree_rcu(reuse, rcu);
> +                     break;
> +             }
> +     }

Don't you need to memmove() the entire rest of the array down one slot
when you hit the matching 'sk' in there?  I can't see how it can work
to only move one entry down.
--
To unsubscribe from this list: send the line "unsubscribe netdev" in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to