libpng主动翻转像素数据错误导致OpenGL ES滤镜效果变样

这实际并非libpng核心代码的问题，是我们项目会用的包装代码存在的行为。
我们的Android NDK项目中使用了libpng，在读取一个颜色查找表图像时，发现效果不对，比iOS实现存在颜色偏差。从png.h看，版本为libpng version 1.2.33 - October 31, 2008。在包装libpng的代码中，有这么一段特殊处理的代码：

unsigned char *rgba = new unsigned char[width * height * 4];  //each pixel(RGBA) has 4 bytes
png_bytep *row_pointers = png_get_rows(png_ptr, info_ptr);
//unlike store the pixel data from top-left corner, store them from bottom-left corner for OGLES Texture drawing...
int pos = (width * height * 4) - (4 * width);
for (int row = 0; row < height; row++) {
    for (int col = 0; col < (4 * width); col += 4) {
        rgba[pos++] = row_pointers[row][col + 0] * row_pointers[row][col + 3] / 255; // blue
        rgba[pos++] = row_pointers[row][col + 1] * row_pointers[row][col + 3] / 255; // green
        rgba[pos++] = row_pointers[row][col + 2] * row_pointers[row][col + 3] / 255; // red
        rgba[pos++] = row_pointers[row][col + 3]; // alpha
    }
    pos = (pos - (width * 4) * 2); //move the pointer back two rows
}

显然，这对图像作了垂直镜像（Flip Vertical），即第一行像素变成最后一行像素。而我提供的png图像是RGB24，上述代码将返回错误的结果。

Input #0, png_pipe, from 'lookup_crenshaw.png':
  Duration: N/A, bitrate: N/A
    Stream #0:0: Video: png, rgb24(pc), 128x128, 25 tbr, 25 tbn, 25 tbc

将其修改为读取RGB三通道值即可。

而使用UIImage（返回RGBA）或Android SDK的Bitmap类（返回ARGB）都是从左上角开始读取图像数据，故它们除了Alpha通道位置不同，RGB数据都相同。

值得注意的是，Bitmap的getPixels接口中有个stride参数，如果是读取整张图像，应该传递图像的宽度值，示例如下。

try {
    InputStream = new FileInputStream("/storage/emulated/0/lookup_dark.png");
    Bitmap bitmap = BitmapFactory.decodeStream(inputStream);
    int[] argb = new int[bitmap.getWidth() * bitmap.getHeight()];
    bitmap.getPixels(argb, 0, bitmap.getWidth(), 0, 0, bitmap.getWidth(), bitmap.getHeight());
    int[] rgba = new int[bitmap.getWidth() * bitmap.getHeight()];
    for (int row = 0; row < bitmap.getHeight(); ++row) {
        int index = row * bitmap.getWidth();
        for (int col = 0; col < bitmap.getWidth(); ++col) {
            int srcARGB = argb[col + index];
            rgba[col + index] = ((srcARGB & 0xFF000000) >> 24) + ((srcARGB & 0xFFFFFF) << 8);
        }
    }
} catch (Exception e) {
    e.printStackTrace();
}