Use NULL instead of 0 for pointer arguments of CallStaticVoidMethod.
CallStaticVoidMethod is varargs function, and calling it with a literal 0 like CallStaticVoidMethod(..., 0) will treat the argument as a 4 byte int in both 32 and 64 bit processes. This is incorrect for pointer arguments where NULL should be used instead. Reviewed-by: Liao, Bruce <bruce.liao@intel.com> Signed-off-by: Yong Yao <yong.yao@intel.com> Change-Id: I9d700d3790a80dbee6826f64baf9ef5d81ca390f
This commit is contained in:
committed by
Narayan Kamath
parent
72d33849fe
commit
8bd6a203ca
@@ -102,7 +102,7 @@ void JNIMediaRecorderListener::notify(int msg, int ext1, int ext2)
|
||||
ALOGV("JNIMediaRecorderListener::notify");
|
||||
|
||||
JNIEnv *env = AndroidRuntime::getJNIEnv();
|
||||
env->CallStaticVoidMethod(mClass, fields.post_event, mObject, msg, ext1, ext2, 0);
|
||||
env->CallStaticVoidMethod(mClass, fields.post_event, mObject, msg, ext1, ext2, NULL);
|
||||
}
|
||||
|
||||
// ----------------------------------------------------------------------------
|
||||
|
||||
@@ -325,7 +325,7 @@ static void android_media_visualizer_effect_callback(int32_t event,
|
||||
fields.midPostNativeEvent,
|
||||
callbackInfo->visualizer_ref,
|
||||
NATIVE_EVENT_SERVER_DIED,
|
||||
0, 0, 0);
|
||||
0, 0, NULL);
|
||||
}
|
||||
}
|
||||
|
||||
|
||||
Reference in New Issue
Block a user