diff options
| author | Gregory P. Smith <greg@krypto.org> | 2020-06-20 22:06:48 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-06-20 22:06:48 (GMT) |
| commit | 3ccb96c9782480e5ce646a4a130569fb92f2965d (patch) | |
| tree | edc57d4253707950df64855fb1809da2817eb42b /Modules/posixmodule.c | |
| parent | eb0d5c38de7f970d8cd8524f4163d831c7720f51 (diff) | |
| download | cpython-3ccb96c9782480e5ce646a4a130569fb92f2965d.zip cpython-3ccb96c9782480e5ce646a4a130569fb92f2965d.tar.gz cpython-3ccb96c9782480e5ce646a4a130569fb92f2965d.tar.bz2 | |
bpo-41056: Use the fildes converter for fd to please Coverity. (GH-21011)
There are a bunch of other fd: int uses in this file, I expect many if not
all of them would be better off using the fildes converter. This particular
one was flagged by Coverity as it presumably flags fpathconf as not accepting
negative fds. I'd expect the other fd's to have been flagged as well
otherwise.
I'm marking this one as skip news as it really is a no-op.
Diffstat (limited to 'Modules/posixmodule.c')
| -rw-r--r-- | Modules/posixmodule.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/Modules/posixmodule.c b/Modules/posixmodule.c index 79779bf..a411f28 100644 --- a/Modules/posixmodule.c +++ b/Modules/posixmodule.c @@ -10988,7 +10988,7 @@ conv_path_confname(PyObject *arg, int *valuep) /*[clinic input] os.fpathconf -> long - fd: int + fd: fildes name: path_confname / @@ -10999,7 +10999,7 @@ If there is no limit, return -1. static long os_fpathconf_impl(PyObject *module, int fd, int name) -/*[clinic end generated code: output=d5b7042425fc3e21 input=5942a024d3777810]*/ +/*[clinic end generated code: output=d5b7042425fc3e21 input=5b8d2471cfaae186]*/ { long limit; |