On Thu, Jun 13, 2024 at 04:09:15PM -0500, Bill O'Donnell wrote: > Cast max_tx_bytes to uint64_t to avoid overflowing expression in > calc_concurrency_logblocks(). > > Coverity-id: 1596603 > > Signed-off-by: Bill O'Donnell <bodonnel@xxxxxxxxxx> Looks ok, Reviewed-by: Darrick J. Wong <djwong@xxxxxxxxxx> (didn't hch rvb this earlier?) --D > --- > mkfs/xfs_mkfs.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/mkfs/xfs_mkfs.c b/mkfs/xfs_mkfs.c > index f4a9bf20..2f801dd4 100644 > --- a/mkfs/xfs_mkfs.c > +++ b/mkfs/xfs_mkfs.c > @@ -3678,7 +3678,7 @@ calc_concurrency_logblocks( > * without blocking for space. Increase the figure by 50% so that > * background threads can also run. > */ > - log_bytes = max_tx_bytes * 3 * cli->log_concurrency / 2; > + log_bytes = (uint64_t)max_tx_bytes * 3 * cli->log_concurrency / 2; > new_logblocks = min(XFS_MAX_LOG_BYTES >> cfg->blocklog, > log_bytes >> cfg->blocklog); > > -- > 2.45.2 > >
