commit ca195cc84fb3f5e497cdbc5f1f53dc1d7efedf48
author Yegappan Lakshmanan <yegappan@yahoo.com> Tue Jun 14 13:42:26 2022 +0100
committer Bram Moolenaar <Bram@vim.org> Tue Jun 14 13:42:26 2022 +0100
tree 709beb58d68cd3433060720a1c81cbfe6228715f
parent cd7496382efc9e6748326c6cda7f01003fa07063

patch 8.2.5089: some functions return a different value on failure

Problem:    Some functions return a different value on failure.
Solution:   Initialize the return value earlier. (Yegappan Lakshmanan,
            closes #10568)

src/autocmd.c

diff --git a/src/autocmd.c b/src/autocmd.c
index 71aeb50..841da14 100644
--- a/src/autocmd.c
+++ b/src/autocmd.c
@@ -3062,6 +3062,8 @@
     char_u	*name = NULL;
     int		group = AUGROUP_ALL;
 
+    if (rettv_list_alloc(rettv) == FAIL)
+	return;
     if (check_for_opt_dict_arg(argvars, 0) == FAIL)
 	return;
 
@@ -3128,8 +3130,6 @@
 	}
     }
 
-    if (rettv_list_alloc(rettv) == FAIL)
-	return;
     event_list = rettv->vval.v_list;
 
     // iterate through all the autocmd events