JNI- unpinned primitve数组错误

时间:2016-02-03 19:59:41

标签: c image-processing android-ndk java-native-interface native

我想在图像处理中实现反转效果。我在java端解码颜色通道,我将2D数组传递给C端,我反转(255值)然后我返回一个处理过的2D数组。

这是我的C代码:

   #include <jni.h>
#include<stddef.h>
#include <stdio.h>
#include<com_example_invert_MainActivity.h>
JNIEXPORT jobjectArray JNICALL Java_com_example_invert_MainActivity_inv
  (JNIEnv *env, jobject obj, jobjectArray arr, jint w, jint h)
{
    double a[w][h][3];
        int i,j,k;
        double x = 0;
        ///////////////////READING THE INPUT ARRAY////////////////////////


         jsize dim1 = (*env)->GetArrayLength(env, arr);

           for (i=0; i<dim1; i++){

                jdoubleArray *line1 =   (*env)->GetObjectArrayElement(env, arr, i);
                int dim2 =       (*env)->GetArrayLength(env, line1);
                jdouble *pos1 = (*env)->GetDoubleArrayElements(env, line1, 0);

                for (j=0; j<dim2; j++){
                    jdoubleArray *line2 =   (*env)->GetObjectArrayElement(env, line1, j);
                    int dim3 =       (*env)->GetArrayLength(env, line2);
                    jdouble *pos2 = (*env)->GetDoubleArrayElements(env, line2, 0);

                    for (k=0; k<dim3; k++){
                             a[i][j][k]= pos2[k];
                        }
                    (*env)->ReleaseDoubleArrayElements(env, arr, pos2, 0);
                    (*env)->ReleaseDoubleArrayElements(env, arr, line2, 0);
                  }

                (*env)->ReleaseDoubleArrayElements(env, arr, pos1, 0);

                (*env)->ReleaseDoubleArrayElements(env, arr, line1, 0);
           }



        /////////////////PROCESSING...///////////////////


        for( i = 0; i<w; i++){
            for( j = 0; j<h; j++){
                for( k = 0; k<3; k++){

                    a[i][j][k] = 255-a[i][j][k];
                }
            }
        }
        //////////////RETURNING THE ARRAY////////////////////////////
        jclass doubleArrayArrayClass = (*env)->FindClass(env,"[[D");
                jclass doubleArrayClass = (*env)->FindClass(env,"[D");

                jobjectArray ret  = (*env)->NewObjectArray(env,w, doubleArrayArrayClass, NULL);
        for( i = 0; i<w; i++){
            jobjectArray dim2 = (*env)->NewObjectArray(env, w, doubleArrayClass, NULL);
            for( j = 0; j<h; j++) {
                jdoubleArray dim1 = (*env)->NewDoubleArray(env,h);
                jdouble tmp[256];
                for( k = 0; k<3; k++){
                    tmp[k] = a[i][j][k];
                }
                (*env)->SetDoubleArrayRegion(env,dim1 , 0, 3, tmp);
                (*env)->SetObjectArrayElement(env, dim2, j, dim1);
                (*env)->DeleteLocalRef(env, dim1);
            }
            (*env)->SetObjectArrayElement(env,ret, i, dim2);
            (*env)->DeleteLocalRef(env,dim2);
        }
        return ret;
}

这是java代码:

public class MainActivity extends ActionBarActivity {
ImageView imageView2;
double[][][] imgArray;
int w,h;
@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);
    imageView2 = (ImageView) findViewById(R.id.imageView1);
    imageView2.setDrawingCacheEnabled(true);
    BitmapDrawable bitmapDrawable = (BitmapDrawable) imageView2.getDrawable();
    final Bitmap bitmap = bitmapDrawable.getBitmap();
    Button button = (Button) findViewById(R.id.button1);


    w = bitmap.getWidth();
    h = bitmap.getHeight();
    imgArray = new double[w][h][3];

   for(int i = 0 ; i<w; i++){
        for(int j =0; j<h; j++){
            imgArray[i][j][0] = Color.red(bitmap.getPixel(i, j));
            imgArray[i][j][1] = Color.green(bitmap.getPixel(i, j));
            imgArray[i][j][2] = Color.blue(bitmap.getPixel(i, j));
        }
    }

    button.setOnClickListener(new OnClickListener() {

        @Override
        public void onClick(View v) {

            imgArray = inv(imgArray, w, h);
            Bitmap newBitmap = Bitmap.createBitmap(w,h,bitmap.getConfig());
            for(int i = 0 ; i<w; i++){
                for(int j =0; j<h; j++){
                    newBitmap.setPixel(i, j, Color.rgb((int)(imgArray[i][j][0]), (int)(imgArray[i][j][1]), (int)(imgArray[i][j][2])));
                }
           }
            imageView2.setImageBitmap(newBitmap);
        }
    });

}
static{
    System.loadLibrary("inv");
}

// internal, private
public native double[][][] inv(double[][][] inputArr, int w, int h);


    ...
}

应用程序因logcat错误而崩溃:W/dalvikvm(5009): JNI: unpinPrimitiveArray(0x424eaea0) failed to find entry (valid=1)

3 个答案:

答案 0 :(得分:1)

ReleaseDoubleArrayElements(A, B, 0)只应与[{1}}对应调用。在您的代码中,B=GetDoubleArrayElements(env, A, 0)上的参数是错误的,( arr 而不是 line1 line2 等)。永远不应该调用它们来匹配Release…

作为优化通知,在Java和JNI中访问三维数组要比访问相同大小的一维数组要慢得多。因此,我强烈建议在Java GetObjectArrayElement()中创建并使用它。

PS 同样适用于输出数组。

PPS 使用imgArray = new double[w*h*3],你会引入一个额外的memcopy;更好的方法是,使用SetDoubleArrayRegion(),将值直接放入 cArray ,然后使用double* cArray = GetDoubleArrayElements(env, jArray, 0)将其发布到Java。 0 表示对 cArray 的更改将在Java端的 jArray 中显示。

答案 1 :(得分:0)

我没有JNI高手,所以这里有一个解决方法:用Java分配输出数组。它是这样的:

// the public API - a helper wrapper for the native `inv`:
public final int[][] invert( int[][] inputArr, int w, int h ) {
    int [][] outputArr = new int[h][w]; // allocate the array in Java
    inv( inputArr, outputArr, w, h );
    return outputArr;
}

// internal, private
private native void inv(int[][] inputArr, int[][] outputArr , int w, int h);

C方法签名将成为:

JNIEXPORT jvoid JNICALL Java_com_example_invert_MainActivity_inv(
  JNIEnv *env, jobject obj, 
  jobjectArray inputArr,  // your original 'arr' parameter
  jobjectArray outputArr, // new: the output to store the results in
  jint w, jint h)

对于C代码,您可以取消所有分配方法,只需使用outputArr将值放在(*env)->SetIntArrayRegion(env, jline, 0, h, a[i]);中。我会把它留给你,因为你可能比我更了解JNI; - )

答案 2 :(得分:0)

我不打算对JNI发表评论,但会提出另一种方法来解决你在Android上的问题。

您的应用程序是Android RenderScript的理想选择。图像逆算法可以实现为单行RenderScript内核,然后可以从Java启动以在GPU(或多核CPU)上并行执行。

这不仅会带来更简单的代码,还会带来更快的代码。

以下是RenderScript的链接。您实际上可以在那里找到一个图像反转示例。

http://developer.android.com/guide/topics/renderscript/compute.html