]> git.ipfire.org Git - thirdparty/vim.git/commitdiff
patch 9.2.0302: runtime(netrw): RFC2396 decoding double escaping spaces v9.2.0302
authorChristian Brabandt <cb@256bit.org>
Sun, 5 Apr 2026 16:34:24 +0000 (16:34 +0000)
committerChristian Brabandt <cb@256bit.org>
Sun, 5 Apr 2026 16:36:42 +0000 (16:36 +0000)
Problem:  runtime(netrw): RFC2396 decoding double escaping spaces
          (lilydjwg, after 3e60f03d942d6bb0f7eac)
Solution: Remove escape() call, since we are using fnameescape() anyhow

fixes: #19913

Signed-off-by: Christian Brabandt <cb@256bit.org>
runtime/pack/dist/opt/netrw/autoload/netrw.vim
src/testdir/test_plugin_netrw.vim
src/version.c

index 1520ff969e2db4592f4d627b808149b09c71e1af..8e5fdb5397f54fdada4c71a2b4ed617c38559070 100644 (file)
@@ -23,6 +23,7 @@
 " 2026 Feb 27 by Vim Project Make the hostname validation more strict
 " 2026 Mar 01 by Vim Project include portnumber in hostname checking #19533
 " 2026 Apr 01 by Vim Project use fnameescape() with netrw#FileUrlEdit()
+" 2026 Apr 05 by Vim Project Fix netrw#RFC2396() #19913
 " Copyright:  Copyright (C) 2016 Charles E. Campbell {{{1
 "             Permission is hereby granted to use and distribute this code,
 "             with or without modifications, provided that this copyright
@@ -8826,8 +8827,7 @@ endfunction
 
 "  netrw#RFC2396: converts %xx into characters {{{2
 function netrw#RFC2396(fname)
-    let fname = escape(substitute(a:fname,'%\(\x\x\)','\=printf("%c","0x".submatch(1))','ge')," \t")
-    return fname
+    return substitute(a:fname, '%\(\x\x\)', '\=printf("%c","0x".submatch(1))', 'ge')
 endfunction
 
 " netrw#UserMaps: supports user-specified maps {{{2
index 2e5da13d750a3573f2957fed6c1cbe7356332bcf..b23467092815cce0b6a0c011b1beb4fbffd76a92 100644 (file)
@@ -604,4 +604,9 @@ func Test_netrw_FileUrlEdit_pipe_injection()
   call assert_false(filereadable(fname), 'Command injection via pipe in file URL')
 endfunc
 
+func Test_netrw_RFC2396()
+  let fname = 'a%20b'
+  call assert_equal('a b', netrw#RFC2396(fname))
+endfunc
+
 " vim:ts=8 sts=2 sw=2 et
index d25ba8f0e7be4024dfad0609a1f9441794eda1f3..ba00c5a7be73418ef9f617bce53a8345efec9712 100644 (file)
@@ -734,6 +734,8 @@ static char *(features[]) =
 
 static int included_patches[] =
 {   /* Add new patch number below this line */
+/**/
+    302,
 /**/
     301,
 /**/