summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorAri Hausman-Cohen <arihc@google.com>2018-10-12 00:45:21 +0000
committerGerrit Code Review <noreply-gerritcodereview@google.com>2018-10-12 00:45:21 +0000
commiteef57caed9af7541d4cf8a8c694a808b5e039bf7 (patch)
tree2fe102824ea9871298fe25bc5918c28a9fb164da
parentc8096012179a66163720ba7373e7bdbf42862a58 (diff)
parent028763d9292a201f34fea501a20f484e8e0ed4fd (diff)
downloadlibhardware-oreo-mr1-1.2-iot-release.tar.gz
Merge "camera: avoid narrowing of the ioctl request"android-o-mr1-iot-release-smart-display-r3oreo-mr1-1.2-iot-release
-rw-r--r--modules/camera/3_4/v4l2_wrapper.cpp2
-rw-r--r--modules/camera/3_4/v4l2_wrapper.h2
2 files changed, 2 insertions, 2 deletions
diff --git a/modules/camera/3_4/v4l2_wrapper.cpp b/modules/camera/3_4/v4l2_wrapper.cpp
index 085683d6..bba826fb 100644
--- a/modules/camera/3_4/v4l2_wrapper.cpp
+++ b/modules/camera/3_4/v4l2_wrapper.cpp
@@ -129,7 +129,7 @@ void V4L2Wrapper::Disconnect() {
// Helper function. Should be used instead of ioctl throughout this class.
template <typename T>
-int V4L2Wrapper::IoctlLocked(int request, T data) {
+int V4L2Wrapper::IoctlLocked(unsigned long request, T data) {
// Potentially called so many times logging entry is a bad idea.
std::lock_guard<std::mutex> lock(device_lock_);
diff --git a/modules/camera/3_4/v4l2_wrapper.h b/modules/camera/3_4/v4l2_wrapper.h
index 4a8d5170..dcb8afea 100644
--- a/modules/camera/3_4/v4l2_wrapper.h
+++ b/modules/camera/3_4/v4l2_wrapper.h
@@ -99,7 +99,7 @@ class V4L2Wrapper {
void Disconnect();
// Perform an ioctl call in a thread-safe fashion.
template <typename T>
- int IoctlLocked(int request, T data);
+ int IoctlLocked(unsigned long request, T data);
// Request/release userspace buffer mode via VIDIOC_REQBUFS.
int RequestBuffers(uint32_t num_buffers);