Dan Carpenter <error27 at gmail.com> wrote: > It seems clear from the surrounding code that xpermits is allowed to be > NULL here. Interesting. The memcpy() won't oops due to this because if it is given a NULL pointer, it will also be given a zero count. I wonder if this means the if-statement your patch adds is actually unnecessary... David