]> git.kernelconcepts.de Git - karo-tx-linux.git/commit
nfsd4: fix bad bounds checking
authorJ. Bruce Fields <bfields@redhat.com>
Tue, 1 Mar 2016 01:21:21 +0000 (20:21 -0500)
committerJ. Bruce Fields <bfields@redhat.com>
Tue, 1 Mar 2016 21:02:57 +0000 (13:02 -0800)
commit4aed9c46afb80164401143aa0fdcfe3798baa9d5
treefa0cae60a7a74660937630b60e4c255774777a85
parentb7052cd7bcf3c1478796e93e3dff2b44c9e82943
nfsd4: fix bad bounds checking

A number of spots in the xdr decoding follow a pattern like

n = be32_to_cpup(p++);
READ_BUF(n + 4);

where n is a u32.  The only bounds checking is done in READ_BUF itself,
but since it's checking (n + 4), it won't catch cases where n is very
large, (u32)(-4) or higher.  I'm not sure exactly what the consequences
are, but we've seen crashes soon after.

Instead, just break these up into two READ_BUF()s.

Cc: stable@vger.kernel.org
Signed-off-by: J. Bruce Fields <bfields@redhat.com>
fs/nfsd/nfs4xdr.c