Message ID | 159107190741.313760.11195530788081068638.stgit@magnolia (mailing list archive) |
---|---|
State | Accepted |
Headers | show |
Series | xfs_repair: catch things that xfs_check misses | expand |
On Mon, Jun 01, 2020 at 09:25:07PM -0700, Darrick J. Wong wrote: > From: Darrick J. Wong <darrick.wong@oracle.com> > > Fix compiler warnings about unsigned int comparisons by replacing them > with an explicit check for the one possible invalid value (-1U). > id_from_string sets exitcode to nonzero when it sees this value, so the > call sites don't have to do that. > > Coverity-id: 1463855, 1463856, 1463857 > Fixes: 67a73d6139d0 ("xfs_quota: refactor code to generate id from name") > Fixes: 36dc471cc9bb ("xfs_quota: allow individual timer extension") > Signed-off-by: Darrick J. Wong <darrick.wong@oracle.com> Looks good, Reviewed-by: Christoph Hellwig <hch@lst.de>
diff --git a/quota/edit.c b/quota/edit.c index cf9508bf..01d358f7 100644 --- a/quota/edit.c +++ b/quota/edit.c @@ -307,11 +307,11 @@ limit_f( id = id_from_string(name, type); - if (id >= 0) - set_limits(id, type, mask, fs_path->fs_name, - &bsoft, &bhard, &isoft, &ihard, &rtbsoft, &rtbhard); - else - exitcode = -1; + if (id == -1) + return 0; + + set_limits(id, type, mask, fs_path->fs_name, + &bsoft, &bhard, &isoft, &ihard, &rtbsoft, &rtbhard); return 0; } @@ -545,9 +545,10 @@ timer_f( if (name) id = id_from_string(name, type); - if (id >= 0) - set_timer(id, type, mask, fs_path->fs_name, value); + if (id == -1) + return 0; + set_timer(id, type, mask, fs_path->fs_name, value); return 0; } @@ -642,11 +643,10 @@ warn_f( } id = id_from_string(name, type); - if (id >= 0) - set_warnings(id, type, mask, fs_path->fs_name, value); - else - exitcode = -1; + if (id == -1) + return 0; + set_warnings(id, type, mask, fs_path->fs_name, value); return 0; }