From 2c2e994d0a1b972d020d7633202bf2f4cfc9f91b Mon Sep 17 00:00:00 2001 From: Eric Sandeen Date: Sat, 26 Jan 2013 22:40:27 +0000 Subject: [PATCH] libxfs: fix setup_cursor array allocation setup_cursor() wants an array of xfs_agbno_t's, but it allocated a multiple of *pointers* to xfs_agbno_t's. xfs_agbno_t is 4 bytes, so this is harmless other than allocating twice as much memory as needed on a 64-bit machine. Signed-off-by: Eric Sandeen Reviewed-by: Mark Tinguely Signed-off-by: Mark Tinguely --- repair/phase5.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/repair/phase5.c b/repair/phase5.c index 7d5cd49a1..1f71caca5 100644 --- a/repair/phase5.c +++ b/repair/phase5.c @@ -206,7 +206,7 @@ setup_cursor(xfs_mount_t *mp, xfs_agnumber_t agno, bt_status_t *curs) ASSERT(big_extent_len > 0); - if ((curs->btree_blocks = malloc(sizeof(xfs_agblock_t *) + if ((curs->btree_blocks = malloc(sizeof(xfs_agblock_t) * big_extent_len)) == NULL) do_error(_("could not set up btree block array\n")); -- 2.47.2