hopefully fixing the rest of potential aborts, thx to a suggestion from fif

diff --git a/bin/booh b/bin/booh
index 0da74c021c40b290e7425056d4feba3a1842a263..ad765a15f12d6dcee4357d5e02823d5353f928ee 100755 (executable)
--- a/bin/booh
+++ b/bin/booh
@@ -2718,12 +2718,12 @@ def new_album
                 Thread.kill(src_nb_thread)
                 src_nb_thread = nil
             end
-            if File.directory?(from_utf8(src_nb_calculated_for)) && src_nb_calculated_for != '/'
-                if File.readable?(from_utf8(src_nb_calculated_for))
+            if File.directory?(from_utf8_safe(src_nb_calculated_for)) && src_nb_calculated_for != '/'
+                if File.readable?(from_utf8_safe(src_nb_calculated_for))
                     src_nb_thread = Thread.new {
                         gtk_thread_protect { src_nb.set_markup(utf8(_("<span size='small'><i>processing...</i></span>"))) }
                         total = { 'image' => 0, 'video' => 0, nil => 0 }
-                        `find '#{from_utf8(src_nb_calculated_for)}' -type d -follow`.each { |dir|
+                        `find '#{from_utf8_safe(src_nb_calculated_for)}' -type d -follow`.each { |dir|
                             if File.basename(dir) =~ /^\./
                                 next
                             else

diff --git a/lib/booh/booh-lib.rb b/lib/booh/booh-lib.rb
index 1dc435ef5cfc001fdb1bea58cd99a42a0663d6e8..e1a7e6fc1750f7570b30b75c39f736b52d1f234d 100644 (file)
--- a/lib/booh/booh-lib.rb
+++ b/lib/booh/booh-lib.rb
@@ -59,6 +59,14 @@ module Booh
         return Iconv::iconv($CURRENT_CHARSET, "UTF-8", string).to_s
     end
 
+    def from_utf8_safe(string)
+        begin
+            return Iconv::iconv($CURRENT_CHARSET, "UTF-8", string).to_s
+        rescue Iconv::IllegalSequence
+            return nil
+        end
+    end
+
     def make_dest_filename_old(orig_filename)
         #- we remove non alphanumeric characters but need to do that
         #- cleverly to not end up with two similar dest filenames. we won't