Also introduces a const to improve readability
Signed-off-by: Mike Grant <mggr@xxxxxxxxx>
---
repair/phase6.c | 12 +++++++-----
1 file changed, 7 insertions(+), 5 deletions(-)
diff --git a/repair/phase6.c b/repair/phase6.c
index 105bce4..6520196 100644
--- a/repair/phase6.c
+++ b/repair/phase6.c
@@ -2267,6 +2267,7 @@ longform_dir2_entry_check(xfs_mount_t *mp,
dir_hash_tab_t *hashtab)
{
struct xfs_buf **bplist;
+ const size_t bpsz = sizeof(struct xfs_buf*);
xfs_dablk_t da_bno;
freetab_t *freetab;
int num_bps;
@@ -2293,10 +2294,10 @@ longform_dir2_entry_check(xfs_mount_t *mp,
freetab->ents[i].s = 0;
}
num_bps = freetab->naents;
- bplist = calloc(num_bps, sizeof(struct xfs_buf*));
+ bplist = calloc(num_bps, bpsz);
if (!bplist)
do_error(_("calloc failed in %s (%zu bytes)\n"),
- __func__, num_bps * sizeof(struct xfs_buf*));
+ __func__, num_bps * bpsz);
/* is this a block, leaf, or node directory? */
libxfs_dir2_isblock(NULL, ip, &isblock);
@@ -2327,11 +2328,12 @@ longform_dir2_entry_check(xfs_mount_t *mp,
if (db >= num_bps) {
/* more data blocks than expected */
num_bps = db + 1;
- bplist = realloc(bplist, num_bps * sizeof(struct
xfs_buf*));
+ bplist = realloc(bplist, num_bps * bpsz);
if (!bplist)
- do_error(_("realloc failed in %s (%zu
bytes)\n"),
+ do_error(
+ _("realloc failed in %s (%zu bytes)\n"),
__func__,
- num_bps * sizeof(struct xfs_buf*));
+ num_bps * bpsz);
}
if (isblock)
--
2.1.0
(apologies for the following junk forcibly appended by my company)
Please visit our new website at www.pml.ac.uk and follow us on Twitter
@PlymouthMarine
Winner of the Environment & Conservation category, the Charity Awards 2014.
Plymouth Marine Laboratory (PML) is a company limited by guarantee registered
in England & Wales, company number 4178503. Registered Charity No. 1091222.
Registered Office: Prospect Place, The Hoe, Plymouth PL1 3DH, UK.
This message is private and confidential. If you have received this message in
error, please notify the sender and remove it from your system. You are
reminded that e-mail communications are not secure and may contain viruses; PML
accepts no liability for any loss or damage which may be caused by viruses.
|