diff options
author | pfg <pfg@FreeBSD.org> | 2017-05-06 02:39:11 +0000 |
---|---|---|
committer | pfg <pfg@FreeBSD.org> | 2017-05-06 02:39:11 +0000 |
commit | 5d76adac38d9c07945c337413ccb2d812df59b4a (patch) | |
tree | a23ceb619b260fc84b10c8b67ffc7844651817c4 /sbin/fsck_ffs/setup.c | |
parent | 10f49ec8d59934616b47d4694057bdb95b38b0b4 (diff) | |
download | FreeBSD-src-5d76adac38d9c07945c337413ccb2d812df59b4a.zip FreeBSD-src-5d76adac38d9c07945c337413ccb2d812df59b4a.tar.gz |
MFC r317283:
fsck_ffs: Unsign some variables and make use of reallocarray(3).
Instead of casting listmax and numdirs to unsigned values just define
them as unsigned and avoid the casts. Use reallocarray(3).
While here, fs_ncg is already unsigned so the cast is unnecessary.
Reviewed by: mckusick
Diffstat (limited to 'sbin/fsck_ffs/setup.c')
-rw-r--r-- | sbin/fsck_ffs/setup.c | 9 |
1 files changed, 3 insertions, 6 deletions
diff --git a/sbin/fsck_ffs/setup.c b/sbin/fsck_ffs/setup.c index 0bcf06c..5c2b49f 100644 --- a/sbin/fsck_ffs/setup.c +++ b/sbin/fsck_ffs/setup.c @@ -268,8 +268,7 @@ setup(char *dev) (unsigned)bmapsize); goto badsb; } - inostathead = Calloc((unsigned)(sblock.fs_ncg), - sizeof(struct inostatlist)); + inostathead = Calloc(sblock.fs_ncg, sizeof(struct inostatlist)); if (inostathead == NULL) { printf("cannot alloc %u bytes for inostathead\n", (unsigned)(sizeof(struct inostatlist) * (sblock.fs_ncg))); @@ -279,10 +278,8 @@ setup(char *dev) dirhash = numdirs; inplast = 0; listmax = numdirs + 10; - inpsort = (struct inoinfo **)Calloc((unsigned)listmax, - sizeof(struct inoinfo *)); - inphead = (struct inoinfo **)Calloc((unsigned)numdirs, - sizeof(struct inoinfo *)); + inpsort = (struct inoinfo **)Calloc(listmax, sizeof(struct inoinfo *)); + inphead = (struct inoinfo **)Calloc(numdirs, sizeof(struct inoinfo *)); if (inpsort == NULL || inphead == NULL) { printf("cannot alloc %ju bytes for inphead\n", (uintmax_t)numdirs * sizeof(struct inoinfo *)); |