diff options
author | Florian Müllner <fmuellner@gnome.org> | 2013-11-29 15:43:45 +0000 |
---|---|---|
committer | Florian Müllner <fmuellner@gnome.org> | 2013-12-02 00:29:34 +0100 |
commit | f9e8c2147edea09039061d4c3ba0eb0ca69afab1 (patch) | |
tree | 7779515e39b8688389566b72b5817268608e72a8 | |
parent | 224e0ee7de1fc09adc9eed031f55569d219c567e (diff) | |
download | clutter-f9e8c2147edea09039061d4c3ba0eb0ca69afab1.tar.gz |
input-device: Guard against double free
Dispose() may be called more than once, so calling g_free directly
on the device name is unsafe. Instead, use g_clear_pointer() to
make sure we don't attempt to free the memory again.
https://bugzilla.gnome.org/show_bug.cgi?id=719563
-rw-r--r-- | clutter/clutter-input-device.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/clutter/clutter-input-device.c b/clutter/clutter-input-device.c index 07951e33f..848db18a3 100644 --- a/clutter/clutter-input-device.c +++ b/clutter/clutter-input-device.c @@ -81,7 +81,7 @@ clutter_input_device_dispose (GObject *gobject) { ClutterInputDevice *device = CLUTTER_INPUT_DEVICE (gobject); - g_free (device->device_name); + g_clear_pointer (&device->device_name, g_free); if (device->associated != NULL) { |