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/h5test.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/h5test.c')
| -rw-r--r-- | test/h5test.c | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/test/h5test.c b/test/h5test.c index 5ca5d13..ad85a8f 100644 --- a/test/h5test.c +++ b/test/h5test.c @@ -827,13 +827,15 @@ h5_dump_info_object(MPI_Info info) * * Purpose: Get the current size of a file (in bytes) * - * Return: Success: Size of file in bytes (could be 0) - * Failure: 0 + * Return: Success: Size of file in bytes + * Failure: -1 * * Programmer: Quincey Koziol * Saturday, March 22, 2003 * * Modifications: + * Albert Cheng, Oct 11, 2006 + * Changed Failure return value to -1. * *------------------------------------------------------------------------- */ @@ -843,10 +845,10 @@ h5_get_file_size(const char *filename) h5_stat_t sb; /* Get the file's statistics */ - if (HDstat(filename, &sb)>=0) + if (HDstat(filename, &sb)==0) return((h5_stat_size_t)sb.st_size); - return(0); + return(-1); } /* end get_file_size() */ /* |