commit fc908d010b32bbb850a6306893ce7db166a932f6
author Marco Nelissen <marcone@google.com> Tue Jun 07 12:26:43 2016 -0700
committer Marco Nelissen <marcone@google.com> Thu Jun 09 14:19:58 2016 -0700
tree 7159c9a2686269fe5ac5d9f801bacbe3c5c0fb71
parent 596e81bc4f32f8a205b4885c03b240b90e7ef090

Don't use sp<>&

because they may end up pointing to NULL after a NULL check was performed.

Bug: 28166152
Change-Id: Iab2ea30395b620628cc6f3d067dd4f6fcda824fe

include/media/IMediaDeathNotifier.h

diff --git a/include/media/IMediaDeathNotifier.h b/include/media/IMediaDeathNotifier.h
index bb3d0d8..aca6678 100644
--- a/include/media/IMediaDeathNotifier.h
+++ b/include/media/IMediaDeathNotifier.h
@@ -30,7 +30,7 @@
     virtual ~IMediaDeathNotifier() { removeObitRecipient(this); }
 
     virtual void died() = 0;
-    static const sp<IMediaPlayerService>& getMediaPlayerService();
+    static const sp<IMediaPlayerService> getMediaPlayerService();
 
 private:
     IMediaDeathNotifier &operator=(const IMediaDeathNotifier &);