diff options
| -rw-r--r-- | src/H5Distore.c | 6 | ||||
| -rw-r--r-- | src/H5Fistore.c | 6 | ||||
| -rw-r--r-- | 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++ ) |