Attempt to invoke virtual method 'com.google.firebase.inappmessaging.model.MessageType kb.i.c()' #4214

gonojuarez · 2022-10-17T20:44:15Z

[READ] Step 1: Are you in the right place?

Issues filed here should be about bugs in the code in this repository.
If you have a general question, need help debugging, or fall into some
other category use one of these other channels:

For general technical questions, post a question on StackOverflow
with the firebase tag.
For general Firebase discussion, use the firebase-talk
google group.
For help troubleshooting your application that does not fall under one
of the above categories, reach out to the personalized
Firebase support channel.

[REQUIRED] Step 2: Describe your environment

Android Studio version: 2021.2.1 Patch 2
Firebase Component: inAppMessaging
Component version: 20.2.0

[REQUIRED] Step 3: Describe the problem

Firebase crash.

Steps to reproduce:

message.getMessageType is null

Relevant Code:

// branch code
private List<Action> extractActions(InAppMessage message) {
 List<Action> actions = new ArrayList<>();
 switch (message.getMessageType()) {
  case BANNER:
   actions.add(((BannerMessage) message).getAction());
  break;
  case CARD:
    actions.add(((CardMessage) message).getPrimaryAction());
    actions.add(((CardMessage) message).getSecondaryAction());
   break;
  case IMAGE_ONLY:
    actions.add(((ImageOnlyMessage) message).getAction());
  break;
  case MODAL: 
    actions.add(((ModalMessage) message).getAction());
  break;
  default:
   // An empty action is treated like a dismiss
     actions.add(Action.builder().build());
  }
   return actions;
}

// Log
Fatal Exception: java.lang.NullPointerException: Attempt to invoke virtual method 'com.google.firebase.inappmessaging.model.MessageType kb.i.c()' on a null object reference
       at com.google.firebase.inappmessaging.display.FirebaseInAppMessagingDisplay.extractActions(FirebaseInAppMessagingDisplay.java:437)
       at com.google.firebase.inappmessaging.display.FirebaseInAppMessagingDisplay.inflateBinding(FirebaseInAppMessagingDisplay.java:308)
       at com.google.firebase.inappmessaging.display.FirebaseInAppMessagingDisplay.access$000(FirebaseInAppMessagingDisplay.java:79)
       at com.google.firebase.inappmessaging.display.FirebaseInAppMessagingDisplay$1.run(FirebaseInAppMessagingDisplay.java:286)
       at android.os.Handler.handleCallback(Handler.java:938)
       at android.os.Handler.dispatchMessage(Handler.java:99)
       at android.os.Looper.loop(Looper.java:246)
       at android.app.ActivityThread.main(ActivityThread.java:8512)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:602)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1139)


// suggested code
private List<Action> extractActions(InAppMessage message) {
 List<Action> actions = new ArrayList<>();
  if(InAppMessage != null){
      switch (message.getMessageType()) {
         case BANNER:
            actions.add(((BannerMessage) message).getAction());
         break;
        case CARD:
           actions.add(((CardMessage) message).getPrimaryAction());
           actions.add(((CardMessage) message).getSecondaryAction());
         break;
         case IMAGE_ONLY:
           actions.add(((ImageOnlyMessage) message).getAction());
         break;
         case MODAL: 
           actions.add(((ModalMessage) message).getAction());
         break;
        default:
        // An empty action is treated like a dismiss
         actions.add(Action.builder().build());
      }
  }

   return actions;
}

The text was updated successfully, but these errors were encountered:

google-oss-bot · 2022-10-17T20:44:17Z

I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.

argzdev · 2022-10-18T16:56:54Z

Hi @gonojuarez, thanks for reporting and for the proposed solution! Let me pull up a PR real quick.

argzdev · 2022-10-19T17:38:16Z

Hi @gonojuarez, it looks like the InAppMessage there shouldn't be a null when reaching the method extractActions since we will start the inapp message rendering only if we have a valid object. Could you provide steps how were you able to reproduce this behavior? Or if there's any chance you could provide a MCVE, that'll help us a lot. Thanks!

thatfiredev · 2022-10-19T18:17:27Z

This seems to be similar to #3596

gonojuarez · 2022-10-20T01:05:28Z

I don't know how to reproduce it.
I read the code and understand the bug are related with the inAppMessage.

argzdev · 2022-10-20T14:57:00Z

Thanks for confirming @gonojuarez. Do you remember which type of message layout was selected in Firebase console?

Also can you confirm how frequent does this issue occur? Maybe that'll give us a hint what triggered the issue.

Thanks!

gonojuarez · 2022-10-20T15:18:28Z

