[packages/vim.git] / 7.1.223

To: vim-dev@vim.org
Subject: Patch 7.1.223
Fcc: outbox
From: Bram Moolenaar <Bram@moolenaar.net>
Mime-Version: 1.0
Content-Type: text/plain; charset=ISO-8859-1
Content-Transfer-Encoding: 8bit
------------

Patch 7.1.223
Problem:    glob() doesn't work properly when 'shell' is "sh" or "bash" and
	    the expanded name contains spaces, '~', single quotes and other
	    special characters.  (Adri Verhoef, Charles Campbell)
Solution:   For Posix shells define a vimglob() function to list the matches
	    instead of using "echo" directly.
Files:	    src/os_unix.c


*** ../vim-7.1.222/src/os_unix.c	Thu Jan  3 18:55:21 2008
--- src/os_unix.c	Sun Jan 13 13:52:53 2008
***************
*** 4946,4951 ****
--- 4946,4954 ----
      char_u	*p;
      int		dir;
  #ifdef __EMX__
+     /*
+      * This is the OS/2 implementation.
+      */
  # define EXPL_ALLOC_INC	16
      char_u	**expl_files;
      size_t	files_alloced, files_free;
***************
*** 5056,5075 ****
      return OK;
  
  #else /* __EMX__ */
! 
      int		j;
      char_u	*tempname;
      char_u	*command;
      FILE	*fd;
      char_u	*buffer;
! #define STYLE_ECHO  0	    /* use "echo" to expand */
! #define STYLE_GLOB  1	    /* use "glob" to expand, for csh */
! #define STYLE_PRINT 2	    /* use "print -N" to expand, for zsh */
! #define STYLE_BT    3	    /* `cmd` expansion, execute the pattern directly */
      int		shell_style = STYLE_ECHO;
      int		check_spaces;
      static int	did_find_nul = FALSE;
      int		ampersent = FALSE;
  
      *num_file = 0;	/* default: no files found */
      *file = NULL;
--- 5059,5084 ----
      return OK;
  
  #else /* __EMX__ */
!     /*
!      * This is the non-OS/2 implementation (really Unix).
!      */
      int		j;
      char_u	*tempname;
      char_u	*command;
      FILE	*fd;
      char_u	*buffer;
! #define STYLE_ECHO	0	/* use "echo", the default */
! #define STYLE_GLOB	1	/* use "glob", for csh */
! #define STYLE_VIMGLOB	2	/* use "vimglob", for Posix sh */
! #define STYLE_PRINT	3	/* use "print -N", for zsh */
! #define STYLE_BT	4	/* `cmd` expansion, execute the pattern
! 				 * directly */
      int		shell_style = STYLE_ECHO;
      int		check_spaces;
      static int	did_find_nul = FALSE;
      int		ampersent = FALSE;
+ 		/* vimglob() function to define for Posix shell */
+     static char *sh_vimglob_func = "vimglob() { while [ $# -ge 1 ]; do echo -n \"$1\"; echo; shift; done }; vimglob >";
  
      *num_file = 0;	/* default: no files found */
      *file = NULL;
