Mercurial > vim
changeset 26450:aea4d5e1e9d5 v8.2.3755
patch 8.2.3755: Coverity warns for using a buffer in another scope
Commit: https://github.com/vim/vim/commit/92c33eb2735045d25fceed3b607f44d7156c59c1
Author: Bram Moolenaar <Bram@vim.org>
Date: Tue Dec 7 11:03:39 2021 +0000
patch 8.2.3755: Coverity warns for using a buffer in another scope
Problem: Coverity warns for using a buffer in another scope.
Solution: Declare the buffer in a common scope.
| author | Bram Moolenaar <Bram@vim.org> |
|---|---|
| date | Tue, 07 Dec 2021 12:15:05 +0100 |
| parents | f2e54e4adab9 |
| children | 27720705109b |
| files | src/evalvars.c src/version.c |
| diffstat | 2 files changed, 4 insertions(+), 3 deletions(-) [+] |
line wrap: on
line diff
--- a/src/evalvars.c +++ b/src/evalvars.c @@ -1393,6 +1393,8 @@ ex_let_option( int failed = FALSE; int opt_p_flags; char_u *tofree = NULL; + char_u numbuf[NUMBUFLEN]; + c1 = *p; *p = NUL; @@ -1416,12 +1418,9 @@ ex_let_option( if (opt_p_flags & P_FUNC && (tv->v_type == VAR_PARTIAL || tv->v_type == VAR_FUNC)) { - char_u numbuf[NUMBUFLEN]; - // If the option can be set to a function reference or a lambda // and the passed value is a function reference, then convert it to // the name (string) of the function reference. - s = tv2string(tv, &tofree, numbuf, 0); } // Avoid setting a string option to the text "v:false" or similar.