[gnome-shell/gnome-3-38] calendar-server: Fix double-free detection abort
- From: Florian Müllner <fmuellner src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-shell/gnome-3-38] calendar-server: Fix double-free detection abort
- Date: Thu, 20 May 2021 14:13:13 +0000 (UTC)
commit f212512c0e877222837c6816bbef24369a2edc86
Author: Gustavo Noronha Silva <gustavo noronha dev br>
Date: Tue May 18 15:28:43 2021 -0300
calendar-server: Fix double-free detection abort
app_notify_events_added uses an intermediate builder to construct an
array that is then added to the main variant using g_variant_builder_add
which should clear the intermediate, but doesn't due to the way it is
passed: by value, rather than as a pointer.
This was debugged with the help of Eduardo Habkost, who believes it
works on x86 due to big structs being passed as pointers.
Fixed: https://gitlab.gnome.org/GNOME/gnome-shell/-/issues/3440
Part-of: <https://gitlab.gnome.org/GNOME/gnome-shell/-/merge_requests/1848>
(cherry picked from commit 404ca91941226faaf2479609f0109244e66d4bcd)
src/calendar-server/gnome-shell-calendar-server.c | 3 +--
1 file changed, 1 insertion(+), 2 deletions(-)
---
diff --git a/src/calendar-server/gnome-shell-calendar-server.c
b/src/calendar-server/gnome-shell-calendar-server.c
index 56cee994d8..5758f0f794 100644
--- a/src/calendar-server/gnome-shell-calendar-server.c
+++ b/src/calendar-server/gnome-shell-calendar-server.c
@@ -405,8 +405,7 @@ app_notify_events_added (App *app)
(gboolean) appt->is_all_day,
(gint64) start_time,
(gint64) end_time,
- extras_builder);
- g_variant_builder_clear (&extras_builder);
+ &extras_builder);
}
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]