Fix error has leaked IntentReceiver in Android

When implementing a BroadcastReceiver , it is important to know that we have to register and unregister the receiver, but it is NOT good practice to cancel the receiver in onStop() and this is why, which you can see in the documentation:

When your implementation requires logging in onResume()there is an important consideration .

If you are registering a receiver in Activity.onResume(), you must unregister it in Activity.onPause(). (You will not receive intents when paused, and this can reduce unnecessary system overhead)

Regarding your error:

MainActivity has leaked IntentReceiver

the documentation itself states:

Don't forget to unregister the dynamically registered receiver using the Context.unregisterReceiver(). If you miss this, the Android system reports a Leaked error. For example, if you registered a receiver inonResume() your activity, it should be unregistered in theonPause() .

Therefore :

In the onResume() method , the registration of the receiver is usually done.

In the onPause() method , the unregistration of the receiver is generally performed.

I add an example:

public class MyActivity extends Activity
{

  private final BroadcastReceiver mybroadcast = new BroadcastReceiver ();

  //REGISTRA!
  public void onResume()
  {
    IntentFilter filter = new IntentFilter();
    filter.addAction("android.provider.Telephony.SMS_RECEIVED");
    registerReceiver(mybroadcast, filter);  

  }

  //CANCELA!
  public void onPause()
  {
    unregisterReceiver(mybroadcast);
  }
}

You should store your BroadcastReceiverin a variable of your MainActivityand in your onStop()putunregisterReceiver(myBroadcastReceiver);

You must take into account that if you go through it onStop()without having BroadcastReceiverpreviously registered it, an exception will be thrown.

All the best