commit 47c0e2355fb18f567471c1a76f1bbe2ebcb5a419
author Nipun Kwatra <nkwatra@google.com> Fri Aug 20 10:41:08 2010 -0700
committer Android (Google) Code Review <android-gerrit@google.com> Fri Aug 20 10:41:08 2010 -0700
tree d0c7d8ecd4573a351ff4a3c9e811e5d3482b338c
parent cfe76c8f6eff8aa770b25d18fc7aa89122e3d25d
parent 4a6b74563ac0fe752fbdfd15f91772473f8a4711

Merge "const correctness, validPixel test."

include/media/stagefright/YUVImage.h

diff --git a/include/media/stagefright/YUVImage.h b/include/media/stagefright/YUVImage.h
index afeac3f..4e98618 100644
--- a/include/media/stagefright/YUVImage.h
+++ b/include/media/stagefright/YUVImage.h
@@ -67,18 +67,22 @@
     // memory.
     static size_t bufferSize(YUVFormat yuvFormat, int32_t width, int32_t height);
 
-    int32_t width() {return mWidth;}
-    int32_t height() {return mHeight;}
+    int32_t width() const {return mWidth;}
+    int32_t height() const {return mHeight;}
+
+    // Returns true if pixel is the range [0, width-1] x [0, height-1]
+    // and false otherwise.
+    bool validPixel(int32_t x, int32_t y) const;
 
     // Get the pixel YUV value at pixel (x,y).
     // Note that the range of x is [0, width-1] and the range of y is [0, height-1].
-    // Returns true if get was succesful and false otherwise.
+    // Returns true if get was successful and false otherwise.
     bool getPixelValue(int32_t x, int32_t y,
             uint8_t *yPtr, uint8_t *uPtr, uint8_t *vPtr) const;
 
     // Set the pixel YUV value at pixel (x,y).
     // Note that the range of x is [0, width-1] and the range of y is [0, height-1].
-    // Returns true if set was succesful and false otherwise.
+    // Returns true if set was successful and false otherwise.
     bool setPixelValue(int32_t x, int32_t y,
             uint8_t yValue, uint8_t uValue, uint8_t vValue);
 

media/libstagefright/yuv/YUVImage.cpp

diff --git a/media/libstagefright/yuv/YUVImage.cpp b/media/libstagefright/yuv/YUVImage.cpp
index 73e3297..b712062 100644
--- a/media/libstagefright/yuv/YUVImage.cpp
+++ b/media/libstagefright/yuv/YUVImage.cpp
@@ -155,8 +155,15 @@
     return true;
 }
 
+bool YUVImage::validPixel(int32_t x, int32_t y) const {
+    return (x >= 0 && x < mWidth &&
+            y >= 0 && y < mHeight);
+}
+
 bool YUVImage::getPixelValue(int32_t x, int32_t y,
         uint8_t *yPtr, uint8_t *uPtr, uint8_t *vPtr) const {
+    CHECK(validPixel(x, y));
+
     uint8_t *yAddr;
     uint8_t *uAddr;
     uint8_t *vAddr;
@@ -171,6 +178,8 @@
 
 bool YUVImage::setPixelValue(int32_t x, int32_t y,
         uint8_t yValue, uint8_t uValue, uint8_t vValue) {
+    CHECK(validPixel(x, y));
+
     uint8_t *yAddr;
     uint8_t *uAddr;
     uint8_t *vAddr;