summaryrefslogtreecommitdiffstats
path: root/media/video
diff options
context:
space:
mode:
authorgroby@chromium.org <groby@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-03-29 18:37:38 +0000
committergroby@chromium.org <groby@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2012-03-29 18:37:38 +0000
commita5f989c4b7887be720c4f2fb8c3b098e5f2aef31 (patch)
tree56173c1902606ed027452d202bd499c757948a15 /media/video
parent67cbaeb9b98b1f418dbe4e6d13a0e2748f88f278 (diff)
downloadchromium_src-a5f989c4b7887be720c4f2fb8c3b098e5f2aef31.zip
chromium_src-a5f989c4b7887be720c4f2fb8c3b098e5f2aef31.tar.gz
chromium_src-a5f989c4b7887be720c4f2fb8c3b098e5f2aef31.tar.bz2
[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
Diffstat (limited to 'media/video')
-rw-r--r--media/video/capture/linux/video_capture_device_linux.cc2
1 files changed, 1 insertions, 1 deletions
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;
}