commit c09c6078f0be5e0655d79af1acc3c1036beb712d
author sarvsaur <quic_sarvsaur@quicinc.com> Mon May 20 16:41:01 2024 +0530
committer Sarv Saurav <quic_sarvsaur@quicinc.com> Mon May 27 23:24:02 2024 -0700
tree 87126696b01ae05c5821662849972ce2f92f6551
parent 7d46e0a3bb9e24e315a083e4b646e8d307e640b5

FM: Buffer type changed & length check

-- JNI type is change for buffer and buffer length validation is added

Change-Id: Ib5c79f395807d70cd2df1fb76a16bfb16eafa311

jni/android_hardware_fm.cpp

diff --git a/jni/android_hardware_fm.cpp b/jni/android_hardware_fm.cpp
index 167e9f2..daad1fd 100644
--- a/jni/android_hardware_fm.cpp
+++ b/jni/android_hardware_fm.cpp
@@ -850,11 +850,15 @@
 
 /* native interface */
 static jint android_hardware_fmradio_FmReceiverJNI_getRawRdsNative
- (JNIEnv * env, jobject thiz, jint fd, jbooleanArray buff, jint count)
+ (JNIEnv * env, jobject thiz, jint fd, jbyteArray buff, jint count)
 {
+    ALOGD("->android_hardware_fmradio_FmReceiverJNI_getRawRdsNative");
 
+    if (env->GetArrayLength(buff) != count) {
+        ALOGE("%s: unexpected buff size", __func__);
+        return FM_JNI_FAILURE;
+    }
     return (read (fd, buff, count));
-
 }
 
 static jint android_hardware_fmradio_FmReceiverJNI_configureSpurTable