I used the card option
In the last 90 days, This bug has affected 12 users and appeared 15 times.

argzdev · 2022-10-20T15:34:35Z

That is interesting, thank you for these details. I'll try to inform our engineer about this.

gonojuarez · 2022-10-20T18:18:00Z

Thank you @argzdev

argzdev · 2022-11-30T12:09:39Z

Hi @gonojuarez, sorry for the delayed response. We're still investigating this. May I ask how are you running your campaigns? Are you triggering multiple campaigns at the same time? If so, could you share what setup are you doing? It might give us a hint how your users encounter the issue.

We think that the issue might be due to a multithreading race condition. Since FirebaseInAppMessagingDisplay keeps a single instance of inAppMessage, I'm assuming If multiple inAppMessage is present, it might be possible that the moment one inAppMessage is set to null due to (see below) and another inAppMessage tries to access the extractAction, then that throws the NullPointerException.

Scenarios when inAppMessage is set to null:

Upon onClick of a button of an inAppMessage with action
Upon dismissFiam of a FIAM
Upon onError of loading an image of a FIAM.

vijayshuru · 2022-12-01T06:06:13Z

Yes, I am also getting the same crash again and again on the app open in testing mode. We have 3 active campaigns to test. I have completed two campaigns. Now my app is not crashing.

argzdev · 2022-12-01T09:10:06Z

Hi @vijayshuru, can you share your steps how to reproduce this behavior? Thanks!

viniciusalvesmello · 2022-12-01T13:58:58Z

I have the same problem 😥!

In this version 11.18.2 it was necessary to update the targetSdkVersion from 30 to 31, due to Google Play rules.

Trace Crashlytics

Fatal Exception: java.lang.NullPointerException: Attempt to invoke virtual method 'com.google.firebase.inappmessaging.model.MessageType com.google.firebase.inappmessaging.model.InAppMessage.getMessageType()' on a null object reference
       at com.google.firebase.inappmessaging.display.FirebaseInAppMessagingDisplay.extractActions(FirebaseInAppMessagingDisplay.java:437)
       at com.google.firebase.inappmessaging.display.FirebaseInAppMessagingDisplay.inflateBinding(FirebaseInAppMessagingDisplay.java:308)
       at com.google.firebase.inappmessaging.display.FirebaseInAppMessagingDisplay.access$000(FirebaseInAppMessagingDisplay.java:79)
       at com.google.firebase.inappmessaging.display.FirebaseInAppMessagingDisplay$1.run(FirebaseInAppMessagingDisplay.java:286)
       at android.os.Handler.handleCallback(Handler.java:883)
       at android.os.Handler.dispatchMessage(Handler.java:100)
       at android.os.Looper.loop(Looper.java:214)
       at android.app.ActivityThread.main(ActivityThread.java:7438)
       at java.lang.reflect.Method.invoke(Method.java)
       at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:493)
       at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:989)

gonojuarez · 2022-12-01T14:19:39Z

@argzdev
In my case, My teams have 3 or 4 campaigns activate using the in-app message.
That campaign contains a picture inside the dialogue and the message.

vijayshuru · 2022-12-02T13:50:50Z

Hi @vijayshuru, can you share you steps how to reproduce this behavior? Thanks!

Sorry, This does not happen with all users. Also a user gets a message once in a day through the campaign. So It is difficult to reproduce.

lotdrops · 2023-01-03T11:58:49Z

I am also facing this issue.
We have around 200k "active devices" on the play store console, and about 50 crashes (about 15 users). We send our campaigns to all users, so it is not something that is reproducible, it occurs "randomly to a few users". And it happens in different devices and android versions

argzdev · 2023-01-16T09:24:44Z

Hi all, while we investigate this further. Kindly leave a thumbs up on the author's post. So that we can track the issue and prioritize a deeper investigation depending on severity. Thanks!

eldhosembabu · 2023-02-16T22:05:40Z

After analyzing the issue, the inappmessage instance shouldn't be null when calling the extractActions method because if we check the call stack, it is getting called from showActiveFIAM method where we are already checking for null inappmessage instance as you can see here.

So i believe the inappmessage instance getting null is because of some race condition happening when dealing with displaying multiple inapp messages one after the other and probably the countdown timers of the prior inappmessage (say impressiontimer) resetting the current inapp message when they call dismiss fiam even though we are cancelling the timers. This is because, we have a timer tick reporting delay of 1 second as you can see here.

So I think the use case in which this issue could happen is when 1st inapp message gets dismissed which clears the inappmessage instance, but still waiting for timers to settle, before which the second inappmessage is processed for displaying and in between the previous "escaped timers" clears the inappmessage instance again and leads to the null pointer exception.

