Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 854cdcd6 authored by Yo Chiang's avatar Yo Chiang Committed by Yi-Yo Chiang
Browse files

Fix com.android.phone system_app_crash 

A Message object is reused and causing a system_app_crash.
Clone the Message object before passing it to getRadioConfigProxy()
so it's not reused.

Bug: 173970223
Test: Boot GSI S on CF Q with DSU, com.android.phone does not crash
Test: atest CuttlefishRilTests
Change-Id: Ifc8ae2e762ddfc25c29a4e0855cc0f6abc0ad414
Merged-In: Ifc8ae2e762ddfc25c29a4e0855cc0f6abc0ad414
parent 2a038be6

src/java/com/android/internal/telephony/RadioConfig.java

+1 −1
Original line number Diff line number Diff line
@@ -503,7 +503,7 @@ public class RadioConfig extends Handler { 
     * Gets the hal capabilities from the device.

     */

    public void getHalDeviceCapabilities(Message result) {

        IRadioConfig radioConfigProxy = getRadioConfigProxy(result);

        IRadioConfig radioConfigProxy = getRadioConfigProxy(Message.obtain(result));

        if (radioConfigProxy != null

                && mRadioConfigVersion.greaterOrEqual(RADIO_CONFIG_HAL_VERSION_1_3)) {

            android.hardware.radio.config.V1_3.IRadioConfig radioConfigProxy13 =