Mercurial > vim

changeset 33578:0c36813bed95

Find changesets by keywords (author, files, the commit message), revision number or hash, or revset expression.
patch 9.0.2033: gcc overflow-warning for f_resolve Commit: https://github.com/vim/vim/commit/215c3261a25f7a99e8711a3b3c6158119c6aea9e Author: Ken Takata <kentkt@csc.jp> Date: Mon Oct 16 09:57:43 2023 +0200 patch 9.0.2033: gcc overflow-warning for f_resolve Problem: gcc overflow-warning for f_resolve Solution: use pointer p instead of pointer q[-1] Suppress the following warning: ``` filepath.c: In function ?f_resolve?: filepath.c:2162:27: warning: writing 1 byte into a region of size 0 [-Wstringop-overflow=] 2162 | q[-1] = NUL; ``` Closes: #13352 closes: #13353 Signed-off-by: Christian Brabandt <cb@256bit.org> Co-authored-by: Ken Takata <kentkt@csc.jp>
author Christian Brabandt <cb@256bit.org>
date Mon, 16 Oct 2023 10:00:11 +0200
parents d6dde6229b36
children 1f154b010b66
files src/filepath.c src/version.c
diffstat 2 files changed, 3 insertions(+), 1 deletions(-) [+]
line wrap: on
line diff
--- a/src/filepath.c
+++ b/src/filepath.c
@@ -2159,7 +2159,7 @@ f_resolve(typval_T *argvars, typval_T *r
 		if (q > p && *q == NUL)
 		{
 		    // Ignore trailing path separator.
-		    q[-1] = NUL;
+		    p[q - p - 1] = NUL;
 		    q = gettail(p);
 		}
 		if (q > p && !mch_isFullName(buf))
--- a/src/version.c
+++ b/src/version.c
@@ -705,6 +705,8 @@ static char *(features[]) =
 static int included_patches[] =
 {   /* Add new patch number below this line */
 /**/
+    2033,
+/**/
     2032,
 /**/
     2031,