From 011b3da8135e0f4786d079a5e2e3fc2ced94f8a4 Mon Sep 17 00:00:00 2001 From: Raymond Lu Date: Wed, 7 Nov 2007 16:52:42 -0500 Subject: [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. --- src/H5Osdspace.c | 8 ++------ 1 file 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))) -- cgit v0.12