]> granicus.if.org Git - php/commitdiff
Fixed Bug #67413 fileinfo: cdf_read_property_info insufficient boundary chec
authorRemi Collet <remi@php.net>
Tue, 10 Jun 2014 12:33:37 +0000 (14:33 +0200)
committerRemi Collet <remi@php.net>
Tue, 10 Jun 2014 12:33:37 +0000 (14:33 +0200)
Upstream:
https://github.com/file/file/commit/93e063ee374b6a75729df9e7201fb511e47e259d

Adapted for C standard.

ext/fileinfo/libmagic/cdf.c

index ee467a6671ed93b91728f9820ed643efaab736a9..429f3b952f68d1ef7f2ebb4925ef5b16c54b7833 100644 (file)
@@ -812,7 +812,11 @@ cdf_read_property_info(const cdf_stream_t *sst, const cdf_header_t *h,
        if (cdf_check_stream_offset(sst, h, e, 0, __LINE__) == -1)
                goto out;
        for (i = 0; i < sh.sh_properties; i++) {
-               size_t ofs = CDF_GETUINT32(p, (i << 1) + 1);
+               size_t ofs, tail = (i << 1) + 1;
+               if (cdf_check_stream_offset(sst, h, p, tail * sizeof(uint32_t),
+                   __LINE__) == -1)
+                       goto out;
+               ofs = CDF_GETUINT32(p, tail);
                q = (const uint8_t *)(const void *)
                    ((const char *)(const void *)p + ofs
                    - 2 * sizeof(uint32_t));