***************
*** 5107,5115 ****
  
      /*
       * Let the shell expand the patterns and write the result into the temp
!      * file.  if expanding `cmd` execute it directly.
!      * If we use csh, glob will work better than echo.
!      * If we use zsh, print -N will work better than glob.
       */
      if (num_pat == 1 && *pat[0] == '`'
  	    && (len = STRLEN(pat[0])) > 2
--- 5116,5132 ----
  
      /*
       * Let the shell expand the patterns and write the result into the temp
!      * file.
!      * STYLE_BT:	NL separated
!      *	    If expanding `cmd` execute it directly.
!      * STYLE_GLOB:	NUL separated
!      *	    If we use *csh, "glob" will work better than "echo".
!      * STYLE_PRINT:	NL or NUL separated
!      *	    If we use *zsh, "print -N" will work better than "glob".
!      * STYLE_VIMGLOB:	NL separated
!      *	    If we use *sh*, we define "vimglob()".
!      * STYLE_ECHO:	space separated.
!      *	    A shell we don't know, stay safe and use "echo".
       */
      if (num_pat == 1 && *pat[0] == '`'
  	    && (len = STRLEN(pat[0])) > 2
***************
*** 5122,5130 ****
  	else if (STRCMP(p_sh + len - 3, "zsh") == 0)
  	    shell_style = STYLE_PRINT;
      }
! 
!     /* "unset nonomatch; print -N >" plus two is 29 */
      len = STRLEN(tempname) + 29;
      for (i = 0; i < num_pat; ++i)
      {
  	/* Count the length of the patterns in the same way as they are put in
--- 5139,5155 ----
  	else if (STRCMP(p_sh + len - 3, "zsh") == 0)
  	    shell_style = STYLE_PRINT;
      }
!     if (shell_style == STYLE_ECHO && strstr((char *)gettail(p_sh),
! 								"sh") != NULL)
! 	shell_style = STYLE_VIMGLOB;
! 
!     /* Compute the length of the command.  We need 2 extra bytes: for the
!      * optional '&' and for the NUL.
!      * Worst case: "unset nonomatch; print -N >" plus two is 29 */
      len = STRLEN(tempname) + 29;
+     if (shell_style == STYLE_VIMGLOB)
+ 	len += STRLEN(sh_vimglob_func);
+ 
      for (i = 0; i < num_pat; ++i)
      {
  	/* Count the length of the patterns in the same way as they are put in
***************
*** 5183,5192 ****
--- 5208,5221 ----
  	    STRCAT(command, "glob >");
  	else if (shell_style == STYLE_PRINT)
  	    STRCAT(command, "print -N >");
+ 	else if (shell_style == STYLE_VIMGLOB)
+ 	    STRCAT(command, sh_vimglob_func);
  	else
  	    STRCAT(command, "echo >");
      }
+ 
      STRCAT(command, tempname);
+ 
      if (shell_style != STYLE_BT)
  	for (i = 0; i < num_pat; ++i)
  	{
***************
*** 5232,5239 ****
      if (flags & EW_SILENT)
  	show_shell_mess = FALSE;
      if (ampersent)
! 	STRCAT(command, "&");		/* put the '&' back after the
! 					   redirection */
  
      /*
       * Using zsh -G: If a pattern has no matches, it is just deleted from
--- 5261,5267 ----
      if (flags & EW_SILENT)
  	show_shell_mess = FALSE;
      if (ampersent)
! 	STRCAT(command, "&");		/* put the '&' after the redirection */
  
      /*
       * Using zsh -G: If a pattern has no matches, it is just deleted from
***************
*** 5265,5271 ****
      show_shell_mess = TRUE;
      vim_free(command);
  
!     if (i)				/* mch_call_shell() failed */
      {
  	mch_remove(tempname);
  	vim_free(tempname);
--- 5293,5299 ----
      show_shell_mess = TRUE;
      vim_free(command);
  
!     if (i != 0)				/* mch_call_shell() failed */
      {
  	mch_remove(tempname);
  	vim_free(tempname);
***************
*** 5336,5342 ****
      }
      vim_free(tempname);
  
! #if defined(__CYGWIN__) || defined(__CYGWIN32__)
      /* Translate <CR><NL> into <NL>.  Caution, buffer may contain NUL. */
      p = buffer;
      for (i = 0; i < len; ++i)
--- 5364,5370 ----
      }
      vim_free(tempname);
  
! # if defined(__CYGWIN__) || defined(__CYGWIN32__)
      /* Translate <CR><NL> into <NL>.  Caution, buffer may contain NUL. */
      p = buffer;
      for (i = 0; i < len; ++i)
***************
*** 5359,5365 ****
  	}
      }
      /* file names are separated with NL */
!     else if (shell_style == STYLE_BT)
      {
  	buffer[len] = NUL;		/* make sure the buffer ends in NUL */
  	p = buffer;
--- 5387,5393 ----
  	}
      }
      /* file names are separated with NL */
