diff options
| author | James McCoy <jamessan@jamessan.com> | 2022-05-19 22:08:20 -0400 |
|---|---|---|
| committer | James McCoy <jamessan@jamessan.com> | 2022-05-20 08:14:37 -0400 |
| commit | 6954c0ba0dd6dfeed7067c7a06c163bd958e3d10 (patch) | |
| tree | 4a673ce4518b23f7a4d8fe4646c09eb1c00a145f | |
| parent | 1da7e2b8ca1c89d69aec0830ce5ff1885cb75905 (diff) | |
| download | rneovim-6954c0ba0dd6dfeed7067c7a06c163bd958e3d10.tar.gz rneovim-6954c0ba0dd6dfeed7067c7a06c163bd958e3d10.tar.bz2 rneovim-6954c0ba0dd6dfeed7067c7a06c163bd958e3d10.zip | |
ci(coverity): annotate register_cfunc as leaking memory
register_cfunc allocates a ufunc_T, but doesn't store the pointer
anywhere before returning. The uf_name member variable is stored in a
hashtable and used to lookup the ufunc_T later, but that's too much for
Coverity to track.
Adding the annotation ensures that any new callers to register_cfunc
don't pop up as new "leaks" in the Coverity scans.
| -rw-r--r-- | src/nvim/eval/userfunc.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/src/nvim/eval/userfunc.c b/src/nvim/eval/userfunc.c index e5f48501f7..2059d423d5 100644 --- a/src/nvim/eval/userfunc.c +++ b/src/nvim/eval/userfunc.c @@ -3598,5 +3598,6 @@ char_u *register_cfunc(cfunc_T cb, cfunc_free_T cb_free, void *state) STRCPY(fp->uf_name, name); hash_add(&func_hashtab, UF2HIKEY(fp)); + // coverity[leaked_storage] return fp->uf_name; } |