diff options
| author | Quincey Koziol <koziol@hdfgroup.org> | 2006-08-21 14:51:16 (GMT) |
|---|---|---|
| committer | Quincey Koziol <koziol@hdfgroup.org> | 2006-08-21 14:51:16 (GMT) |
| commit | 143cfe24b447f7470c8e25852bfec5705f6b7f47 (patch) | |
| tree | e6e5de769a7248cc13a655cd63e157e7c76c917c /src/H5HFdblock.c | |
| parent | fc409b2cd0ef53cb6dea21a2f3b8398e9f4f804e (diff) | |
| download | hdf5-143cfe24b447f7470c8e25852bfec5705f6b7f47.zip hdf5-143cfe24b447f7470c8e25852bfec5705f6b7f47.tar.gz hdf5-143cfe24b447f7470c8e25852bfec5705f6b7f47.tar.bz2 | |
[svn-r12602] Description:
Correct bug in doubling table algorithm which was generating incorrect
row & column for offset larger than could be represented in 32 bits.
Also, beef up the error checking in direct block code a bit
Tested On:
FreeBSD/32 4.11 (sleipnir)
Linux/32 2.4 (heping)
Linux/64 2.4 (mir)
Solaris/64 2.9 (shanti)
Diffstat (limited to 'src/H5HFdblock.c')
| -rw-r--r-- | src/H5HFdblock.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/src/H5HFdblock.c b/src/H5HFdblock.c index 548e32f..6082d38 100644 --- a/src/H5HFdblock.c +++ b/src/H5HFdblock.c @@ -352,7 +352,7 @@ HDfprintf(stderr, "%s: request = %Zu\n", FUNC, request); if(request < hdr->man_dtable.cparam.start_block_size) min_dblock_size = hdr->man_dtable.cparam.start_block_size; else { - min_dblock_size = 1 << (1 + H5V_log2_gen((hsize_t)request)); + min_dblock_size = ((size_t)1) << (1 + H5V_log2_gen((hsize_t)request)); HDassert(min_dblock_size <= hdr->man_dtable.cparam.max_direct_size); } /* end else */ @@ -554,6 +554,7 @@ HDfprintf(stderr, "%s: iblock_addr = %a\n", FUNC, iblock_addr); /* Compute # of rows in child indirect block */ nrows = (H5V_log2_gen(hdr->man_dtable.row_block_size[row]) - hdr->man_dtable.first_row_bits) + 1; + HDassert(nrows < iblock->nrows); /* child must be smaller than parent */ /* Compute indirect block's entry */ entry = (row * hdr->man_dtable.cparam.width) + col; @@ -592,6 +593,7 @@ HDfprintf(stderr, "%s: iblock->block_off = %Hu\n", FUNC, iblock->block_off); /* Look up row & column in new indirect block for object */ if(H5HF_dtable_lookup(&hdr->man_dtable, (obj_off - iblock->block_off), &row, &col) < 0) HGOTO_ERROR(H5E_HEAP, H5E_CANTCOMPUTE, FAIL, "can't compute row & column of object") + HDassert(row < iblock->nrows); /* child must be smaller than parent */ #ifdef QAK HDfprintf(stderr, "%s: row = %u, col = %u\n", FUNC, row, col); #endif /* QAK */ |