summaryrefslogtreecommitdiffstats
path: root/c++
diff options
context:
space:
mode:
authorBinh-Minh Ribler <bmribler@hdfgroup.org>2009-10-04 05:27:26 (GMT)
committerBinh-Minh Ribler <bmribler@hdfgroup.org>2009-10-04 05:27:26 (GMT)
commita7cf6ea11ad76b6ed14944653259067b94afbfcf (patch)
treeb1f3e9aec47a9476afa53c19d4e156e17ac98b60 /c++
parentc26ed138a65b2840c057e437fa061a4749ba7ea1 (diff)
downloadhdf5-a7cf6ea11ad76b6ed14944653259067b94afbfcf.zip
hdf5-a7cf6ea11ad76b6ed14944653259067b94afbfcf.tar.gz
hdf5-a7cf6ea11ad76b6ed14944653259067b94afbfcf.tar.bz2
[svn-r17588] Purpose:
Fixed bugzilla #1061 Description: Added a check for NULL before accessing a pointer Also, removed a few warnings about returning values Platforms tested: Linux/32 2.6 (jam) FreeBSD/64 6.3 (liberty) SunOS 5.10 (linew)
Diffstat (limited to 'c++')
-rw-r--r--c++/src/H5Object.cpp5
-rw-r--r--c++/src/H5PredType.cpp1
2 files changed, 3 insertions, 3 deletions
diff --git a/c++/src/H5Object.cpp b/c++/src/H5Object.cpp
index 329b128..9245635 100644
--- a/c++/src/H5Object.cpp
+++ b/c++/src/H5Object.cpp
@@ -214,7 +214,7 @@ int H5Object::iterateAttrs( attr_operator_t user_op, unsigned *_idx, void *op_da
userData->object = this;
// call the C library routine H5Aiterate2 to iterate the attributes
- hsize_t idx = (hsize_t)*_idx;
+ hsize_t idx = _idx ? (hsize_t)*_idx : 0;
int ret_value = H5Aiterate2(getId(), H5_INDEX_NAME, H5_ITER_INC, &idx,
userAttrOpWrpr, (void *) userData);
@@ -223,7 +223,8 @@ int H5Object::iterateAttrs( attr_operator_t user_op, unsigned *_idx, void *op_da
if( ret_value >= 0 ) {
/* Pass back update index value to calling code */
- *_idx = (unsigned)idx;
+ if (_idx)
+ *_idx = (unsigned)idx;
return( ret_value );
}
diff --git a/c++/src/H5PredType.cpp b/c++/src/H5PredType.cpp
index 2ec3ba5..347f02f 100644
--- a/c++/src/H5PredType.cpp
+++ b/c++/src/H5PredType.cpp
@@ -289,7 +289,6 @@ void PredType::commit( H5Object& loc, const H5std_string& name )
bool PredType::committed()
{
throw DataTypeIException("PredType::committed", "Error: Attempting to check for commit status on a predefined datatype." );
- return (0);
}
#endif // DOXYGEN_SHOULD_SKIP_THIS