diff options
author | Albert Cheng <acheng@hdfgroup.org> | 2003-08-27 15:27:56 (GMT) |
---|---|---|
committer | Albert Cheng <acheng@hdfgroup.org> | 2003-08-27 15:27:56 (GMT) |
commit | 6d647e5406a743fa65f3f4245301b1731edce951 (patch) | |
tree | a121663dc853c4b8cc995f2eb7f5db2bb1bed3b6 | |
parent | b259ca6f86657e1842f4ade4b15802f34080c11c (diff) | |
download | hdf5-6d647e5406a743fa65f3f4245301b1731edce951.zip hdf5-6d647e5406a743fa65f3f4245301b1731edce951.tar.gz hdf5-6d647e5406a743fa65f3f4245301b1731edce951.tar.bz2 |
[svn-r7418] Purpose:
bug fix
Description:
h5diff would incur a segmentation fault in Alpha clusters when
the percentage option is used. This was due to memory aliagment
issue when the options variable is declared as a local variable
in main().
Moved the declaration to a global declaration.
Platforms tested:
"h5committested"
And tested at cluster QT of SNL.
Misc. update:
-rw-r--r-- | tools/h5diff/h5diff.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/tools/h5diff/h5diff.c b/tools/h5diff/h5diff.c index c80653e..801a762 100644 --- a/tools/h5diff/h5diff.c +++ b/tools/h5diff/h5diff.c @@ -51,6 +51,11 @@ typedef struct options_t int count; /* count value */ } options_t; +/* Due to alignment issue in Alpha clusters, options must be declared here + * not as a local variable in main(). + */ +options_t options = {0,0,0,0,0,0,0}; + /*------------------------------------------------------------------------- * prototypes *------------------------------------------------------------------------- @@ -181,7 +186,6 @@ int main(int argc, const char *argv[]) int nobjects1, nobjects2; info_t *info1=NULL; info_t *info2=NULL; - options_t options = {0,0,0,0,0,0,0}; const char *file1_name = NULL; const char *file2_name = NULL; const char *obj1_name = NULL; |