LineageOS/android_development
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

am dd63895f: Fix camera ID validation

Browse Source 
* commit 'dd63895f1fb79667688219bdb1d4c71fce26fbab':
  Fix camera ID validation
This commit is contained in:
Vladimir Chtchetkine
2011-09-20 16:25:24 -07:00
committed by Android Git Automerger
parent 3e61d01e62 dd63895f1f
commit 2ec9af364a
2 changed files with 5 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
tools/emulator/system/camera/EmulatedCameraFactory.cpp
View File

@@ -123,7 +123,7 @@ int EmulatedCameraFactory::cameraDeviceOpen(int camera_id, hw_device_t** device)
return -EINVAL;
}
if (camera_id >= getEmulatedCameraNum()) {
if (camera_id < 0 || camera_id >= getEmulatedCameraNum()) {
LOGE("%s: Camera id %d is out of bounds (%d)",
__FUNCTION__, camera_id, getEmulatedCameraNum());
return -EINVAL;
@@ -141,7 +141,7 @@ int EmulatedCameraFactory::getCameraInfo(int camera_id, struct camera_info* info
return -EINVAL;
}
if (camera_id >= getEmulatedCameraNum()) {
if (camera_id < 0 || camera_id >= getEmulatedCameraNum()) {
LOGE("%s: Camera id %d is out of bounds (%d)",
__FUNCTION__, camera_id, getEmulatedCameraNum());
return -EINVAL;

7
tools/emulator/system/camera/EmulatedFakeCamera.cpp
View File

@@ -50,11 +50,10 @@ status_t EmulatedFakeCamera::Initialize()
}
/* Fake camera facing is defined by the qemu.sf.fake_camera boot property. */
const char* facing = EmulatedCamera::FACING_BACK;
char prop[PROPERTY_VALUE_MAX];
if (property_get("qemu.sf.fake_camera", prop, NULL) > 0) {
facing = prop;
}
property_get("qemu.sf.fake_camera", prop, EmulatedCamera::FACING_BACK);
const char* facing = prop;
mParameters.set(EmulatedCamera::FACING_KEY, facing);
LOGD("%s: Fake camera is facing %s", __FUNCTION__, facing);