!     else if (shell_style == STYLE_BT || shell_style == STYLE_VIMGLOB)
      {
  	buffer[len] = NUL;		/* make sure the buffer ends in NUL */
  	p = buffer;
***************
*** 5438,5444 ****
      {
  	(*file)[i] = p;
  	/* Space or NL separates */
! 	if (shell_style == STYLE_ECHO || shell_style == STYLE_BT)
  	{
  	    while (!(shell_style == STYLE_ECHO && *p == ' ')
  						   && *p != '\n' && *p != NUL)
--- 5466,5473 ----
      {
  	(*file)[i] = p;
  	/* Space or NL separates */
! 	if (shell_style == STYLE_ECHO || shell_style == STYLE_BT
! 					      || shell_style == STYLE_VIMGLOB)
  	{
  	    while (!(shell_style == STYLE_ECHO && *p == ' ')
  						   && *p != '\n' && *p != NUL)
*** ../vim-7.1.222/src/version.c	Sun Jan 13 13:30:34 2008
--- src/version.c	Sun Jan 13 13:45:04 2008
***************
*** 668,669 ****
--- 668,671 ----
  {   /* Add new patch number below this line */
+ /**/
+     223,
  /**/

-- 
User:       I'm having problems with my text editor.
Help desk:  Which editor are you using?
User:       I don't know, but it's version VI (pronounced: 6).
Help desk:  Oh, then you should upgrade to version VIM (pronounced: 994).

 /// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net   \\\
///        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
\\\        download, build and distribute -- http://www.A-A-P.org        ///
 \\\            help me help AIDS victims -- http://ICCF-Holland.org    ///
Commit	Line	Data
d57b4abe ER	1	To: vim-dev@vim.org
	2	Subject: Patch 7.1.223
	3	Fcc: outbox
	4	From: Bram Moolenaar <Bram@moolenaar.net>
	5	Mime-Version: 1.0
	6	Content-Type: text/plain; charset=ISO-8859-1
	7	Content-Transfer-Encoding: 8bit
	8	------------
	9
	10	Patch 7.1.223
	11	Problem: glob() doesn't work properly when 'shell' is "sh" or "bash" and
	12	the expanded name contains spaces, '~', single quotes and other
	13	special characters. (Adri Verhoef, Charles Campbell)
	14	Solution: For Posix shells define a vimglob() function to list the matches
	15	instead of using "echo" directly.
	16	Files: src/os_unix.c
	17
	18
	19	*** ../vim-7.1.222/src/os_unix.c Thu Jan 3 18:55:21 2008
	20	--- src/os_unix.c Sun Jan 13 13:52:53 2008
	21	***************
	22	* 4946,4951 **
	23	--- 4946,4954 ----
	24	char_u *p;
	25	int dir;
	26	#ifdef __EMX__
	27	+ /*
	28	+ * This is the OS/2 implementation.
	29	+ */
	30	# define EXPL_ALLOC_INC 16
	31	char_u **expl_files;
	32	size_t files_alloced, files_free;
	33	***************
	34	* 5056,5075 **
	35	return OK;
	36
	37	#else /* __EMX__ */
	38	!
	39	int j;
	40	char_u *tempname;
	41	char_u *command;
	42	FILE *fd;
	43	char_u *buffer;
	44	! #define STYLE_ECHO 0 /* use "echo" to expand */
	45	! #define STYLE_GLOB 1 /* use "glob" to expand, for csh */
	46	! #define STYLE_PRINT 2 /* use "print -N" to expand, for zsh */
	47	! #define STYLE_BT 3 /* `cmd` expansion, execute the pattern directly */
	48	int shell_style = STYLE_ECHO;
	49	int check_spaces;
	50	static int did_find_nul = FALSE;
	51	int ampersent = FALSE;
	52
	53	num_file = 0; / default: no files found */
	54	*file = NULL;
	55	--- 5059,5084 ----
	56	return OK;
	57
	58	#else /* __EMX__ */
	59	! /*
	60	! * This is the non-OS/2 implementation (really Unix).
	61	! */
	62	int j;
	63	char_u *tempname;
	64	char_u *command;
65	FILE *fd;
66	char_u *buffer;
67	! #define STYLE_ECHO 0 /* use "echo", the default */
68	! #define STYLE_GLOB 1 /* use "glob", for csh */
69	! #define STYLE_VIMGLOB 2 /* use "vimglob", for Posix sh */
70	! #define STYLE_PRINT 3 /* use "print -N", for zsh */
71	! #define STYLE_BT 4 /* `cmd` expansion, execute the pattern
72	! * directly */
73	int shell_style = STYLE_ECHO;
74	int check_spaces;
75	static int did_find_nul = FALSE;
76	int ampersent = FALSE;
77	+ /* vimglob() function to define for Posix shell */
78	+ static char *sh_vimglob_func = "vimglob() { while [ $# -ge 1 ]; do echo -n \"$1\"; echo; shift; done }; vimglob >";
79
80	num_file = 0; / default: no files found */
81	*file = NULL;
82	***************
83	* 5107,5115 **
84
85	/*
86	* Let the shell expand the patterns and write the result into the temp
87	! * file. if expanding `cmd` execute it directly.
88	! * If we use csh, glob will work better than echo.
89	! * If we use zsh, print -N will work better than glob.
90	*/
91	if (num_pat == 1 && *pat[0] == '`'
92	&& (len = STRLEN(pat[0])) > 2
93	--- 5116,5132 ----
94
95	/*
96	* Let the shell expand the patterns and write the result into the temp
97	! * file.
98	! * STYLE_BT: NL separated
99	! * If expanding `cmd` execute it directly.
100	! * STYLE_GLOB: NUL separated
101	! * If we use *csh, "glob" will work better than "echo".
102	! * STYLE_PRINT: NL or NUL separated
103	! * If we use *zsh, "print -N" will work better than "glob".
104	! * STYLE_VIMGLOB: NL separated
105	! * If we use sh, we define "vimglob()".
106	! * STYLE_ECHO: space separated.
107	! * A shell we don't know, stay safe and use "echo".
108	*/
109	if (num_pat == 1 && *pat[0] == '`'
110	&& (len = STRLEN(pat[0])) > 2
111	***************
112	* 5122,5130 **
113	else if (STRCMP(p_sh + len - 3, "zsh") == 0)
114	shell_style = STYLE_PRINT;
115	}
116	!
117	! /* "unset nonomatch; print -N >" plus two is 29 */
118	len = STRLEN(tempname) + 29;
119	for (i = 0; i < num_pat; ++i)
120	{
121	/* Count the length of the patterns in the same way as they are put in
122	--- 5139,5155 ----
123	else if (STRCMP(p_sh + len - 3, "zsh") == 0)
124	shell_style = STYLE_PRINT;
125	}
126	! if (shell_style == STYLE_ECHO && strstr((char *)gettail(p_sh),
127	! "sh") != NULL)
128	! shell_style = STYLE_VIMGLOB;
129	!
130	! /* Compute the length of the command. We need 2 extra bytes: for the
131	! * optional '&' and for the NUL.
132	! * Worst case: "unset nonomatch; print -N >" plus two is 29 */
133	len = STRLEN(tempname) + 29;
134	+ if (shell_style == STYLE_VIMGLOB)
135	+ len += STRLEN(sh_vimglob_func);
136	+
137	for (i = 0; i < num_pat; ++i)
138	{
139	/* Count the length of the patterns in the same way as they are put in
140	***************
141	* 5183,5192 **
142	--- 5208,5221 ----
143	STRCAT(command, "glob >");
144	else if (shell_style == STYLE_PRINT)
145	STRCAT(command, "print -N >");
146	+ else if (shell_style == STYLE_VIMGLOB)
147	+ STRCAT(command, sh_vimglob_func);
148	else
149	STRCAT(command, "echo >");
150	}
151	+
152	STRCAT(command, tempname);
153	+
154	if (shell_style != STYLE_BT)
155	for (i = 0; i < num_pat; ++i)
156	{
157	***************
158	* 5232,5239 **
159	if (flags & EW_SILENT)
160	show_shell_mess = FALSE;
161	if (ampersent)
162	! STRCAT(command, "&"); /* put the '&' back after the
163	! redirection */
164
165	/*
166	* Using zsh -G: If a pattern has no matches, it is just deleted from
167	--- 5261,5267 ----
168	if (flags & EW_SILENT)
169	show_shell_mess = FALSE;
170	if (ampersent)
171	! STRCAT(command, "&"); /* put the '&' after the redirection */
172
173	/*
174	* Using zsh -G: If a pattern has no matches, it is just deleted from
175	***************
176	* 5265,5271 **
177	show_shell_mess = TRUE;
178	vim_free(command);
179
180	! if (i) /* mch_call_shell() failed */
181	{
182	mch_remove(tempname);
183	vim_free(tempname);
184	--- 5293,5299 ----
185	show_shell_mess = TRUE;
186	vim_free(command);
187
188	! if (i != 0) /* mch_call_shell() failed */
189	{
190	mch_remove(tempname);
191	vim_free(tempname);
192	***************
193	* 5336,5342 **
194	}
195	vim_free(tempname);
196
197	! #if defined(__CYGWIN__) \|\| defined(__CYGWIN32__)
198	/* Translate <CR><NL> into <NL>. Caution, buffer may contain NUL. */
199	p = buffer;
200	for (i = 0; i < len; ++i)
201	--- 5364,5370 ----
202	}
203	vim_free(tempname);
204
205	! # if defined(__CYGWIN__) \|\| defined(__CYGWIN32__)
206	/* Translate <CR><NL> into <NL>. Caution, buffer may contain NUL. */
207	p = buffer;
208	for (i = 0; i < len; ++i)
209	***************
210	* 5359,5365 **
211	}
212	}
213	/* file names are separated with NL */
214	! else if (shell_style == STYLE_BT)
215	{
216	buffer[len] = NUL; /* make sure the buffer ends in NUL */
217	p = buffer;
218	--- 5387,5393 ----
219	}
220	}
221	/* file names are separated with NL */
222	! else if (shell_style == STYLE_BT \|\| shell_style == STYLE_VIMGLOB)
223	{
224	buffer[len] = NUL; /* make sure the buffer ends in NUL */
225	p = buffer;
226	***************
227	* 5438,5444 **
228	{
229	(*file)[i] = p;
230	/* Space or NL separates */
231	! if (shell_style == STYLE_ECHO \|\| shell_style == STYLE_BT)
232	{
233	while (!(shell_style == STYLE_ECHO && *p == ' ')
234	&& p != '\n' && p != NUL)
235	--- 5466,5473 ----
236	{
237	(*file)[i] = p;
238	/* Space or NL separates */
239	! if (shell_style == STYLE_ECHO \|\| shell_style == STYLE_BT
240	! \|\| shell_style == STYLE_VIMGLOB)
241	{
242	while (!(shell_style == STYLE_ECHO && *p == ' ')
243	&& p != '\n' && p != NUL)
244	*** ../vim-7.1.222/src/version.c Sun Jan 13 13:30:34 2008
245	--- src/version.c Sun Jan 13 13:45:04 2008
246	***************
247	* 668,669 **
248	--- 668,671 ----
249	{ /* Add new patch number below this line */
250	+ /**/
251	+ 223,
252	/**/
253
254	--
255	User: I'm having problems with my text editor.
256	Help desk: Which editor are you using?
257	User: I don't know, but it's version VI (pronounced: 6).
258	Help desk: Oh, then you should upgrade to version VIM (pronounced: 994).
259
260	/// Bram Moolenaar -- Bram@Moolenaar.net -- http://www.Moolenaar.net \\\
261	/// sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ \\\
262	\\\ download, build and distribute -- http://www.A-A-P.org ///
263	\\\ help me help AIDS victims -- http://ICCF-Holland.org ///