Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

The Huawei device failed crashed when displaying media notification #903

Closed
1 task
MrWhhh opened this issue Dec 17, 2023 · 11 comments
Closed
1 task

The Huawei device failed crashed when displaying media notification #903

MrWhhh opened this issue Dec 17, 2023 · 11 comments

Comments

@MrWhhh
Copy link

MrWhhh commented Dec 17, 2023

Version

Media3 1.2.0

More version details

android.app.RemoteServiceException: Bad notification(tag=null, id=1001) posted from package com.xxx.xxx, crashing app(uid=10289, pid=26950): long overflow
	at android.app.ActivityThread.throwRemoteServiceException(ActivityThread.java:2591)
	at android.app.ActivityThread.access$4000(ActivityThread.java:311)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:3001)
	at android.os.Handler.dispatchMessage(Handler.java:117)
	at android.os.Looper.loopOnce(Looper.java:205)
	at android.os.Looper.loop(Looper.java:293)
	at android.app.ActivityThread.loopProcess(ActivityThread.java:9961)
	at android.app.ActivityThread.main(ActivityThread.java:9950)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:586)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1240)

Devices that reproduce the issue

Huawei

Devices that do not reproduce the issue

No response

Reproducible in the demo app?

Yes

Reproduction steps

lay

Expected result

Media playback success

Actual result

android.app.RemoteServiceException: Bad notification(tag=null, id=1001) posted from package com.xxx.xxx, crashing app(uid=10289, pid=26950): long overflow
	at android.app.ActivityThread.throwRemoteServiceException(ActivityThread.java:2591)
	at android.app.ActivityThread.access$4000(ActivityThread.java:311)
	at android.app.ActivityThread$H.handleMessage(ActivityThread.java:3001)
	at android.os.Handler.dispatchMessage(Handler.java:117)
	at android.os.Looper.loopOnce(Looper.java:205)
	at android.os.Looper.loop(Looper.java:293)
	at android.app.ActivityThread.loopProcess(ActivityThread.java:9961)
	at android.app.ActivityThread.main(ActivityThread.java:9950)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:586)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1240)
Back traces ends.

Media

all meida

Bug Report

@oceanjules
Copy link
Contributor

Hello @MrWhhh,

The information you provided is not enough to make it actionable for us. It would be useful to see what device you used and clear steps to reproduce the issue.

It would also be useful to see the bug report. If you're unable to share bug reports or test content publicly, please send them to android-media-github@google.com using a subject "Issue #903". Please also update this issue to indicate you’ve done this.

@MrWhhh
Copy link
Author

MrWhhh commented Dec 20, 2023

@oceanjules The device I use is Huawei mate60pro, the system is Hongmeng4, and the music playing service is inherited from MediaLibraryService(). When I call exoplayer to play, other devices can normally send notifications and play, but there is a crash on Huawei mate60pro. Error log as above

@WXFY
Copy link

WXFY commented Dec 20, 2023

I have also encountered this error. It appears on the Huawei P40, and HarmonyOS 4 must appear.

@wuhenge
Copy link

wuhenge commented Dec 20, 2023

When I turn off all notification permissions of the APP, there will be no crash, so this error should be a crash caused by the notification.

@marcbaechinger
Copy link
Contributor

@MrWhhh Please provide repro steps with the demo app if you want us look into this. If we don't know how to repro we probably work on other issues first that are more actionable. I'm not sure when and whether we get such a Huawei device in our hands, but if we do and we don't have the repro steps with the demo app, it would be unfortunate.

Regarding this happening on Huawei only: while being not a politician nor a prophet - I'd also try to talk to Huawei directly. Personally, I have taken notice of Hongmeng4 in this issue for the first time. You know that story behind it probably better then I do.

@MrWhhh
Copy link
Author

MrWhhh commented Dec 21, 2023

@marcbaechinger

browserFuture = MediaBrowser.Builder(
                this,
                SessionToken(
                    this,
                    ComponentName(this, MusicService::class.java)
                )
            )
                .buildAsync()

serivce inherits from MediaLibraryService() in the same way as Demo

class MusicService : MediaLibraryService() {

    lateinit var player: ExoPlayer
    lateinit var mediaLibrarySession: MediaLibrarySession

    private val serviceScope = CoroutineScope(SupervisorJob() + Dispatchers.Main.immediate)

    override fun onCreate() {
        super.onCreate()

        player = ExoPlayer.Builder(this)
            .setAudioAttributes(AudioAttributes.DEFAULT, !DataStoreEnum.RESTS_APP_PLAY.read(false))
            .setHandleAudioBecomingNoisy(true)
            .build()
        player.addListener(MyPlayer())

        val forwardingPlayer = @OptIn(UnstableApi::class) object : ForwardingPlayer(player){}
        mediaLibrarySession =
            MediaLibrarySession
                .Builder(this, forwardingPlayer, MySessionCallback())
                .build()
 }
 
 
    override fun onGetSession(controllerInfo: MediaSession.ControllerInfo): MediaLibrarySession? =
        mediaLibrarySession

