Gracefully handle pixbuf error instead of crashing #248

Cimbali · Jul 4, 2022 · c76262d · c76262d
1 parent 3dd3363
commit c76262d
Showing 1 changed file with 6 additions and 3 deletions.
diff --git a/pympress/pointer.py b/pympress/pointer.py
@@ -114,10 +114,13 @@ def load_pointer(self, name):
         Args:
             name (`str`): Name of the pointer to load
         """
-        if name in ['red', 'green', 'blue']:
-            self.pointer = GdkPixbuf.Pixbuf.new_from_file(util.get_icon_path('pointer_' + name + '.png'))
-        else:
+        if name not in ['red', 'green', 'blue']:
             raise ValueError('Wrong color name')
+        path = util.get_icon_path('pointer_' + name + '.png')
+        try:
+            self.pointer = GdkPixbuf.Pixbuf.new_from_file(path)
+        except Exception:
+            logger.exception(_('Failed loading pixbuf for pointer "{}" from: {}'.format(name, path)))
 
 
     def change_pointercolor(self, action, target):