Merge pull request #401 from kdmsnr/fix-bug-path-with-space

Escape pathname to prevent epubmaker from making files in different path
kmuto · Mar 1, 2015 · ba64f73 · ba64f73
2 parents 34ddcba + 5738615
commit ba64f73
Showing 1 changed file with 3 additions and 2 deletions.
diff --git a/lib/epubmaker/epubcommon.rb b/lib/epubmaker/epubcommon.rb
@@ -12,6 +12,7 @@
 require 'epubmaker/producer'
 require 'review/i18n'
 require 'cgi'
+require 'shellwords'
 
 module EPUBMaker
 
@@ -411,8 +412,8 @@ def produce_write_common(basedir, tmpdir)
     end
 
     def export_zip(tmpdir, epubfile)
-      Dir.chdir(tmpdir) {|d| `#{@producer.params["epubmaker"]["zip_stage1"]} #{epubfile} mimetype` }
-      Dir.chdir(tmpdir) {|d| `#{@producer.params["epubmaker"]["zip_stage2"]} #{epubfile} META-INF OEBPS #{@producer.params["epubmaker"]["zip_addpath"]}` }
+      Dir.chdir(tmpdir) {|d| `#{@producer.params["epubmaker"]["zip_stage1"]} #{epubfile.shellescape} mimetype` }
+      Dir.chdir(tmpdir) {|d| `#{@producer.params["epubmaker"]["zip_stage2"]} #{epubfile.shellescape} META-INF OEBPS #{@producer.params["epubmaker"]["zip_addpath"]}` }
     end
 
     def legacy_cover_and_title_file(loadfile, writefile)