diff options
| author | Trond Myklebust <trond.myklebust@primarydata.com> | 2014-02-12 09:36:59 -0500 |
|---|---|---|
| committer | Trond Myklebust <trond.myklebust@primarydata.com> | 2014-02-19 21:21:01 -0500 |
| commit | 2c64c57dfc4b7946f7abd8af653f55af581bc2c3 (patch) | |
| tree | ac1741fe6177634056d41ba1437d122fc778a403 /fs/nfs/pnfs.c | |
| parent | 311324ad1713666a6e803aecf0d4e1a136a5b34a (diff) | |
| download | op-kernel-dev-2c64c57dfc4b7946f7abd8af653f55af581bc2c3.zip op-kernel-dev-2c64c57dfc4b7946f7abd8af653f55af581bc2c3.tar.gz | |
NFSv4.1: Fix wraparound issues in pnfs_seqid_is_newer()
Subtraction of signed integers does not have well defined wraparound
semantics in the C99 standard. In order to be wraparound-safe, we
have to use unsigned subtraction, and then cast the result.
Signed-off-by: Trond Myklebust <trond.myklebust@primarydata.com>
Diffstat (limited to 'fs/nfs/pnfs.c')
| -rw-r--r-- | fs/nfs/pnfs.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/fs/nfs/pnfs.c b/fs/nfs/pnfs.c index 4755858..6e67ada 100644 --- a/fs/nfs/pnfs.c +++ b/fs/nfs/pnfs.c @@ -662,7 +662,7 @@ pnfs_destroy_all_layouts(struct nfs_client *clp) */ static bool pnfs_seqid_is_newer(u32 s1, u32 s2) { - return (s32)s1 - (s32)s2 > 0; + return (s32)(s1 - s2) > 0; } /* update lo->plh_stateid with new if is more recent */ |
