xfs: Use kvcalloc() instead of kvzalloc()
authorGustavo A. R. Silva <gustavoars@kernel.org>
Mon, 11 Oct 2021 23:02:22 +0000 (16:02 -0700)
committerDarrick J. Wong <djwong@kernel.org>
Mon, 11 Oct 2021 23:13:29 +0000 (16:13 -0700)
Use 2-factor argument multiplication form kvcalloc() instead of
kvzalloc().

Link: https://github.com/KSPP/linux/issues/162
Signed-off-by: Gustavo A. R. Silva <gustavoars@kernel.org>
Reviewed-by: Darrick J. Wong <djwong@kernel.org>
Signed-off-by: Darrick J. Wong <djwong@kernel.org>
fs/xfs/xfs_ioctl.c

index 0c795dc..174cd89 100644 (file)
@@ -1547,7 +1547,7 @@ xfs_ioc_getbmap(
        if (bmx.bmv_count > ULONG_MAX / recsize)
                return -ENOMEM;
 
-       buf = kvzalloc(bmx.bmv_count * sizeof(*buf), GFP_KERNEL);
+       buf = kvcalloc(bmx.bmv_count, sizeof(*buf), GFP_KERNEL);
        if (!buf)
                return -ENOMEM;
 
@@ -1601,11 +1601,11 @@ xfs_ioc_getfsmap(
         */
        count = min_t(unsigned int, head.fmh_count,
                        131072 / sizeof(struct fsmap));
-       recs = kvzalloc(count * sizeof(struct fsmap), GFP_KERNEL);
+       recs = kvcalloc(count, sizeof(struct fsmap), GFP_KERNEL);
        if (!recs) {
                count = min_t(unsigned int, head.fmh_count,
                                PAGE_SIZE / sizeof(struct fsmap));
-               recs = kvzalloc(count * sizeof(struct fsmap), GFP_KERNEL);
+               recs = kvcalloc(count, sizeof(struct fsmap), GFP_KERNEL);
                if (!recs)
                        return -ENOMEM;
        }