From fb02e9f1e95a72cc7e09465d706f67652ff3bb5e Mon Sep 17 00:00:00 2001 From: Jan Edmund Lazo Date: Sun, 5 May 2019 11:38:34 -0400 Subject: vim-patch:8.1.1046: the "secure" variable is used inconsistently Problem: the "secure" variable is used inconsistently. (Justin M. Keyes) Solution: Set it to one instead of incrementing. https://github.com/vim/vim/commit/82b033eff82d3ed0da77fd5f5a1c023766acabba --- src/nvim/buffer.c | 2 +- src/nvim/option.c | 5 ++--- 2 files changed, 3 insertions(+), 4 deletions(-) (limited to 'src') diff --git a/src/nvim/buffer.c b/src/nvim/buffer.c index d12ba14911..cdb226b94d 100644 --- a/src/nvim/buffer.c +++ b/src/nvim/buffer.c @@ -5116,7 +5116,7 @@ chk_modeline( save_SID = current_SID; current_SID = SID_MODELINE; // Make sure no risky things are executed as a side effect. - secure++; + secure = 1; retval = do_set(s, OPT_MODELINE | OPT_LOCAL | flags); diff --git a/src/nvim/option.c b/src/nvim/option.c index b93f98f6c5..743f6c8311 100644 --- a/src/nvim/option.c +++ b/src/nvim/option.c @@ -1840,11 +1840,10 @@ int do_set( // effects in secure mode. Also when the value was // set with the P_INSECURE flag and is not // completely replaced. - if (secure + if ((opt_flags & OPT_MODELINE) || sandbox != 0 - || (opt_flags & OPT_MODELINE) || (!value_is_replaced && (*p & P_INSECURE))) { - secure++; + secure = 1; } // Handle side effects, and set the global value -- cgit