[Emacs-diffs] /srv/bzr/emacs/trunk r99406: * arc-mode.el (archive-zip-extract): Quote the argument passed to unzip (Bug#5475).

[Chong Yidong]

------------------------------------------------------------
revno: 99406
committer: Chong Yidong <address@hidden>
branch nick: trunk
timestamp: Thu 2010-01-28 15:06:36 -0500
message:
  * arc-mode.el (archive-zip-extract): Quote the argument passed to unzip 
(Bug#5475).
modified:
  lisp/ChangeLog
  lisp/arc-mode.el

=== modified file 'lisp/ChangeLog'
--- a/lisp/ChangeLog    2010-01-28 19:53:11 +0000
+++ b/lisp/ChangeLog    2010-01-28 20:06:36 +0000
@@ -1,3 +1,8 @@
+2010-01-28  Chong Yidong  <address@hidden>
+
+       * arc-mode.el (archive-zip-extract): Quote the argument passed to
+       unzip (Bug#5475).
+
 2010-01-28  Nil Geisweiller  <address@hidden>  (tiny change)
 
        * progmodes/flymake.el (flymake-allowed-file-name-masks)

=== modified file 'lisp/arc-mode.el'
--- a/lisp/arc-mode.el  2010-01-13 08:35:10 +0000
+++ b/lisp/arc-mode.el  2010-01-28 20:06:36 +0000
@@ -1784,7 +1784,10 @@
 (defun archive-zip-extract (archive name)
   (if (equal (car archive-zip-extract) "pkzip")
       (archive-*-extract archive name archive-zip-extract)
-    (archive-extract-by-stdout archive name archive-zip-extract)))
+    ;; unzip expands wildcards in NAME, so we need to quote it.
+    ;; FIXME: Does pkzip need similar treatment?
+    (archive-extract-by-stdout archive (shell-quote-argument name)
+                              archive-zip-extract)))
 
 (defun archive-zip-write-file-member (archive descr)
   (archive-*-write-file-member