Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit e680491e authored by Marco Nelissen's avatar Marco Nelissen Committed by Android Git Automerger
Browse files

am 1efe46de: Merge "Fix id3 parser crash"

* commit '1efe46de':
  Fix id3 parser crash
parents cec272df 1efe46de
Loading
Loading
Loading
Loading
+9 −2
Original line number Diff line number Diff line
@@ -630,7 +630,10 @@ void ID3::Iterator::findFrame() {
                | (mParent.mData[mOffset + 4] << 8)
                | mParent.mData[mOffset + 5];

            mFrameSize += 6;
            if (mFrameSize == 0) {
                return;
            }
            mFrameSize += 6; // add tag id and size field

            if (mOffset + mFrameSize > mParent.mSize) {
                ALOGV("partial frame at offset %zu (size = %zu, bytes-remaining = %zu)",
@@ -671,7 +674,11 @@ void ID3::Iterator::findFrame() {
                baseSize = U32_AT(&mParent.mData[mOffset + 4]);
            }

            mFrameSize = 10 + baseSize;
            if (baseSize == 0) {
                return;
            }

            mFrameSize = 10 + baseSize; // add tag id, size field and flags

            if (mOffset + mFrameSize > mParent.mSize) {
                ALOGV("partial frame at offset %zu (size = %zu, bytes-remaining = %zu)",