diff options
author | Remi Collet <remi@php.net> | 2014-06-03 11:05:00 +0200 |
---|---|---|
committer | Remi Collet <remi@php.net> | 2014-06-03 11:05:00 +0200 |
commit | 4fcb9a9d1b1063a65fbeb27395de4979c75bd962 (patch) | |
tree | b47942d06a381cf254f4392e0c675f04dace6df3 | |
parent | 38be99b739c6ad55b01fe304a083e7a1e36c05ee (diff) | |
download | php-git-4fcb9a9d1b1063a65fbeb27395de4979c75bd962.tar.gz |
Fix bug #67326 fileinfo: cdf_read_short_sector insufficient boundary check
Upstream fix https://github.com/file/file/commit/6d209c1c489457397a5763bca4b28e43aac90391.patch
Only revelant part applied
-rw-r--r-- | ext/fileinfo/libmagic/cdf.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ext/fileinfo/libmagic/cdf.c b/ext/fileinfo/libmagic/cdf.c index 4712e84942..16649f193f 100644 --- a/ext/fileinfo/libmagic/cdf.c +++ b/ext/fileinfo/libmagic/cdf.c @@ -365,10 +365,10 @@ cdf_read_short_sector(const cdf_stream_t *sst, void *buf, size_t offs, size_t ss = CDF_SHORT_SEC_SIZE(h); size_t pos = CDF_SHORT_SEC_POS(h, id); assert(ss == len); - if (pos > CDF_SEC_SIZE(h) * sst->sst_len) { + if (pos + len > CDF_SEC_SIZE(h) * sst->sst_len) { DPRINTF(("Out of bounds read %" SIZE_T_FORMAT "u > %" SIZE_T_FORMAT "u\n", - pos, CDF_SEC_SIZE(h) * sst->sst_len)); + pos + len, CDF_SEC_SIZE(h) * sst->sst_len)); return -1; } (void)memcpy(((char *)buf) + offs, |