diff options
| author | Scot Breitenfeld <brtnfld@hdfgroup.org> | 2012-07-27 03:18:03 (GMT) |
|---|---|---|
| committer | Scot Breitenfeld <brtnfld@hdfgroup.org> | 2012-07-27 03:18:03 (GMT) |
| commit | 65100aa371af1ae14fedd282c928615a307a8e57 (patch) | |
| tree | 4bfac499838b2d855bc04017620ae517a39fe59e /hl/src/H5LT.c | |
| parent | f385b0d537b4fd5fec2e67b93e15c7c3a764c4a3 (diff) | |
| download | hdf5-65100aa371af1ae14fedd282c928615a307a8e57.zip hdf5-65100aa371af1ae14fedd282c928615a307a8e57.tar.gz hdf5-65100aa371af1ae14fedd282c928615a307a8e57.tar.bz2 | |
[svn-r22611] Fixed: HDFFV-8013 H5LTget_attribute_string appears to not close an object identifier
Tested: gnu and intel (jam)
Diffstat (limited to 'hl/src/H5LT.c')
| -rw-r--r-- | hl/src/H5LT.c | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/hl/src/H5LT.c b/hl/src/H5LT.c index 678642f..eed2721 100644 --- a/hl/src/H5LT.c +++ b/hl/src/H5LT.c @@ -3024,8 +3024,11 @@ herr_t H5LTget_attribute_string( hid_t loc_id, return -1; /* Get the attribute */ - if ( H5LT_get_attribute_disk( obj_id, attr_name, data ) < 0 ) - return -1; + if ( H5LT_get_attribute_disk( obj_id, attr_name, data ) < 0 ) + { + H5Oclose(obj_id); + return -1; + } /* Close the object */ if(H5Oclose(obj_id) < 0) @@ -3458,6 +3461,8 @@ static herr_t H5LT_get_attribute_mem(hid_t loc_id, return 0; out: + if(obj_id > 0) + H5Oclose(obj_id); if(attr_id > 0) H5Aclose(attr_id); return -1; |