From 0dcd39bad5e5aa70ba8f60cb3fbf0658585f0151 Mon Sep 17 00:00:00 2001 From: Bram Moolenaar Date: Wed, 3 Feb 2021 19:44:25 +0100 Subject: patch 8.2.2458: Coverity warns for :retab using freed memory Problem: Coverity warns for :retab using freed memory. Solution: Use the updated line pointer when moving text properties. --- src/indent.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) (limited to 'src/indent.c') diff --git a/src/indent.c b/src/indent.c index a9d406e3a..6ac6182df 100644 --- a/src/indent.c +++ b/src/indent.c @@ -1662,7 +1662,9 @@ ex_retab(exarg_T *eap) ptr = new_line + start_col; for (col = 0; col < len; col++) ptr[col] = (col < num_tabs) ? '\t' : ' '; - ml_replace(lnum, new_line, FALSE); + if (ml_replace(lnum, new_line, FALSE) == OK) + // "new_line" may have been copied + new_line = curbuf->b_ml.ml_line_ptr; if (first_line == 0) first_line = lnum; last_line = lnum; -- cgit v1.2.1