diff options
author | Adrian Hunter <adrian.hunter@nokia.com> | 2009-06-22 10:31:09 -0400 |
---|---|---|
committer | Artem Bityutskiy <Artem.Bityutskiy@nokia.com> | 2009-07-05 11:45:15 -0400 |
commit | e3dc5a665d39112e98cfd5bbc7fda2963c00c12c (patch) | |
tree | b4f4c2ecec85bfb029f4ec31c56bb7477a86aba1 /fs | |
parent | 8e4a718ff38d8539938ec3421935904c27e00c39 (diff) |
UBIFS: fix integer overflow warning
Fix the following warning:
fs/ubifs/io.c: In function 'ubifs_wbuf_init':
fs/ubifs/io.c:860: warning: integer overflow in expression
And limit maximum hrtimer delta to ULONG_MAX because the
argument is 'unsigned long'.
Signed-off-by: Adrian Hunter <adrian.hunter@nokia.com>
Signed-off-by: Artem Bityutskiy <Artem.Bityutskiy@nokia.com>
Diffstat (limited to 'fs')
-rw-r--r-- | fs/ubifs/io.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/fs/ubifs/io.c b/fs/ubifs/io.c index bc5857199ec2..2d41ae1d6607 100644 --- a/fs/ubifs/io.c +++ b/fs/ubifs/io.c | |||
@@ -857,7 +857,9 @@ int ubifs_wbuf_init(struct ubifs_info *c, struct ubifs_wbuf *wbuf) | |||
857 | * and hard limits. | 857 | * and hard limits. |
858 | */ | 858 | */ |
859 | hardlimit = ktime_set(DEFAULT_WBUF_TIMEOUT_SECS, 0); | 859 | hardlimit = ktime_set(DEFAULT_WBUF_TIMEOUT_SECS, 0); |
860 | wbuf->delta = (DEFAULT_WBUF_TIMEOUT_SECS * NSEC_PER_SEC) * 2 / 10; | 860 | wbuf->delta = DEFAULT_WBUF_TIMEOUT_SECS * 1000000000ULL * 2 / 10; |
861 | if (wbuf->delta > ULONG_MAX) | ||
862 | wbuf->delta = ULONG_MAX; | ||
861 | wbuf->softlimit = ktime_sub_ns(hardlimit, wbuf->delta); | 863 | wbuf->softlimit = ktime_sub_ns(hardlimit, wbuf->delta); |
862 | hrtimer_set_expires_range_ns(&wbuf->timer, wbuf->softlimit, | 864 | hrtimer_set_expires_range_ns(&wbuf->timer, wbuf->softlimit, |
863 | wbuf->delta); | 865 | wbuf->delta); |