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 <sandeen@redhat.com>
Reviewed-by: Mark Tinguely <tinguely@sgi.com>
Signed-off-by: Mark Tinguely <tinguely@sgi.com>

diff --git a/repair/phase5.c b/repair/phase5.c
index 7d5cd49a1fcf857344d080edc45f8a4cd0b85052..1f71caca5952e8f8a21c165a9298af3de848c830 100644 (file)
--- 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"));