summaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorEgbert Eich <eich@suse.com>2022-11-11 05:41:53 (GMT)
committerGitHub <noreply@github.com>2022-11-11 05:41:53 (GMT)
commit34ec3bb7bc129f52bda4d82601f3bce65426459d (patch)
tree27aa5c9065a21991f3aa539c3cc2ac305163eb04 /src
parent1750b4b0af5158009aa2f861c65fb4bf8fc364de (diff)
downloadhdf5-34ec3bb7bc129f52bda4d82601f3bce65426459d.zip
hdf5-34ec3bb7bc129f52bda4d82601f3bce65426459d.tar.gz
hdf5-34ec3bb7bc129f52bda4d82601f3bce65426459d.tar.bz2
Make sure info block for external links has at least 3 bytes (#2234)
According to the specification, the information block for external links contains 1 byte of version/flag information and two 0 terminated strings for the object linked to and the full path. Although not very useful, the minimum string length for each (with terminating 0) would be one byte. Checking this will help to avoid SEGVs triggered by bogus files. This fixes CVE-2018-16438 / Bug #2233. Signed-off-by: Egbert Eich <eich@suse.com>
Diffstat (limited to 'src')
-rw-r--r--src/H5Olink.c2
1 files changed, 2 insertions, 0 deletions
diff --git a/src/H5Olink.c b/src/H5Olink.c
index 6146bbb..dabf87e 100644
--- a/src/H5Olink.c
+++ b/src/H5Olink.c
@@ -239,6 +239,8 @@ H5O__link_decode(H5F_t *f, H5O_t H5_ATTR_UNUSED *open_oh, unsigned H5_ATTR_UNUSE
/* A UD link. Get the user-supplied data */
UINT16DECODE(p, len)
+ if (lnk->type == H5L_TYPE_EXTERNAL && len < 3)
+ HGOTO_ERROR(H5E_OHDR, H5E_OVERFLOW, NULL, "external link information length < 3")
lnk->u.ud.size = len;
if (len > 0) {
/* Make sure that length doesn't exceed buffer size, which could