If the use case I think is correct, we could do the following fixes:

Simply adding a null check before calling the extractActions method - This will prevent the crash, but might postpone the inapp message to be displayed during the next event.
We could try reducing the timer reporting interval - This could reduce the crash if my theory is right.
Refactor the code to properly isolate the instances used for processing one inappmessage from other. This can be a long term fix if we find No. 1 and No. 2 fix is not useful enough.

I'll create a PR with null check and post here soon.

xiaomochn · 2023-02-21T03:45:14Z

I am also facing this issue, which version of the bom upgrade can solve the problem

google-oss-bot added the needs-triage label Oct 17, 2022

argzdev added api: inappmessaging and removed needs-triage labels Oct 18, 2022

argzdev mentioned this issue Oct 18, 2022

null checker for message type #4217

Closed

argzdev added the needs-info label Oct 19, 2022

google-oss-bot added needs-attention and removed needs-info labels Oct 20, 2022

argzdev added needs-info and removed needs-attention labels Oct 20, 2022

google-oss-bot added needs-attention and removed needs-info labels Oct 20, 2022

argzdev added needs-info and removed needs-attention labels Nov 30, 2022

google-oss-bot added needs-attention and removed needs-info labels Dec 1, 2022

argzdev removed the needs-attention label Jan 16, 2023

argzdev mentioned this issue Jan 30, 2023

inappmessaging NullPointerException #4607

Closed

eldhosembabu mentioned this issue Feb 16, 2023

Fiam crash fix #4691

Merged

eldhosembabu closed this as completed in #4691 Feb 21, 2023

firebase locked and limited conversation to collaborators Mar 24, 2023

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Attempt to invoke virtual method 'com.google.firebase.inappmessaging.model.MessageType kb.i.c()' #4214

Attempt to invoke virtual method 'com.google.firebase.inappmessaging.model.MessageType kb.i.c()' #4214

gonojuarez commented Oct 17, 2022 •

edited

Loading

google-oss-bot commented Oct 17, 2022

argzdev commented Oct 18, 2022

argzdev commented Oct 19, 2022 •

edited

Loading

thatfiredev commented Oct 19, 2022

gonojuarez commented Oct 20, 2022

argzdev commented Oct 20, 2022

gonojuarez commented Oct 20, 2022

argzdev commented Oct 20, 2022

gonojuarez commented Oct 20, 2022

argzdev commented Nov 30, 2022

vijayshuru commented Dec 1, 2022

argzdev commented Dec 1, 2022 •

edited

Loading

viniciusalvesmello commented Dec 1, 2022

gonojuarez commented Dec 1, 2022 •

edited

Loading

vijayshuru commented Dec 2, 2022

lotdrops commented Jan 3, 2023

argzdev commented Jan 16, 2023

eldhosembabu commented Feb 16, 2023 •

edited

Loading

xiaomochn commented Feb 21, 2023

Attempt to invoke virtual method 'com.google.firebase.inappmessaging.model.MessageType kb.i.c()' #4214

Attempt to invoke virtual method 'com.google.firebase.inappmessaging.model.MessageType kb.i.c()' #4214

Comments

gonojuarez commented Oct 17, 2022 • edited Loading

[READ] Step 1: Are you in the right place?

[REQUIRED] Step 2: Describe your environment

[REQUIRED] Step 3: Describe the problem

Steps to reproduce:

Relevant Code:

google-oss-bot commented Oct 17, 2022

argzdev commented Oct 18, 2022

argzdev commented Oct 19, 2022 • edited Loading

thatfiredev commented Oct 19, 2022

gonojuarez commented Oct 20, 2022

argzdev commented Oct 20, 2022

gonojuarez commented Oct 20, 2022

argzdev commented Oct 20, 2022

gonojuarez commented Oct 20, 2022

argzdev commented Nov 30, 2022

vijayshuru commented Dec 1, 2022

argzdev commented Dec 1, 2022 • edited Loading

viniciusalvesmello commented Dec 1, 2022

Trace Crashlytics

gonojuarez commented Dec 1, 2022 • edited Loading

vijayshuru commented Dec 2, 2022

lotdrops commented Jan 3, 2023

argzdev commented Jan 16, 2023

eldhosembabu commented Feb 16, 2023 • edited Loading

xiaomochn commented Feb 21, 2023

gonojuarez commented Oct 17, 2022 •

edited

Loading

argzdev commented Oct 19, 2022 •

edited

Loading

argzdev commented Dec 1, 2022 •

edited

Loading

gonojuarez commented Dec 1, 2022 •

edited

Loading

eldhosembabu commented Feb 16, 2023 •

edited

Loading