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 /src/nvim/path.c | |
| 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 'src/nvim/path.c')
0 files changed, 0 insertions, 0 deletions
