diff options
author | scr@chromium.org <scr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-22 22:31:44 +0000 |
---|---|---|
committer | scr@chromium.org <scr@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-12-22 22:31:44 +0000 |
commit | 37e77e0728caea3744ca15956260904bff58efc2 (patch) | |
tree | 4e18e3e3a94332475c41e1dd03cda8f8e4d394a0 /base/file_util_posix.cc | |
parent | 80a9e31459509b307e67c7bf88bd7043a176e5d9 (diff) | |
download | chromium_src-37e77e0728caea3744ca15956260904bff58efc2.zip chromium_src-37e77e0728caea3744ca15956260904bff58efc2.tar.gz chromium_src-37e77e0728caea3744ca15956260904bff58efc2.tar.bz2 |
Coverity fix NEGATIVE_RETURNS.
CID=102629
R=mcgrathr@chromium.org, jar@chromium.org
BUG=
TEST=
Review URL: http://codereview.chromium.org/9007018
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@115630 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'base/file_util_posix.cc')
-rw-r--r-- | base/file_util_posix.cc | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/base/file_util_posix.cc b/base/file_util_posix.cc index 5c14abd..58cd732 100644 --- a/base/file_util_posix.cc +++ b/base/file_util_posix.cc @@ -927,7 +927,10 @@ bool DetermineDevShmExecutable() { if (fd >= 0) { ScopedFD shm_fd_closer(&fd); Delete(path, false); - size_t pagesize = sysconf(_SC_PAGESIZE); + long sysconf_result = sysconf(_SC_PAGESIZE); + CHECK_GE(sysconf_result, 0); + size_t pagesize = static_cast<size_t>(sysconf_result); + CHECK_GE(sizeof(pagesize), sizeof(sysconf_result)); void *mapping = mmap(NULL, pagesize, PROT_READ, MAP_SHARED, fd, 0); if (mapping != MAP_FAILED) { if (mprotect(mapping, pagesize, PROT_READ | PROT_EXEC) == 0) |