From 80043b04c036548fe4c47a1fc5acb8a544fbcb8d Mon Sep 17 00:00:00 2001 From: Quincey Koziol Date: Fri, 26 Apr 2002 07:38:50 -0500 Subject: [svn-r5271] Purpose: Bug Fix. Description: When the fill value for a dataset is not set, the size is set to zero, causing problems with filling unused areas in previously defined chunks (i.e. when the dataset shrinks in size) Solution: Use the size of the elements in the dataset directly, instead of relying on the size of the fill value (which must be the same size). Platforms tested: FreeBSD 4.5 (sleipnir) --- src/H5Distore.c | 6 +++++- src/H5Fistore.c | 6 +++++- test/set_extent.c | 2 +- 3 files changed, 11 insertions(+), 3 deletions(-) diff --git a/src/H5Distore.c b/src/H5Distore.c index 1c2b137..f5bee8a 100644 --- a/src/H5Distore.c +++ b/src/H5Distore.c @@ -2940,7 +2940,11 @@ H5F_istore_initialize_by_extent(H5F_t *f, const H5O_layout_t *layout, HGOTO_ERROR(H5E_IO, H5E_WRITEERROR, FAIL, "unable to select hyperslab"); /* Fill the selection in the memory buffer */ - if(H5S_select_fill(fill.buf, fill.size, space_chunk, chunk) < 0) + /* Use the size of the elements in the chunk directly instead of */ + /* relying on the fill.size, which might be set to 0 if there is */ + /* no fill-value defined for the dataset -QAK */ + H5_CHECK_OVERFLOW(size[rank],hsize_t,size_t); + if(H5S_select_fill(fill.buf, (size_t)size[rank], space_chunk, chunk) < 0) HGOTO_ERROR(H5E_DATASET, H5E_CANTENCODE, FAIL, "filling selection failed"); if(H5F_istore_unlock(f, dxpl_id, layout, &pline, TRUE, diff --git a/src/H5Fistore.c b/src/H5Fistore.c index 1c2b137..f5bee8a 100644 --- a/src/H5Fistore.c +++ b/src/H5Fistore.c @@ -2940,7 +2940,11 @@ H5F_istore_initialize_by_extent(H5F_t *f, const H5O_layout_t *layout, HGOTO_ERROR(H5E_IO, H5E_WRITEERROR, FAIL, "unable to select hyperslab"); /* Fill the selection in the memory buffer */ - if(H5S_select_fill(fill.buf, fill.size, space_chunk, chunk) < 0) + /* Use the size of the elements in the chunk directly instead of */ + /* relying on the fill.size, which might be set to 0 if there is */ + /* no fill-value defined for the dataset -QAK */ + H5_CHECK_OVERFLOW(size[rank],hsize_t,size_t); + if(H5S_select_fill(fill.buf, (size_t)size[rank], space_chunk, chunk) < 0) HGOTO_ERROR(H5E_DATASET, H5E_CANTENCODE, FAIL, "filling selection failed"); if(H5F_istore_unlock(f, dxpl_id, layout, &pline, TRUE, diff --git a/test/set_extent.c b/test/set_extent.c index bae973f..c6fabd8 100644 --- a/test/set_extent.c +++ b/test/set_extent.c @@ -40,7 +40,7 @@ int main( void ) int buf1[ 70 ][ 70 ]; int buf2[ 90 ][ 90 ]; int i, j, n = 0; - int fillvalue = 0; /* Fill value for the dataset */ + int fillvalue = 1; /* Fill value for the dataset */ for( i = 0; i < 90; i++ ) -- cgit v0.12