To: vim_dev@googlegroups.com Subject: Patch 8.2.1880 Fcc: outbox From: Bram Moolenaar Mime-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit ------------ Patch 8.2.1880 Problem: Vim9: Asan complains about adding zero to NULL. Solution: Check for argument count first. Files: src/vim9compile.c *** ../vim-8.2.1879/src/vim9compile.c 2020-10-21 16:42:18.517821898 +0200 --- src/vim9compile.c 2020-10-21 16:47:57.057129882 +0200 *************** *** 1475,1485 **** isn->isn_arg.shuffle.shfl_up = argoff - 1; } ! // Check the types of the arguments. ! argtypes = ((type_T **)stack->ga_data) + stack->ga_len - argcount; ! if (argcount > 0 && internal_func_check_arg_types( ! argtypes, func_idx, argcount) == FAIL) return FAIL; if ((isn = generate_instr(cctx, ISN_BCALL)) == NULL) return FAIL; --- 1475,1487 ---- isn->isn_arg.shuffle.shfl_up = argoff - 1; } ! if (argcount > 0) ! { ! // Check the types of the arguments. ! argtypes = ((type_T **)stack->ga_data) + stack->ga_len - argcount; ! if (internal_func_check_arg_types(argtypes, func_idx, argcount) == FAIL) return FAIL; + } if ((isn = generate_instr(cctx, ISN_BCALL)) == NULL) return FAIL; *** ../vim-8.2.1879/src/version.c 2020-10-21 16:42:18.517821898 +0200 --- src/version.c 2020-10-21 16:48:57.637012434 +0200 *************** *** 752,753 **** --- 752,755 ---- { /* Add new patch number below this line */ + /**/ + 1880, /**/ -- hundred-and-one symptoms of being an internet addict: 96. On Super Bowl Sunday, you followed the score by going to the Yahoo main page instead of turning on the TV. /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \\\ /// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\ \\\ an exciting new programming language -- http://www.Zimbu.org /// \\\ help me help AIDS victims -- http://ICCF-Holland.org ///