diff options
author | Eliseo Martínez <eliseomarmol@gmail.com> | 2015-03-17 19:08:44 +0100 |
---|---|---|
committer | Eliseo Martínez <eliseomarmol@gmail.com> | 2015-03-22 11:32:30 +0100 |
commit | c6784d9f6f2bcb648072486d90390e3760c579c6 (patch) | |
tree | f49476f09e85c5b591dc489bb762aa7e226b6968 /runtime/doc/manpages/it | |
parent | a2b4535747afcbd374cd97bbfd47457c52319723 (diff) | |
download | rneovim-c6784d9f6f2bcb648072486d90390e3760c579c6.tar.gz rneovim-c6784d9f6f2bcb648072486d90390e3760c579c6.tar.bz2 rneovim-c6784d9f6f2bcb648072486d90390e3760c579c6.zip |
coverity/105985: Resource leak: RI.
Problem : Resource leak @ 94, 98, 102.
Diagnostic : Real issue.
Rationale : Coverity doesn't know that uv_pipe_open will save file
descriptor to close them later. So, it signals file
descriptors being leaked. This would then seem like a false
positive we can fix by teaching coverity about uv_pipe_open
through model file.
But then we realize that the above is only true if
uv_pipe_open succeeds. It it fails, then descriptors are
really being leaked, which is why this is considered a real
issue and not a false positive after all.
Resolution : Add error handling to correctly close descriptors if
uv_pipe_open fails at any point.
Add model for uv_pipe_open so that Coverity knows it will
save descriptors when no error.
Helped-by: oni-link <knil.ino@gmail.com>
Diffstat (limited to 'runtime/doc/manpages/it')
0 files changed, 0 insertions, 0 deletions