BLEManagerImpl.cpp crash

[robertfarnum]

Problem

Android CHIPTool JNIEnv::NewByteArray crash. Stack trace attached (BLEManagerImpl.cpp.stack.txt)

• expected behavior
  App does not crash
• actual behavior
  App SIGABRTs; see stack trace.
• steps to reproduce
  Provision device on Wi-Fi
• system configuration
• screenshots, images, logs, etc.

Proposed Solution

None currently found

BLEManagerImpl.cpp.stack.txt

[mrjerryjohns]

The Android CHIPTool is no longer being actively maintained by the original maintainers.

[robertfarnum]

The issue is in an android platform library, not just in the CHIPTool. Is there a test suite that could or should enable reproducing the issue?

[robertfarnum]

@chrisdecenzo The actual cause of the above panic is due to an incorrect signature in the AndroidDeviceControlWrapper.cpp:

CHIP_ERROR err = JniReferences::GetInstance().FindMethod(env, mJavaObjectRef, "onCommissioningStatusUpdate",
"(JLJAVA/LANG/STRING;I)V", &onCommissioningStatusUpdateMethod);

should be:

CHIP_ERROR err = JniReferences::GetInstance().FindMethod(env, mJavaObjectRef, "onCommissioningStatusUpdate",
"(JLjava/lang/String;I)V", &onCommissioningStatusUpdateMethod);

[robertfarnum]

Related to #21493.

[andy31415]

I believe the wrong signature was fixed in #21980