summaryrefslogtreecommitdiffstats
path: root/tools/h5repack/h5repack_refs.c
diff options
context:
space:
mode:
authorPedro Vicente Nunes <pvn@hdfgroup.org>2007-04-03 20:44:52 (GMT)
committerPedro Vicente Nunes <pvn@hdfgroup.org>2007-04-03 20:44:52 (GMT)
commitea63d5fb19e8b260178e6f76e8ed0504ded148a3 (patch)
tree893a048c8ebb693e3d3fd1f2d8ece524629ac494 /tools/h5repack/h5repack_refs.c
parent0fb88ded47762bcafdd4317a3cd48f4c706a011c (diff)
downloadhdf5-ea63d5fb19e8b260178e6f76e8ed0504ded148a3.zip
hdf5-ea63d5fb19e8b260178e6f76e8ed0504ded148a3.tar.gz
hdf5-ea63d5fb19e8b260178e6f76e8ed0504ded148a3.tar.bz2
[svn-r13582]
Bug fix Substitute a malloc call for a calloc call while allocating a reference buffer Add a test for an empty reference case Tested linux
Diffstat (limited to 'tools/h5repack/h5repack_refs.c')
-rw-r--r--tools/h5repack/h5repack_refs.c21
1 files changed, 7 insertions, 14 deletions
diff --git a/tools/h5repack/h5repack_refs.c b/tools/h5repack/h5repack_refs.c
index f5e74c2..3b272c8 100644
--- a/tools/h5repack/h5repack_refs.c
+++ b/tools/h5repack/h5repack_refs.c
@@ -63,7 +63,6 @@ int do_copy_refobjs(hid_t fidin,
hsize_t nelmts; /* number of elements in dataset */
int rank; /* rank of dataset */
hsize_t dims[H5S_MAX_RANK]; /* dimensions of dataset */
- int next; /* external files */
unsigned int i, j;
int k;
@@ -149,13 +148,6 @@ int do_copy_refobjs(hid_t fidin,
if ((msize=H5Tget_size(mtype_id))==0)
goto error;
-
-/*-------------------------------------------------------------------------
- * check for external files
- *-------------------------------------------------------------------------
- */
- if ((next=H5Pget_external_count (dcpl_id))<0)
- goto error;
/*-------------------------------------------------------------------------
* check if the dataset creation property list has filters that
* are not registered in the current configuration
@@ -163,7 +155,7 @@ int do_copy_refobjs(hid_t fidin,
* 2) the internal filters might be turned off
*-------------------------------------------------------------------------
*/
- if (next==0 && h5tools_canreadf((NULL),dcpl_id)==1)
+ if (h5tools_canreadf((NULL),dcpl_id)==1)
{
/*-------------------------------------------------------------------------
* test for a valid output dataset
@@ -202,7 +194,7 @@ int do_copy_refobjs(hid_t fidin,
if ((obj_type = H5Rget_obj_type(dset_in,H5R_OBJECT,buf))<0)
goto error;
- refbuf=HDmalloc((unsigned)nelmts*msize);
+ refbuf=HDcalloc((unsigned)nelmts,msize);
if ( refbuf==NULL){
printf( "cannot allocate memory\n" );
goto error;
@@ -544,6 +536,7 @@ static int copy_refs_attr(hid_t loc_in,
if ((msize=H5Tget_size(mtype_id))==0)
goto error;
+
/*-------------------------------------------------------------------------
* object references are a special case
* we cannot just copy the buffers, but instead we recreate the reference
@@ -575,7 +568,7 @@ static int copy_refs_attr(hid_t loc_in,
if ((obj_type = H5Rget_obj_type(attr_id,H5R_OBJECT,buf))<0)
goto error;
- refbuf=HDmalloc((unsigned)nelmts*msize);
+ refbuf=HDcalloc((unsigned)nelmts,msize);
if ( refbuf==NULL){
printf( "cannot allocate memory\n" );
goto error;
@@ -607,14 +600,15 @@ static int copy_refs_attr(hid_t loc_in,
if ((attr_out=H5Acreate(loc_out,name,ftype_id,space_id,H5P_DEFAULT))<0)
goto error;
- if (nelmts) {
+ if (nelmts)
+ {
if(H5Awrite(attr_out,mtype_id,refbuf)<0)
goto error;
}
-
if (H5Aclose(attr_out)<0)
goto error;
+
if (refbuf)
free(refbuf);
if (buf)
@@ -824,4 +818,3 @@ static const char* MapIdToName(hid_t refobj_id,
}
-