diff options
author | Tim-Philipp Müller <tim@centricular.com> | 2021-03-03 01:08:25 +0000 |
---|---|---|
committer | Tim-Philipp Müller <tim@centricular.com> | 2021-03-15 13:01:04 +0000 |
commit | 93361467a333724ef144238c1e691b6b7a23cc09 (patch) | |
tree | b2add8a12d2a6743230353a4bb19e60249d413cc | |
parent | 53432a5efd46b1b0d4fb0c0cf7873786e92a93e7 (diff) |
tag: id3v2: fix frame size check and potential invalid reads1.14
Check the right variable when checking if there's
enough data left to read the frame size.
Closes https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/-/issues/876
Part-of: <https://gitlab.freedesktop.org/gstreamer/gst-plugins-base/-/merge_requests/1068>
-rw-r--r-- | gst-libs/gst/tag/id3v2frames.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/gst-libs/gst/tag/id3v2frames.c b/gst-libs/gst/tag/id3v2frames.c index 8e9f78254..f39659bf7 100644 --- a/gst-libs/gst/tag/id3v2frames.c +++ b/gst-libs/gst/tag/id3v2frames.c @@ -109,7 +109,7 @@ id3v2_parse_frame (ID3TagsWorking * work) if (work->frame_flags & (ID3V2_FRAME_FORMAT_COMPRESSION | ID3V2_FRAME_FORMAT_DATA_LENGTH_INDICATOR)) { - if (work->hdr.frame_data_size <= 4) + if (frame_data_size <= 4) return FALSE; if (ID3V2_VER_MAJOR (work->hdr.version) == 3) { work->parse_size = GST_READ_UINT32_BE (frame_data); |