diff options
author | groby@chromium.org <groby@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-29 18:37:38 +0000 |
---|---|---|
committer | groby@chromium.org <groby@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2012-03-29 18:37:38 +0000 |
commit | a5f989c4b7887be720c4f2fb8c3b098e5f2aef31 (patch) | |
tree | 56173c1902606ed027452d202bd499c757948a15 /media/video | |
parent | 67cbaeb9b98b1f418dbe4e6d13a0e2748f88f278 (diff) | |
download | chromium_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.cc | 2 |
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; } |