Commit 5c5c8a11 authored Feb 15, 2019 by Eric Laurent

audio: use proper app ops API to check capture audio permission

Use AppOpsManager.checkOp() instead of AppOpsManager.noteOp() to verify
capture audio permission when creating an AudioRecord client.
This avoid false triggering of mic use UI indication if capture is not
actually active.

Test: verify main audio capture use cases
Change-Id: I38c7429bf2c8cdeaba907c868b94e39b69626c55

parent bb6d61d7

media/utils/ServiceUtilities.cpp

+1 −1

Original line number	Diff line number	Diff line
		@@ -85,7 +85,7 @@ static bool checkRecordingInternal(const String16& opPackageName, pid_t pid,
		return false;
		}
		} else {
		if (appOps.noteOp(op, uid, resolvedOpPackageName) != AppOpsManager::MODE_ALLOWED) {
		if (appOps.checkOp(op, uid, resolvedOpPackageName) != AppOpsManager::MODE_ALLOWED) {
		ALOGE("Request denied by app op: %d", op);
		return false;
		}