Cuando se implementa un [BroadcastReceiver][1], es importante saber que tenemos que registrar y desregistrar el receiver.

en cuanto a tu error, la documentación misma lo señala :

> No olvides cancelar el registro de receiver registrado en forma
> dinámica mediante el método `Context.unregisterReceiver()`. Si se olvida
> de esto, el sistema Android reporta un Leaked error. Por ejemplo, **si
> registraste un receiver en `onResume()`** de su actividad, se debe
> **cancelar el registro en el método `onPause()`**.


Por lo tanto :

> **En el método [onResume()][2] generalmente se realiza el registro del receiver.** 

> **En el método [onPause()][3] generamente se realiza el desregistro del receiver.**


**NO es buena practica cancelar el receiver en onStop() y esta es la razón,** la cual puedes ver en la documentación:

Cuando tu implementación requiere el registro en `onResume()` existe una consideración [importante][1].


> Si está registrando un receiver en `Activity.onResume()`, debes anular
> el registro en `Activity.onPause()`. (No recibirá intents cuando este en
> pausa, y esto puede reducir innecesaria sobrecarga del sistema)

Agrego un ejemplo:

    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);
      }
    }

  [1]: https://developer.android.com/reference/android/content/BroadcastReceiver.html
  [2]: https://developer.android.com/reference/android/app/Activity.html#onStart()
  [3]: https://developer.android.com/reference/android/app/Activity.html#onStop()