diff options
author | Marco Nelissen <marcone@google.com> | 2016-06-07 12:26:43 -0700 |
---|---|---|
committer | The Android Automerger <android-build@google.com> | 2016-06-23 15:05:17 -0700 |
commit | 42a25c46b844518ff0d0b920c20c519e1417be69 (patch) | |
tree | a1854d073160a8c22ca22933dcdcc6ca2f48069e /media/libmedia/mediarecorder.cpp | |
parent | 8e438e153f661e9df8db0ac41d587e940352df06 (diff) | |
download | frameworks_av-42a25c46b844518ff0d0b920c20c519e1417be69.zip frameworks_av-42a25c46b844518ff0d0b920c20c519e1417be69.tar.gz frameworks_av-42a25c46b844518ff0d0b920c20c519e1417be69.tar.bz2 |
Don't use sp<>&
because they may end up pointing to NULL after a NULL check was performed.
Bug: 28166152
Change-Id: Iab2ea30395b620628cc6f3d067dd4f6fcda824fe
Diffstat (limited to 'media/libmedia/mediarecorder.cpp')
-rw-r--r-- | media/libmedia/mediarecorder.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/media/libmedia/mediarecorder.cpp b/media/libmedia/mediarecorder.cpp index 8bbd8f1..0eb7de5 100644 --- a/media/libmedia/mediarecorder.cpp +++ b/media/libmedia/mediarecorder.cpp @@ -617,7 +617,7 @@ MediaRecorder::MediaRecorder(const String16& opPackageName) : mSurfaceMediaSourc { ALOGV("constructor"); - const sp<IMediaPlayerService>& service(getMediaPlayerService()); + const sp<IMediaPlayerService> service(getMediaPlayerService()); if (service != NULL) { mMediaRecorder = service->createMediaRecorder(opPackageName); } |