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

Commit 7875f687 authored by Chong Zhang's avatar Chong Zhang
Browse files

Fix stride number in C2AllocationGralloc::map

The code has a typo where "if" should be an "else if",
which causes the stride on mapper2 always set to 0.

bug: 143551816
test:
- play vp9 test clip in the bug for sw renderer on Pixel3
(modify xml to force sw decoding), video should look normal.
- run MediaMetadataRetrieverTest#testThumbnailVP9Hdr with output
file dump, output should look normal.

Change-Id: I1a587544fcf864d4a239252a8a2dc2569161e279
parent 8c5bb45d
Loading
Loading
Loading
Loading
+1 −1
Original line number Diff line number Diff line
@@ -633,7 +633,7 @@ c2_status_t C2AllocationGralloc::map(
            int32_t stride;
            if (mMapper2) {
                stride = int32_t(mInfo2.stride);
            } if (mMapper3) {
            } else if (mMapper3) {
                stride = int32_t(mInfo3.stride);
            } else {
                stride = int32_t(mInfo4.stride);