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.
diff --git a/src/indent.c b/src/indent.c
index a9d406e..6ac6182 100644
--- a/src/indent.c
+++ b/src/indent.c
@@ -1662,7 +1662,9 @@
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;