<div dir="ltr">gentle ping for review.</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Wed, Jun 16, 2021 at 5:59 PM Hirokazu Honda <<a href="mailto:hiroh@chromium.org">hiroh@chromium.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">CameraBuffer::plane() should be accessed in zero based index.<br>
Fix the wrong indexing in CameraBuffer::plane() in<br>
cros_camera_buffer.cpp.<br>
<br>
Signed-off-by: Hirokazu Honda <<a href="mailto:hiroh@chromium.org" target="_blank">hiroh@chromium.org</a>><br>
---<br>
 src/android/mm/cros_camera_buffer.cpp | 6 +++---<br>
 1 file changed, 3 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/src/android/mm/cros_camera_buffer.cpp b/src/android/mm/cros_camera_buffer.cpp<br>
index 59e26d1b..c4157606 100644<br>
--- a/src/android/mm/cros_camera_buffer.cpp<br>
+++ b/src/android/mm/cros_camera_buffer.cpp<br>
@@ -113,13 +113,13 @@ Span<uint8_t> CameraBuffer::Private::plane(unsigned int plane)<br>
                break;<br>
        default:<br>
                switch (plane) {<br>
-               case 1:<br>
+               case 0:<br>
                        addr = mem.ycbcr.y;<br>
                        break;<br>
-               case 2:<br>
+               case 1:<br>
                        addr = mem.ycbcr.cb;<br>
                        break;<br>
-               case 3:<br>
+               case 2:<br>
                        addr = <a href="http://mem.ycbcr.cr" rel="noreferrer" target="_blank">mem.ycbcr.cr</a>;<br>
                        break;<br>
                }<br>
-- <br>
2.32.0.272.g935e593368-goog<br>
<br>
</blockquote></div>