android memory leak in notification service

Question

I have a service which creates a notification and then updates it with certain information periodically. After about 12 mins or so the phone crashes and reboots, I believe it is caused by a memory leak in the following code to do with how I am updating the notification, could someone please check/advise me if this is the case and what I am doing wrong.

onCreate:

mNotificationManager = (NotificationManager) getSystemService(NOTIFICATION_SERVICE);

createNotification:

private void createNotification() {
  Intent contentIntent = new Intent(this,MainScreen.class);
  contentIntent.setFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP | Intent.FLAG_ACTIVITY_SINGLE_TOP);
  PendingIntent appIntent =PendingIntent.getActivity(this,0, contentIntent, 0);

  contentView = new RemoteViews(getPackageName(), R.layout.notification);
  contentView.setImageViewResource(R.id.image, R.drawable.icon);
  contentView.setTextViewText(R.id.text, "");

  notification = new Notification();
  notification.when=System.currentTimeMillis();
  notification.contentView = contentView;
  notification.contentIntent = appIntent;
}

updateNotification:

private void updateNotification(String text){
  contentView.setTextViewText(R.id.text, text);
  mNotificationManager.notify(0, notification);
}

Thanks in advance.

It's just GC getting bigger and bigger until it causes problems and starts killing stuff and reboots — stealthcopter, Aug 29 '10 at 17:01
Make sure that it is for sure your application. Try fully uninstalling it and see if it still happens. Might be some other app you installed recently. — Moncader, Aug 30 '10 at 09:56
Are you sure it's this service? As in if you comment out everything the service does does it still have the same problem? — fredley, Aug 31 '10 at 09:25
Do you see OutOfMemoryError anywhere? Either as a pop-up or in the Logcat? — gary, Sep 03 '10 at 03:06
I've seen the same behaviour myself - the solution was to remove the progress bar. — plouh, Sep 13 '10 at 21:49
I have posted a very similar case with code and logcat output here: > http://stackoverflow.com/q/3608684/435855 I hope it helps — Urizev, Aug 31 '10 at 12:19
Same here, but phone isn't rebooting.. Only the notification gets frozen after some time. — Martin Pfeffer, Apr 11 '17 at 06:58

score 9 · Accepted Answer · answered Dec 19 '10 at 19:30

9

I stumbled upon the same problem. Looks like that if you don't "cache" the RemoteView and Notification in the service, but re-create them from scratch in the "update" routine this problem disappears. Yes, I know it is not efficient, but at least the phone does not reboot from out of memory errors.

answered Dec 19 '10 at 19:30

haimg

4,547
35
47

Hmmm weird, I'll try this after the holidays to confirm. – stealthcopter Dec 22 '10 at 21:06
Hi, I am updating a notification's Remote View (precisely 3 textviews inside it) continuously (after each second) using a service. The phone gets super slow and freezes after some time. Should I recreate notification for this problem too? – berserk Jan 09 '14 at 05:07
@berserk, Yes, you should. – grepsedawk Mar 10 '14 at 08:30
@Sakiboy as you have said dont update it too often...but i am using progress to show download...so i have to update it as per my download...what can i do? – H Raval Aug 05 '16 at 10:51
@HRaval , check out my answer here: http://stackoverflow.com/a/38402214/2371425. Basically you only want to update the notification if and only if the progress actually changed. – Sakiboy Aug 09 '16 at 00:09

Alesqui · Answer 2 · 2013-05-10T12:28:44.127

I had the very same problem. My solution is close to the one that @haimg said, but I do cache the notification (just the RemoteView is recreated). By doing so, the notification won't flash again if you are looking at it.

Example:

public void createNotification(Context context){
    Notification.Builder builder = new Notification.Builder(context);

    // Set notification stuff...

    // Build the notification
    notification = builder.build();
}

public void updateNotification(){
    notification.bigContentView = getBigContentView();
    notification.contentView = getCompactContentView();

    mNM.notify(NOTIFICATION_ID, notification);
}

And in the methods getBigContentView and getCompactContentView I return a new RemoteViews with the updated layout.

android memory leak in notification service

2 Answers2

Linked