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

Commit a62d5577 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "CCodec: fix wrapping logic" am: d4c493df am: 72c93f5f am:... 

Merge "CCodec: fix wrapping logic" am: d4c493df am: 72c93f5f am: 38c98dd6 am: 44a98a18 am: 15634e89

Original change: https://android-review.googlesource.com/c/platform/frameworks/av/+/2222023



Change-Id: Id66347b59f8fb4b6d81890651239d7a6c727200a
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents 73851ca9 15634e89

media/codec2/sfplugin/Codec2Buffer.cpp

+2 −3
Original line number Diff line number Diff line
@@ -533,7 +533,7 @@ public: 
                        * align(mHeight, 64) / plane.rowSampling;

            }



            if (minPtr == mView.data()[0] && (maxPtr - minPtr + 1) <= planeSize) {

            if (minPtr == mView.data()[0] && (maxPtr - minPtr) <= planeSize) {

                // FIXME: this is risky as reading/writing data out of bound results

                //        in an undefined behavior, but gralloc does assume a

                //        contiguous mapping
@@ -545,8 +545,7 @@ public: 
                    mediaImage->mPlane[i].mHorizSubsampling = plane.colSampling;

                    mediaImage->mPlane[i].mVertSubsampling = plane.rowSampling;

                }

                mWrapped = new ABuffer(const_cast<uint8_t *>(minPtr),

                                       maxPtr - minPtr + 1);

                mWrapped = new ABuffer(const_cast<uint8_t *>(minPtr), maxPtr - minPtr);

                ALOGV("Converter: wrapped (capacity=%zu)", mWrapped->capacity());

            }

        }