summaryrefslogtreecommitdiffstats
path: root/media
diff options
context:
space:
mode:
authorJames Dong <jdong@google.com>2011-10-18 22:29:20 -0700
committerJames Dong <jdong@google.com>2011-10-18 22:29:20 -0700
commite579e28089c6b86f56d257030b778c5e22b1a5a5 (patch)
tree870e0181874d16cba160ab7ad46202ad7a00292a /media
parentc4ff709bd714286ea4b1eaf8d932c43a02d5430d (diff)
downloadframeworks_av-e579e28089c6b86f56d257030b778c5e22b1a5a5.zip
frameworks_av-e579e28089c6b86f56d257030b778c5e22b1a5a5.tar.gz
frameworks_av-e579e28089c6b86f56d257030b778c5e22b1a5a5.tar.bz2
Check whether media recorder client exists before dumping.
Change-Id: I1f3a644a958975e4cf6c02099c53e30cc4d2fd82 related-to-bug: 5477177
Diffstat (limited to 'media')
-rw-r--r--media/libmediaplayerservice/MediaPlayerService.cpp12
1 files changed, 7 insertions, 5 deletions
diff --git a/media/libmediaplayerservice/MediaPlayerService.cpp b/media/libmediaplayerservice/MediaPlayerService.cpp
index b5eef94..3ebe989 100644
--- a/media/libmediaplayerservice/MediaPlayerService.cpp
+++ b/media/libmediaplayerservice/MediaPlayerService.cpp
@@ -374,11 +374,13 @@ status_t MediaPlayerService::dump(int fd, const Vector<String16>& args)
} else {
for (int i = 0, n = mMediaRecorderClients.size(); i < n; ++i) {
sp<MediaRecorderClient> c = mMediaRecorderClients[i].promote();
- snprintf(buffer, 255, " MediaRecorderClient pid(%d)\n", c->mPid);
- result.append(buffer);
- write(fd, result.string(), result.size());
- result = "\n";
- c->dump(fd, args);
+ if (c != 0) {
+ snprintf(buffer, 255, " MediaRecorderClient pid(%d)\n", c->mPid);
+ result.append(buffer);
+ write(fd, result.string(), result.size());
+ result = "\n";
+ c->dump(fd, args);
+ }
}
}