mirror of
https://github.com/HDFGroup/hdf5.git
synced 2026-09-25 04:09:44 +03:00
Refix of the attempts in PR-5209 (#5722)
This PR addresses the root cause of the issue by adding a sanity-check immediately after reading the file space page size from the file. The same fuzzer in GH-5376 was used to verify that the assert before the vulnerability had occurred and that an error indicating a corrupted file space page size replaced it.
This commit is contained in:
@@ -730,6 +730,8 @@ H5F__super_read(H5F_t *f, H5P_genplist_t *fa_plist, bool initial_read)
|
||||
if (!(flags & H5O_MSG_FLAG_WAS_UNKNOWN)) {
|
||||
H5O_fsinfo_t fsinfo; /* File space info message from superblock extension */
|
||||
|
||||
memset(&fsinfo, 0, sizeof(H5O_fsinfo_t));
|
||||
|
||||
/* f->shared->null_fsm_addr: Whether to drop free-space to the floor */
|
||||
/* The h5clear tool uses this property to tell the library
|
||||
* to drop free-space to the floor
|
||||
|
||||
@@ -184,6 +184,9 @@ H5O__fsinfo_decode(H5F_t *f, H5O_t H5_ATTR_UNUSED *open_oh, unsigned H5_ATTR_UNU
|
||||
if (H5_IS_BUFFER_OVERFLOW(p, H5F_sizeof_size(f), p_end))
|
||||
HGOTO_ERROR(H5E_OHDR, H5E_OVERFLOW, NULL, "ran off end of input buffer while decoding");
|
||||
H5F_DECODE_LENGTH(f, p, fsinfo->page_size); /* File space page size */
|
||||
/* Basic sanity check */
|
||||
if (fsinfo->page_size == 0 || fsinfo->page_size > H5F_FILE_SPACE_PAGE_SIZE_MAX)
|
||||
HGOTO_ERROR(H5E_OHDR, H5E_BADVALUE, NULL, "invalid page size in file space info");
|
||||
|
||||
if (H5_IS_BUFFER_OVERFLOW(p, 2, p_end))
|
||||
HGOTO_ERROR(H5E_OHDR, H5E_OVERFLOW, NULL, "ran off end of input buffer while decoding");
|
||||
|
||||
Reference in New Issue
Block a user