From 9b77fd1272b954aa7a63ba9cceea2704045f05ce Mon Sep 17 00:00:00 2001 From: Greg Kaiser Date: Mon, 5 Dec 2022 17:06:02 -0800 Subject: [PATCH] Fix NULL check We were checking 'info' twice and not checking 'msgs'. So we fix that up, and fix a typo in the message. Bug: 219974678 Test: TreeHugger Change-Id: Ia52178b758dc3d9753731b043da01743ff59e10c --- camera/device/default/ExternalCameraDeviceSession.cpp | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/camera/device/default/ExternalCameraDeviceSession.cpp b/camera/device/default/ExternalCameraDeviceSession.cpp index 68c6d2eed3..736bc3cbcd 100644 --- a/camera/device/default/ExternalCameraDeviceSession.cpp +++ b/camera/device/default/ExternalCameraDeviceSession.cpp @@ -714,8 +714,8 @@ Status ExternalCameraDeviceSession::switchToOffline( return Status::ILLEGAL_ARGUMENT; } - if (info == nullptr || results == nullptr || info == nullptr || session == nullptr) { - ALOGE("%s, output arguments (%p, %p, %p, %p) much not be null", __FUNCTION__, msgs, results, + if (msgs == nullptr || results == nullptr || info == nullptr || session == nullptr) { + ALOGE("%s, output arguments (%p, %p, %p, %p) must not be null", __FUNCTION__, msgs, results, info, session); }