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

media/libmedia/IMediaDeathNotifier.cpp

diff --git a/media/libmedia/IMediaDeathNotifier.cpp b/media/libmedia/IMediaDeathNotifier.cpp
index d4360ea..c43ef66 100644
--- a/media/libmedia/IMediaDeathNotifier.cpp
+++ b/media/libmedia/IMediaDeathNotifier.cpp
@@ -31,7 +31,7 @@
 SortedVector< wp<IMediaDeathNotifier> > IMediaDeathNotifier::sObitRecipients;
 
 // establish binder interface to MediaPlayerService
-/*static*/const sp<IMediaPlayerService>&
+/*static*/const sp<IMediaPlayerService>
 IMediaDeathNotifier::getMediaPlayerService()
 {
     ALOGV("getMediaPlayerService");