diff options
| author | Albert Cheng <acheng@hdfgroup.org> | 2006-10-12 03:55:06 (GMT) |
|---|---|---|
| committer | Albert Cheng <acheng@hdfgroup.org> | 2006-10-12 03:55:06 (GMT) |
| commit | e065eee4a8932e6c09557bc539d6674138e9d7fe (patch) | |
| tree | 794fd744c2d3b5efbc7523005c20ee9461f1cdff /test/links.c | |
| parent | 9b24333b1944e5b5c37aeff0b94fabfbacb15b20 (diff) | |
| download | hdf5-e065eee4a8932e6c09557bc539d6674138e9d7fe.zip hdf5-e065eee4a8932e6c09557bc539d6674138e9d7fe.tar.gz hdf5-e065eee4a8932e6c09557bc539d6674138e9d7fe.tar.bz2 | |
[svn-r12751] Purpose:
Bug fix (related to 544)
Description:
h5_get_file_size() was coded to return 0 if failed but
file size can be 0. Changed the failure return value to -1
which is allowed by the returned type of off_t which is a signed
type.
Also changed the checking code of the stat call to just == 0
since that is how it is defined.
Test:
Could test it in heping only. Both Sol and Copper failed to
compiled due to error in the Direct IO VFD code.
Diffstat (limited to 'test/links.c')
| -rw-r--r-- | test/links.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/test/links.c b/test/links.c index 1a5f898..b1da4af 100644 --- a/test/links.c +++ b/test/links.c @@ -3767,7 +3767,7 @@ ud_hard_links(hid_t fapl) /* Get the size of the empty file for reference */ if(H5Fclose(fid) < 0) TEST_ERROR - if((empty_size = h5_get_file_size(filename))==0) TEST_ERROR + if((empty_size = h5_get_file_size(filename))<0) TEST_ERROR if((fid = H5Fcreate(filename, H5F_ACC_TRUNC, H5P_DEFAULT, fapl)) < 0) TEST_ERROR @@ -3933,7 +3933,7 @@ ud_link_reregister(hid_t fapl) /* Get the size of the empty file for reference */ if(H5Fclose(fid) < 0) TEST_ERROR - if((empty_size=h5_get_file_size(filename))==0) TEST_ERROR + if((empty_size=h5_get_file_size(filename))<0) TEST_ERROR if((fid=H5Fcreate(filename, H5F_ACC_TRUNC, H5P_DEFAULT, fapl)) < 0) TEST_ERROR |