diff options
| author | Colin Kennedy <colinvfx@gmail.com> | 2024-03-04 21:25:15 -0800 |
|---|---|---|
| committer | zeertzjq <zeertzjq@outlook.com> | 2024-03-11 11:38:15 +0800 |
| commit | e8bc23db62c37e91d0c277b4bddf652db6a9d5f1 (patch) | |
| tree | 0b7909c3f67c1b480402919a3923dadfe5f712dd /src/nvim/quickfix.c | |
| parent | 141182d6c6c06ad56413b81a518ba9b777a0cbe0 (diff) | |
| download | rneovim-e8bc23db62c37e91d0c277b4bddf652db6a9d5f1.tar.gz rneovim-e8bc23db62c37e91d0c277b4bddf652db6a9d5f1.tar.bz2 rneovim-e8bc23db62c37e91d0c277b4bddf652db6a9d5f1.zip | |
vim-patch:9.1.0149: null pointer member access when accessing 'winfixbuf' property
Problem: qf_goto_win_with_qfl_file may check if prevwin has 'winfixbuf'
set without checking if it's valid first.
Solution: Reverse the condition. Add a test, a modeline, and a missing
CheckFeature. (Searn Dewar)
closes: vim/vim#14140
https://github.com/vim/vim/commit/5131f224da93f2e042a4b22545ef62b1b2ab8460
Co-authored-by: Sean Dewar <6256228+seandewar@users.noreply.github.com>
Diffstat (limited to 'src/nvim/quickfix.c')
| -rw-r--r-- | src/nvim/quickfix.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/nvim/quickfix.c b/src/nvim/quickfix.c index a88b781f32..3dd4e35f65 100644 --- a/src/nvim/quickfix.c +++ b/src/nvim/quickfix.c @@ -2699,7 +2699,7 @@ static void qf_goto_win_with_qfl_file(int qf_fnum) // Didn't find it, go to the window before the quickfix // window, unless 'switchbuf' contains 'uselast': in this case we // try to jump to the previously used window first. - if ((swb_flags & SWB_USELAST) && !prevwin->w_p_wfb && win_valid(prevwin)) { + if ((swb_flags & SWB_USELAST) && win_valid(prevwin) && !prevwin->w_p_wfb) { win = prevwin; } else if (altwin != NULL) { win = altwin; |