From a5f989c4b7887be720c4f2fb8c3b098e5f2aef31 Mon Sep 17 00:00:00 2001 From: "groby@chromium.org" Date: Thu, 29 Mar 2012 18:37:38 +0000 Subject: [Coverity] Fixed possible resource leak Return value of 0 from open is a valid (if uncommon) result. Current code would leak that CID=101985 BUG= TEST= Review URL: https://chromiumcodereview.appspot.com/9837091 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@129656 0039d316-1c4b-4281-b951-d872f2087c98 --- media/video/capture/linux/video_capture_device_linux.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'media/video') diff --git a/media/video/capture/linux/video_capture_device_linux.cc b/media/video/capture/linux/video_capture_device_linux.cc index 6bc1a3a..7b99c204 100644 --- a/media/video/capture/linux/video_capture_device_linux.cc +++ b/media/video/capture/linux/video_capture_device_linux.cc @@ -81,7 +81,7 @@ void VideoCaptureDevice::GetDeviceNames(Names* device_names) { Name name; name.unique_id = path.value() + info.filename; - if ((fd = open(name.unique_id.c_str() , O_RDONLY)) <= 0) { + if ((fd = open(name.unique_id.c_str() , O_RDONLY)) < 0) { // Failed to open this device. continue; } -- cgit v1.1