[Spice-commits] tools/spicy.c

Victor Toso de Carvalho victortoso at kemper.freedesktop.org
Mon Mar 6 16:13:47 UTC 2017


 tools/spicy.c |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit ecb4de270587fba087073f6816a8105ea5bf4ee5
Author: Victor Toso <me at victortoso.com>
Date:   Mon Mar 6 11:09:41 2017 +0100

    spicy: use gboolean for GtkSettings properties
    
    As gboolean is 4 bytes long while bool is 1 byte, valgrind will complain
    about it.
    
    Invalid write of size 4
       at 0xAE9F4FB: value_lcopy_boolean (gvaluetypes.c:78)
       by 0xAE7CFD3: g_object_get_valist (gobject.c:2236)
       by 0xAE7D40B: g_object_get (gobject.c:2324)
       by 0x40758F: keyboard_grab_cb (spicy.c:566)
       by 0xAE753E4: g_closure_invoke (gclosure.c:804)
       by 0xAE87431: signal_emit_unlocked_R (gsignal.c:3635)
       by 0xAE9005E: g_signal_emit_valist (gsignal.c:3391)
       by 0xAE9043E: g_signal_emit (gsignal.c:3447)
       by 0x4E4A6FB: try_keyboard_grab (spice-widget.c:889)
       by 0x4E4CAB1: focus_in_event (spice-widget.c:1834)
       by 0x9325E70: _gtk_marshal_BOOLEAN__BOXEDv (gtkmarshalers.c:131)
       by 0xAE75561: _g_closure_invoke_va (gclosure.c:867)
    
    Signed-off-by: Victor Toso <victortoso at redhat.com>
    Acked-by: Pavel Grunt <pgrunt at redhat.com>

diff --git a/tools/spicy.c b/tools/spicy.c
index cd1677f..2772cfa 100644
--- a/tools/spicy.c
+++ b/tools/spicy.c
@@ -75,8 +75,8 @@ struct _SpiceWindow {
     gint             win_x;
     gint             win_y;
 #endif
-    bool             enable_accels_save;
-    bool             enable_mnemonics_save;
+    gboolean         enable_accels_save;
+    gboolean         enable_mnemonics_save;
 };
 
 struct _SpiceWindowClass


More information about the Spice-commits mailing list