From 7231438f124eff0ce0b9a8998ac7838abc094a8b Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sat, 12 Nov 2016 08:06:58 -0500 Subject: vim-patch:7.4.1640 Problem: Crash when an autocommand changes a quickfix list. (Dominique) Solution: Check wether an entry is still valid. (Yegappan Lakshmanan, Hirohito Higashi) https://github.com/vim/vim/commit/ffec3c53496d49668669deabc0724ec78e2274fd --- src/nvim/quickfix.c | 43 ++++++++++++++++++++++++++++++++++++++++++- src/nvim/version.c | 2 +- 2 files changed, 43 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/nvim/quickfix.c b/src/nvim/quickfix.c index d6697902ef..71fcb5e60f 100644 --- a/src/nvim/quickfix.c +++ b/src/nvim/quickfix.c @@ -1254,6 +1254,32 @@ static char_u *qf_guess_filepath(char_u *filename) } +/// When loading a file from the quickfix, the auto commands may modify it. +/// This may invalidate the current quickfix entry. This function checks +/// whether a entry is still present in the quickfix. +/// Similar to location list. +static bool is_qf_entry_present(qf_info_T *qi, qfline_T *qf_ptr) +{ + qf_list_T *qfl; + qfline_T *qfp; + int i; + + qfl = &qi->qf_lists[qi->qf_curlist]; + + // Search for the entry in the current list + for (i = 0, qfp = qfl->qf_start; i < qfl->qf_count; i++, qfp = qfp->qf_next) { + if (qfp == qf_ptr) { + break; + } + } + + if (i == qfl->qf_count) { // Entry is not found + return false; + } + + return true; +} + /* * jump to a quickfix line * if dir == FORWARD go "errornr" valid entries forward @@ -1585,14 +1611,29 @@ win_found: oldwin == curwin ? curwin : NULL); } } else { + int old_qf_curlist = qi->qf_curlist; + bool is_abort = false; + ok = buflist_getfile(qf_ptr->qf_fnum, (linenr_T)1, GETF_SETMARK | GETF_SWITCH, forceit); if (qi != &ql_info && !win_valid(oldwin)) { EMSG(_("E924: Current window was closed")); + is_abort = true; + opened_window = false; + } else if (old_qf_curlist != qi->qf_curlist + || !is_qf_entry_present(qi, qf_ptr)) { + if (qi == &ql_info) { + EMSG(_("E925: Current quickfix was changed")); + } else { + EMSG(_("E926: Current location list was changed")); + } + is_abort = true; + } + + if (is_abort) { ok = false; qi = NULL; qf_ptr = NULL; - opened_window = false; } } } diff --git a/src/nvim/version.c b/src/nvim/version.c index 7721e9dc8c..69438c50d0 100644 --- a/src/nvim/version.c +++ b/src/nvim/version.c @@ -804,7 +804,7 @@ static int included_patches[] = { 1643, 1642, 1641, - // 1640, + 1640, // 1639, // 1638, // 1637 NA -- cgit From c0fd830be4ef3eafb756c463aee6408323ba4e58 Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sat, 12 Nov 2016 13:52:50 -0500 Subject: Bump all nvim-specific error codes above E5000 In order to not conflict with new error codes that Vim adds, all Neovim error codes should be above 5000. The three existing sub-5000 error codes (E926, E951, and E952) are now E50003, E5004, and E5005 respectively. E953 was removed in 6167ce6df2753d5474ad49aea19f5957128ab015, so just remove it from the help. --- src/nvim/eval/encode.c | 4 ++-- src/nvim/undo.c | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/nvim/eval/encode.c b/src/nvim/eval/encode.c index 670437ceda..f6c58d8562 100644 --- a/src/nvim/eval/encode.c +++ b/src/nvim/eval/encode.c @@ -836,7 +836,7 @@ char *encode_tv2json(typval_T *tv, size_t *len) msgpack_pack_double(packer, (double) (flt)) #define TYPVAL_ENCODE_CONV_FUNC(fun) \ - return conv_error(_("E951: Error while dumping %s, %s: " \ + return conv_error(_("E5004: Error while dumping %s, %s: " \ "attempt to dump function reference"), \ mpstack, objname) @@ -880,7 +880,7 @@ char *encode_tv2json(typval_T *tv, size_t *len) #define TYPVAL_ENCODE_CONV_LIST_BETWEEN_ITEMS() #define TYPVAL_ENCODE_CONV_RECURSE(val, conv_type) \ - return conv_error(_("E952: Unable to dump %s: " \ + return conv_error(_("E5005: Unable to dump %s: " \ "container references itself in %s"), \ mpstack, objname) diff --git a/src/nvim/undo.c b/src/nvim/undo.c index d6428d63f7..4d56046bc1 100644 --- a/src/nvim/undo.c +++ b/src/nvim/undo.c @@ -691,7 +691,7 @@ char *u_get_undo_file_name(const char *const buf_ffname, const bool reading) int ret; char *failed_dir; if ((ret = os_mkdir_recurse(dir_name, 0755, &failed_dir)) != 0) { - EMSG3(_("E926: Unable to create directory \"%s\" for undo file: %s"), + EMSG3(_("E5003: Unable to create directory \"%s\" for undo file: %s"), failed_dir, os_strerror(ret)); xfree(failed_dir); } else { -- cgit From caa33aaaf8b044cf3a8311213d0841d2d9e591a5 Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sat, 12 Nov 2016 14:18:29 -0500 Subject: vim-patch:7.4.1647 Problem: Using freed memory after setqflist() and ":caddbuffer". (Dominique) Solution: Set qf_ptr when adding the first item to the quickfix list. https://github.com/vim/vim/commit/8b20179c657b4266dff115486ca68c6a50324071 --- src/nvim/quickfix.c | 6 +++++- src/nvim/version.c | 2 +- 2 files changed, 6 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/nvim/quickfix.c b/src/nvim/quickfix.c index 71fcb5e60f..6bb863985e 100644 --- a/src/nvim/quickfix.c +++ b/src/nvim/quickfix.c @@ -910,6 +910,8 @@ static int qf_add_entry(qf_info_T *qi, qfline_T **prevp, char_u *dir, if (qi->qf_lists[qi->qf_curlist].qf_count == 0) { /* first element in the list */ qi->qf_lists[qi->qf_curlist].qf_start = qfp; + qi->qf_lists[qi->qf_curlist].qf_ptr = qfp; + qi->qf_lists[qi->qf_curlist].qf_index = 0; qfp->qf_prev = qfp; /* first element points to itself */ } else { assert(*prevp); @@ -3621,7 +3623,9 @@ int set_errorlist(win_T *wp, list_T *list, int action, char_u *title) else qi->qf_lists[qi->qf_curlist].qf_nonevalid = FALSE; qi->qf_lists[qi->qf_curlist].qf_ptr = qi->qf_lists[qi->qf_curlist].qf_start; - qi->qf_lists[qi->qf_curlist].qf_index = 1; + if (qi->qf_lists[qi->qf_curlist].qf_count > 0) { + qi->qf_lists[qi->qf_curlist].qf_index = 1; + } qf_update_buffer(qi); diff --git a/src/nvim/version.c b/src/nvim/version.c index 69438c50d0..7e88672e69 100644 --- a/src/nvim/version.c +++ b/src/nvim/version.c @@ -797,7 +797,7 @@ static int included_patches[] = { // 1650, 1649, 1648, - // 1647, + 1647, // 1646 NA // 1645, // 1644, -- cgit From 830bf8665b6a98705b16d977ca7d11c2ca51dca8 Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sat, 12 Nov 2016 15:45:10 -0500 Subject: vim-patch:7.4.1650 Problem: Quickfix test fails. Solution: Accept any number of matches. https://github.com/vim/vim/commit/f68f1d70799631d38461c36cd59d08cf839b010d --- src/nvim/version.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/nvim/version.c b/src/nvim/version.c index 7e88672e69..b286f03301 100644 --- a/src/nvim/version.c +++ b/src/nvim/version.c @@ -794,7 +794,7 @@ static int included_patches[] = { // 1653 NA 1652, // 1651 NA - // 1650, + 1650, 1649, 1648, 1647, -- cgit From 2e5736e2cd4ef14ec23c1e0176fbcb17c9f55eba Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sat, 12 Nov 2016 15:58:19 -0500 Subject: vim-patch:7.4.1664 Problem: Crash in :cgetexpr. Solution: Check for NULL pointer. (Dominique) Add a test. https://github.com/vim/vim/commit/89c64d557dbe0bacfdd7b2872411b00cc1523d85 --- src/nvim/quickfix.c | 4 +++- src/nvim/version.c | 2 +- 2 files changed, 4 insertions(+), 2 deletions(-) (limited to 'src') diff --git a/src/nvim/quickfix.c b/src/nvim/quickfix.c index 6bb863985e..188de92809 100644 --- a/src/nvim/quickfix.c +++ b/src/nvim/quickfix.c @@ -483,8 +483,10 @@ qf_init_ext ( p_str += len; } else if (tv->v_type == VAR_LIST) { /* Get the next line from the supplied list */ - while (p_li && p_li->li_tv.v_type != VAR_STRING) + while (p_li && (p_li->li_tv.v_type != VAR_STRING + || p_li->li_tv.vval.v_string == NULL)) { p_li = p_li->li_next; /* Skip non-string items */ + } if (!p_li) /* End of the list */ break; diff --git a/src/nvim/version.c b/src/nvim/version.c index b286f03301..d0b802418d 100644 --- a/src/nvim/version.c +++ b/src/nvim/version.c @@ -780,7 +780,7 @@ static int included_patches[] = { // 1667 NA // 1666 NA // 1665 NA - // 1664, + 1664, 1663, // 1662 NA // 1661 NA -- cgit From 4e66bc99e4b08d518a9ad0692ee8409a90ffb0ff Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sat, 12 Nov 2016 16:07:02 -0500 Subject: Mark 7.4.1697 as applied The patch was merged in 9e1c6596 but version.c didn't get updated accordingly. --- src/nvim/version.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src') diff --git a/src/nvim/version.c b/src/nvim/version.c index d0b802418d..01823ce96e 100644 --- a/src/nvim/version.c +++ b/src/nvim/version.c @@ -747,7 +747,7 @@ static int included_patches[] = { 1700, // 1699, // 1698 NA - // 1697, + 1697, // 1696, 1695, // 1694 NA -- cgit From 34317846d6fa9c2d3b23742abc455eafca0e92e4 Mon Sep 17 00:00:00 2001 From: James McCoy Date: Sat, 12 Nov 2016 17:06:54 -0500 Subject: lint --- src/nvim/quickfix.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src') diff --git a/src/nvim/quickfix.c b/src/nvim/quickfix.c index 188de92809..f0d77f9e2b 100644 --- a/src/nvim/quickfix.c +++ b/src/nvim/quickfix.c @@ -482,10 +482,10 @@ qf_init_ext ( p_str += len; } else if (tv->v_type == VAR_LIST) { - /* Get the next line from the supplied list */ + // Get the next line from the supplied list while (p_li && (p_li->li_tv.v_type != VAR_STRING || p_li->li_tv.vval.v_string == NULL)) { - p_li = p_li->li_next; /* Skip non-string items */ + p_li = p_li->li_next; // Skip non-string items } if (!p_li) /* End of the list */ @@ -914,7 +914,7 @@ static int qf_add_entry(qf_info_T *qi, qfline_T **prevp, char_u *dir, qi->qf_lists[qi->qf_curlist].qf_start = qfp; qi->qf_lists[qi->qf_curlist].qf_ptr = qfp; qi->qf_lists[qi->qf_curlist].qf_index = 0; - qfp->qf_prev = qfp; /* first element points to itself */ + qfp->qf_prev = qfp; // first element points to itself } else { assert(*prevp); qfp->qf_prev = *prevp; -- cgit