From 21d40475bb68b2e6f36f29a1995c8d3aba79d5bf Mon Sep 17 00:00:00 2001 From: Eliseo Martínez Date: Thu, 30 Apr 2015 10:07:07 +0200 Subject: Fix warnings: ex_cmds2.c: ex_listdo(): Np dereference: FP. Problem : Dereference of null pointer @ 1903. Diagnostic : False positive. Rationale : Suggested error path first assumes wp == NULL and later one win_valid(wp), which is not possible. Resolution : Assert wp != NULL if win_valid(wp). --- src/nvim/ex_cmds2.c | 1 + 1 file changed, 1 insertion(+) (limited to 'src/nvim/ex_cmds2.c') diff --git a/src/nvim/ex_cmds2.c b/src/nvim/ex_cmds2.c index e453d68247..83f693ec14 100644 --- a/src/nvim/ex_cmds2.c +++ b/src/nvim/ex_cmds2.c @@ -1897,6 +1897,7 @@ void ex_listdo(exarg_T *eap) /* go to window "wp" */ if (!win_valid(wp)) break; + assert(wp); win_goto(wp); if (curwin != wp) break; /* something must be wrong */ -- cgit From 33adbf31399c7069386b6c72f9d15e67f9f569e0 Mon Sep 17 00:00:00 2001 From: Eliseo Martínez Date: Thu, 30 Apr 2015 10:13:53 +0200 Subject: Fix warnings: ex_cmds2.c: ex_listdo(): Np dereference: FP. Problem : Dereference of null pointer @ 1909. Diagnostic : False positive. Rationale : Suggested path error first assumes tp == NULL and later one valid_tabpage(tp), which is not possible. Resolution : Assert tp != NULL if valid_tabpage(tp). --- src/nvim/ex_cmds2.c | 1 + 1 file changed, 1 insertion(+) (limited to 'src/nvim/ex_cmds2.c') diff --git a/src/nvim/ex_cmds2.c b/src/nvim/ex_cmds2.c index 83f693ec14..b79d09885c 100644 --- a/src/nvim/ex_cmds2.c +++ b/src/nvim/ex_cmds2.c @@ -1906,6 +1906,7 @@ void ex_listdo(exarg_T *eap) /* go to window "tp" */ if (!valid_tabpage(tp)) break; + assert(tp); goto_tabpage_tp(tp, TRUE, TRUE); tp = tp->tp_next; } else if (eap->cmdidx == CMD_bufdo) { -- cgit