java.lang.NullPointerException android.view.View.onResolvePointerIcon

[AntTheAlchemist]

Google Play has reported this crash 80 times in the past 28 days. I believe it's something newish, because I can't remember seeing it before. It seems to be exclusive to TVs running Android 14.

Stack trace:

Exception java.lang.NullPointerException: Attempt to invoke virtual method 'int android.view.InputDevice.getVendorId()' on a null object reference
  at android.view.View.onResolvePointerIcon (View.java:29569)
  at android.view.ViewGroup.dispatchResolvePointerIcon (ViewGroup.java:2092)
  at android.view.ViewGroup.onResolvePointerIcon (ViewGroup.java:2067)
  at android.view.ViewGroup.dispatchResolvePointerIcon (ViewGroup.java:2092)
  at android.view.ViewGroup.onResolvePointerIcon (ViewGroup.java:2067)
  at android.view.ViewGroup.dispatchResolvePointerIcon (ViewGroup.java:2092)
  at android.view.ViewGroup.onResolvePointerIcon (ViewGroup.java:2067)
  at android.view.ViewGroup.dispatchResolvePointerIcon (ViewGroup.java:2092)
  at android.view.ViewGroup.onResolvePointerIcon (ViewGroup.java:2067)
  at android.view.ViewRootImpl.updatePointerIcon (ViewRootImpl.java:7116)
  at android.view.ViewRootImpl.resetPointerIcon (ViewRootImpl.java:7092)
  at android.view.ViewRootImpl.-$$Nest$mresetPointerIcon (Unknown Source)
  at android.view.ViewRootImpl$ViewRootHandler.handleMessageImpl (ViewRootImpl.java:6046)
  at android.view.ViewRootImpl$ViewRootHandler.handleMessage (ViewRootImpl.java:5862)
  at android.os.Handler.dispatchMessage (Handler.java:106)

[slouken]

As far as I can tell, this is internal to the OS, not related to SDL. Have you investigated and found any more information?

[AntTheAlchemist]

Ah yes, I saw "InputDevice / VedrorID" and automatically assumed SDL was involved - my bad. This seems to be an Android 14 TV internal framework bug. Guys over at Cloudstream (thanks @CranberrySoup) and some posts on Stackoverflow suggest that a lot of people outside SDL are experiencing the same. Please close as not our bug...

[1bsyl]

maybe something like ?

import android.content.Context;
import android.graphics.drawable.Icon;
import android.view.MotionEvent;
import android.view.View;

public class SafeView extends View {

    public SafeView(Context context) {
        super(context);
    }

    @Override
    public Icon onResolvePointerIcon(MotionEvent event, int pointerIndex) {
        try {
            return super.onResolvePointerIcon(event, pointerIndex);
        } catch (NullPointerException e) {
            // Log the exception if needed, but prevent the crash
            e.printStackTrace();
            return null;
        }
    }
}

[1bsyl]

I will test a patch in production and put some news in a while.

[1bsyl]

Here's a PR, I'll report if there are any issue