[Xfce4-commits] <xfce4-screenshooter:master> Fix a crash when the config file does not exist yet (bug #6460).

Jérôme Guelfucci noreply at xfce.org
Fri May 21 10:18:01 CEST 2010


Updating branch refs/heads/master
         to 0dee91880c1cdb78a42ec0f4710c7457442ec34b (commit)
       from b6cff1c598bdf969a99a0d85ddd196230d2573a7 (commit)

commit 0dee91880c1cdb78a42ec0f4710c7457442ec34b
Author: Jérôme Guelfucci <jeromeg at xfce.org>
Date:   Fri May 21 10:14:26 2010 +0200

    Fix a crash when the config file does not exist yet (bug #6460).
    
    Calling xfce_rc_close on a NULL rc file is not exactly a good idea.
    Patch by Thomas (tg42 at gmx.de).

 lib/screenshooter-utils.c |    6 +++---
 1 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/lib/screenshooter-utils.c b/lib/screenshooter-utils.c
index 9615c66..ec84657 100644
--- a/lib/screenshooter-utils.c
+++ b/lib/screenshooter-utils.c
@@ -94,11 +94,11 @@ screenshooter_read_rc_file (const gchar *file, ScreenshotData *sd)
           g_free (title);
           title =
             g_strdup (xfce_rc_read_entry (rc, "title", _("Screenshot")));
-        }
 
-      TRACE ("Close the rc file");
+          TRACE ("Close the rc file");
 
-      xfce_rc_close (rc);
+          xfce_rc_close (rc);
+        }
     }
 
   /* And set the sd values */



More information about the Xfce4-commits mailing list