Messages in this thread | | | From | David Howells <> | Subject | Re: [patch] afs: potential null dereference | Date | Mon, 22 Mar 2010 12:05:20 +0000 |
| |
Dan Carpenter <error27@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
| |