| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|
|
|
|
| |
There is no need for this since we use clang-format
Fixes #3004
|
|
|
|
|
|
| |
Fixes failures when running tests after building with thread-safety
w/ code stacks on.
Fixes #2990
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Remove unnecessary 'magic' field from cache structs
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
* Committing clang-format changes
---------
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
|
|
|
| |
Resolves #92
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Prevent buffer overrun in H5S_select_deserialize
The call to H5S_select_deserialize from H5S_decode doesn't have
the buffer size available to it, so to allow decoding there
I set it to assume a max size buffer for now.
Making the buffer size known in H5S_decode could be done by
modifying the external API's H5Sdecode, or splitting H5Sdecode
into two functions using a macro (similar to H5Sencode), with the
macro taking one argument and assuming a max buffer size.
* Conditional buffer check in H5S_select_deserialize
Moved and renamed a macro for only checking buffer overflow when
buffer size is known from H5Odtype.c to H5private.h,
so it can be used throughout the library.
Also silenced some build warnings about types.
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Move functions into more focused source code modules, along with a small #
of directly secondary effects. No actual changes to the contents of any
moved routines.
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
* Committing clang-format changes
---------
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Add failure value where it's missing from 1+ macros. Clean up
whitespace / continuation characters ('\'). Made hash-table macros generic
for use in both the package header and test header. Remove duplicated
copy & pasted macros (by hoisting difference into #ifdef'd macro).
Updated and re-flowed comments to read better.
Also clean up a few compiler warnings in production builds.
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
* Committing clang-format changes
* Remove unused variable warning in H5C.c (#2844)
* Remove trailing /* NDEBUG */ comment from #endif's
* Committing clang-format changes
---------
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
|
| |
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
* Reduce code duplication in macros
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
* "insert" -> "remove" in error string
Signed-off-by: Quincey Koziol <koziol@lbl.gov>
* Remove unused macro also
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
---------
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
Signed-off-by: Quincey Koziol <quincey@koziol.cc>
|
|
|
|
|
|
| |
Removes unnecessary headers and adds missing headers, ensuring
that headers can be included independently and in any order.
Fixes #2789
|
| |
|
|
|
| |
Fixes #688
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
|
| |
The datatype object header message decode function was updated to do
bounds checking on the decode buffer. This buffer may arrive with
no buffer size via H5Tdecode(), in which case the buffer size will
have been set to SIZE_MAX by the library.
This fix changes the string length calls to strlen when we don't
know the buffer size (and avoids a potential compiler bug
with icc 17).
|
|
|
|
| |
These cases can trip when processing malformed files and it's better
to invoke normal HDF5 error handling than crash a process.
|
|
|
|
|
| |
Initial implementation of selection I/O with type conversion. Allows
Parallel collective I/O with type conversion, as long as selection I/O
is enabled.
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
* Hardens v1 B-tree deserialize function
* Harden the H5HL deserialize functionality
|
|
|
|
| |
* Adds bounds checking to H5Odtype.c
* Minor tidy of dtypes test
|
|
|
|
|
| |
* Add buffer bounds checks
* Convert asserts to real error handling to better detect broken files
* General cleanup
|
| |
|
|
|
|
|
| |
Adds:
* Bounds checks on buffer access
* Better memory cleanup
|
|
|
|
| |
* Add bounds checking
* Add memory cleanup
|
| |
|
|
|
|
| |
* Use the H5_IS_BUFFER_OVERFLOW macro
* Attempt to close local heap on errors if left open
|
| |
|
| |
|
|
|
|
| |
* Check buffer bounds
* Clean up memory on errors
|
|
|
|
| |
Adds bounds checking on the buffer in the attribute info message's
decode function (H5O__ainfo_decode).
|
| |
|
|
|
|
|
|
|
|
|
|
|
| |
Malformed object header continuation messages can result in a too-small
buffer being passed to the decode function, which could lead to reading
past the end of the buffer. Additionally, errors in processing these
malformed messages can lead to allocated memory not being cleaned up.
This fix adds bounds checking and cleanup code to the object header
continuation message processing.
Fixes #2604
|
|
|
| |
* More cache macro parameter consistency
|
|
|
| |
* Pass failure value consistently to all macro errors
|
|
|
|
|
|
| |
* Move cache image stats macros to cache image source file
* Disable formatting on the macros
|
|
|
| |
* Make cache macro parameter names more consistent
|