diff options
| author | Raymond Lu <songyulu@hdfgroup.org> | 2007-11-07 21:52:42 (GMT) |
|---|---|---|
| committer | Raymond Lu <songyulu@hdfgroup.org> | 2007-11-07 21:52:42 (GMT) |
| commit | 011b3da8135e0f4786d079a5e2e3fc2ced94f8a4 (patch) | |
| tree | ea00674c961090dfcf39521cce38c0577af90f74 | |
| parent | 82f785ee0c4fa1c6ee42b3e28b2670d45a94202a (diff) | |
| download | hdf5-011b3da8135e0f4786d079a5e2e3fc2ced94f8a4.zip hdf5-011b3da8135e0f4786d079a5e2e3fc2ced94f8a4.tar.gz hdf5-011b3da8135e0f4786d079a5e2e3fc2ced94f8a4.tar.bz2 | |
[svn-r14243] Bug fix: Took out the verification of dimension size being zero in the function
H5O_sdspace_decode because this is an allowed situation.
Tested on kagiso only because it's a simple change.
| -rw-r--r-- | src/H5Osdspace.c | 8 |
1 files changed, 2 insertions, 6 deletions
diff --git a/src/H5Osdspace.c b/src/H5Osdspace.c index 79adf43..c97498a 100644 --- a/src/H5Osdspace.c +++ b/src/H5Osdspace.c @@ -163,13 +163,9 @@ H5O_sdspace_decode(H5F_t *f, hid_t UNUSED dxpl_id, unsigned UNUSED mesg_flags, if(sdim->rank > 0) { if(NULL == (sdim->size = (hsize_t *)H5FL_ARR_MALLOC(hsize_t, (size_t)sdim->rank))) HGOTO_ERROR(H5E_RESOURCE, H5E_NOSPACE, NULL, "memory allocation failed") - for(i = 0; i < sdim->rank; i++) { - H5F_DECODE_LENGTH(f, p, sdim->size[i]); - /* Rudimentary check for overflow of the dimension size */ - if(sdim->size[i] == 0) - HGOTO_ERROR(H5E_DATASPACE, H5E_BADSIZE, NULL, "invalid size detected") - } /* end for */ + for(i = 0; i < sdim->rank; i++) + H5F_DECODE_LENGTH(f, p, sdim->size[i]); if(flags & H5S_VALID_MAX) { if(NULL == (sdim->max = (hsize_t *)H5FL_ARR_MALLOC(hsize_t, (size_t)sdim->rank))) |