diff options
| author | Raymond Lu <songyulu@hdfgroup.org> | 2011-09-02 15:50:01 (GMT) |
|---|---|---|
| committer | Raymond Lu <songyulu@hdfgroup.org> | 2011-09-02 15:50:01 (GMT) |
| commit | 3cf78c8f18f849c629c7f7ac9b32601a8994d2b7 (patch) | |
| tree | 6390035c2d7f5265cd889b39f4995bc587c5e846 /release_docs | |
| parent | d47f033ad7c3b1201aa8b217d0ad008d0990eb20 (diff) | |
| download | hdf5-3cf78c8f18f849c629c7f7ac9b32601a8994d2b7.zip hdf5-3cf78c8f18f849c629c7f7ac9b32601a8994d2b7.tar.gz hdf5-3cf78c8f18f849c629c7f7ac9b32601a8994d2b7.tar.bz2 | |
[svn-r21356] Issue 7674 - CLANG compiler complained about the line (about 800):
tempint = (tempint >> packed_data_offset) & packed_data_mask;
The right shift may cause undefined behavior if PACKED_DATA_OFFSET is
32-bit or more. For every kind of native integers, I changed the code
to make it zero if PACKED_DATA_OFFSET is greater than or equal to the
size of integer.
Tested on jam and on MacOS with CLANG compiler.
Diffstat (limited to 'release_docs')
| -rw-r--r-- | release_docs/RELEASE.txt | 9 |
1 files changed, 9 insertions, 0 deletions
diff --git a/release_docs/RELEASE.txt b/release_docs/RELEASE.txt index 113a126..df09f48 100644 --- a/release_docs/RELEASE.txt +++ b/release_docs/RELEASE.txt @@ -114,6 +114,15 @@ Bug Fixes since HDF5-1.8.7 Library ------- + - CLANG compiler with the options -fcatch-undefined-behavior and -ftrapv + discovered 3 problems in tests and tools' library (Issue 7674): + 1. In dsets.c, left shifting an unsigned int for 32 bits or more + caused undefined behavior. + 2. In dt_arith.c, the INIT_INTEGER macro definition has an overflow + when the value is negative minimal and is being subtracted one. + 3. In tools/lib/h5tools_str.c, right shifting an int value for 32 bits + or more caused undefined behavior. + All the problems have been corrected. (SLU - 2011/9/2) - Corrected mismatched function name typo of h5pget_dxpl_mpio_c and h5pfill_value_defined_c. HDFFV-7641 (AKC 2011/08/22) - Corrected internal error in library where objects that use named |