GitHub/neovim
1
0
Fork 0
mirror of https://github.com/neovim/neovim synced 2025-07-31 02:42:18 +00:00
Code Issues Packages Projects Releases Wiki Activity

file_search: free stackp if vim_findfile() failed

Browse Source 
ff_free_stack_element() accepts NULL ptr and returns early.
This removes the need to check if stackp is not NULL.
This commit is contained in:
Jan Edmund Lazo
2018-07-22 21:12:25 -04:00
parent 563957e3cb
commit f0a702d116
1 changed files with 6 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
src/nvim/file_search.c
View File

@ -577,7 +577,7 @@ char_u *vim_findfile(void *search_ctx_arg)
char_u *file_path;
char_u *rest_of_wildcards;
char_u *path_end = NULL;
ff_stack_T *stackp;
ff_stack_T *stackp = NULL;
size_t len;
char_u *p;
char_u *suf;
@ -964,6 +964,7 @@ char_u *vim_findfile(void *search_ctx_arg)
}
fail:
ff_free_stack_element(stackp);
xfree(file_path);
return NULL;
}
@ -1222,6 +1223,10 @@ static ff_stack_T *ff_pop(ff_search_ctx_T *search_ctx)
*/
static void ff_free_stack_element(ff_stack_T *stack_ptr)
{
if (stack_ptr == NULL) {
return;
}
/* free handles possible NULL pointers */
xfree(stack_ptr->ffs_fix_path);
xfree(stack_ptr->ffs_wc_path);