    override fun onDestroy() {
        serviceScope.cancel()
        player.release()
        mediaLibrarySession.release()
        super.onDestroy()
    }

}

@marcbaechinger
Copy link
Contributor

@MrWhhh Please provide repro steps with the demo app if you want us look into this.

@WXFY
Copy link

WXFY commented Dec 27, 2023

I myself pull the source code in the HarmonyOS 4 phone debugging found that the following code may cause long overflow exception.

long playbackStartTimeMs = getPlaybackStartTimeEpochMs(player);
boolean displayElapsedTimeWithChronometer = playbackStartTimeMs != C.TIME_UNSET;
builder
.setWhen(playbackStartTimeMs)
.setShowWhen(displayElapsedTimeWithChronometer)
.setUsesChronometer(displayElapsedTimeWithChronometer);

getPlaybackStartTimeEpochMs method returns the value Long.MIN_VALUE + 1 and then sets the setWhen method to have a long overflow exception. I tried to divide the value by 2 and it was normal.

@marcbaechinger
Copy link
Contributor

@WXFY Thanks for investigation and reporting.

setWhen is documented to be the time at which the event occured. So setting C.TIME_UNSET as we do isn't correct and we can avoid this with a trivial change I think:

builder
    .setWhen(displayElapsedTimeWithChronometer ? playbackStartTimeMs : 0L)
    .setShowWhen(displayElapsedTimeWithChronometer)
    .setUsesChronometer(displayElapsedTimeWithChronometer);

I think a time at which the event occurred shouldn't be a negative value. I sent the change above in DefaultMediaNotificationProvider for review as it seems we violate what is documented. There will be an entry in this issue when the change lands here on GitHub.

It looks like this isn't a problem on other devices, I have tested the code below with API 28 and API 34 on Pixels and I can not repro the crash:

long playbackStartTimeMs = getPlaybackStartTimeEpochMs(player);
boolean displayElapsedTimeWithChronometer = playbackStartTimeMs != C.TIME_UNSET;
builder
    .setWhen(C.TIME_UNSET)
    .setShowWhen(displayElapsedTimeWithChronometer)
    .setUsesChronometer(displayElapsedTimeWithChronometer);

If you wanna do me a favor and you have the source already available, you can test the proposed fix above and test with the HarmonyOS 4 device to verify this actually helps.

copybara-service bot pushed a commit that referenced this issue Dec 29, 2023
When the 'when' timer of the notification is disabled
`DefaultMediaNotificationProvider` may set `C.TIME_UNSET`
as the time. Users reported problems on some devices with
this and the docs ask for an event time that probably
shouldn't be a negative number.

This change sets `0L` instead of `C.TIME_UNSET` when the
timer is disabled.

Issue: #903

#minor-release

PiperOrigin-RevId: 594451074
@WXFY
Copy link

WXFY commented Jan 2, 2024

@WXFY Thanks for investigation and reporting.

setWhen is documented to be the time at which the event occured. So setting C.TIME_UNSET as we do isn't correct and we can avoid this with a trivial change I think:

builder
    .setWhen(displayElapsedTimeWithChronometer ? playbackStartTimeMs : 0L)
    .setShowWhen(displayElapsedTimeWithChronometer)
    .setUsesChronometer(displayElapsedTimeWithChronometer);

I think a time at which the event occurred shouldn't be a negative value. I sent the change above in DefaultMediaNotificationProvider for review as it seems we violate what is documented. There will be an entry in this issue when the change lands here on GitHub.

It looks like this isn't a problem on other devices, I have tested the code below with API 28 and API 34 on Pixels and I can not repro the crash:

long playbackStartTimeMs = getPlaybackStartTimeEpochMs(player);
boolean displayElapsedTimeWithChronometer = playbackStartTimeMs != C.TIME_UNSET;
builder
    .setWhen(C.TIME_UNSET)
    .setShowWhen(displayElapsedTimeWithChronometer)
    .setUsesChronometer(displayElapsedTimeWithChronometer);

If you wanna do me a favor and you have the source already available, you can test the proposed fix above and test with the HarmonyOS 4 device to verify this actually helps.

I tested the Huawei P40 HarmonyOS 4 with your modified code and it works normally.

@marcbaechinger marcbaechinger changed the title The Huawei device failed to play the device The Huawei device failed crashed when displaying media notification Jan 3, 2024
@marcbaechinger
Copy link
Contributor

Thanks for confirming. I'm closing this issue as the fix seems to work.

microkatz pushed a commit that referenced this issue Jan 11, 2024
When the 'when' timer of the notification is disabled
`DefaultMediaNotificationProvider` may set `C.TIME_UNSET`
as the time. Users reported problems on some devices with
this and the docs ask for an event time that probably
shouldn't be a negative number.

This change sets `0L` instead of `C.TIME_UNSET` when the
timer is disabled.

Issue: #903

#minor-release

PiperOrigin-RevId: 594451074
(cherry picked from commit 426bc94)
@androidx androidx locked and limited conversation to collaborators Mar 4, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

6 participants