summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorAlbert Cheng <acheng@hdfgroup.org>2003-08-27 15:27:56 (GMT)
committerAlbert Cheng <acheng@hdfgroup.org>2003-08-27 15:27:56 (GMT)
commit6d647e5406a743fa65f3f4245301b1731edce951 (patch)
treea121663dc853c4b8cc995f2eb7f5db2bb1bed3b6
parentb259ca6f86657e1842f4ade4b15802f34080c11c (diff)
downloadhdf5-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.c6
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;