patch 9.1.0478: potential deref of NULL pointer in fuzzy_match_str_with_pos
Commit: https://github.com/vim/vim/commit/7c57940e3bd883cebb1d73d4b4effa9220d4c4c8 Author: glepnir <glephun...@gmail.com> Date: Wed Jun 12 20:31:13 2024 +0200 patch 9.1.0478: potential deref of NULL pointer in fuzzy_match_str_with_pos Problem: potential deref of NULL pointer in fuzzy_match_str_with_pos() on cleanup (after v9.1.0476) Solution: Only free the pointer if it is non-NULL (glepnir) closes: #14973 Signed-off-by: glepnir <glephun...@gmail.com> Signed-off-by: Christian Brabandt <c...@256bit.org> diff --git a/src/search.c b/src/search.c index 864bfe3b8..20bf399b2 100644 --- a/src/search.c +++ b/src/search.c @@ -5198,11 +5198,16 @@ fuzzy_match_str_with_pos(char_u *str UNUSED, char_u *pat UNUSED) cleanup: vim_free(tv_str.vval.v_string); - list_free(match_str_list); - list_free(match_pos_list); - list_free(match_score_list); - list_free(retlist); - list_free(l); + if (match_str_list != NULL) + list_free(match_str_list); + if (match_pos_list != NULL) + list_free(match_pos_list); + if (match_score_list != NULL) + list_free(match_score_list); + if (retlist != NULL) + list_free(retlist); + if (l != NULL) + list_free(l); ga_clear(match_positions); return NULL; #else diff --git a/src/version.c b/src/version.c index cc2494521..4b695c01e 100644 --- a/src/version.c +++ b/src/version.c @@ -704,6 +704,8 @@ static char *(features[]) = static int included_patches[] = { /* Add new patch number below this line */ +/**/ + 478, /**/ 477, /**/ -- -- You received this message from the "vim_dev" maillist. Do not top-post! Type your reply below the text you are replying to. For more information, visit http://www.vim.org/maillist.php --- You received this message because you are subscribed to the Google Groups "vim_dev" group. To unsubscribe from this group and stop receiving emails from it, send an email to vim_dev+unsubscr...@googlegroups.com. To view this discussion on the web visit https://groups.google.com/d/msgid/vim_dev/E1sHSy3-008qyG-43%40256bit.org.
