[gnome-online-accounts/rhel-7.1: 5/34] kerberos: don't free alarm in set_alarm, free in callers
- From: Debarshi Ray <debarshir src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-online-accounts/rhel-7.1: 5/34] kerberos: don't free alarm in set_alarm, free in callers
- Date: Mon, 4 Aug 2014 13:42:21 +0000 (UTC)
commit 460d6283be72cd21958d3a7b44868b95a474f38a
Author: Michael Cronenworth <mike cchtml com>
Date: Mon Mar 17 09:33:59 2014 -0400
kerberos: don't free alarm in set_alarm, free in callers
It's a little unexpected that set_alarm "eats" the alarm passed in.
This commit makes it the caller's responsibility to free the alarm
to more closely match typical practice.
https://bugzilla.gnome.org/show_bug.cgi?id=726353
src/goaidentity/goakerberosidentity.c | 4 +++-
1 files changed, 3 insertions(+), 1 deletions(-)
---
diff --git a/src/goaidentity/goakerberosidentity.c b/src/goaidentity/goakerberosidentity.c
index 048f36b..2c1cdf7 100644
--- a/src/goaidentity/goakerberosidentity.c
+++ b/src/goaidentity/goakerberosidentity.c
@@ -787,7 +787,6 @@ set_alarm (GoaKerberosIdentity *self,
new_cancellable = g_cancellable_new ();
goa_alarm_set_time (alarm, alarm_time, new_cancellable);
- g_date_time_unref (alarm_time);
g_clear_object (cancellable);
*cancellable = new_cancellable;
@@ -882,6 +881,9 @@ reset_alarms (GoaKerberosIdentity *self)
self->priv->expiration_alarm,
expiration_time, &self->priv->expiration_alarm_cancellable);
+ g_date_time_unref (renewal_time);
+ g_date_time_unref (expiring_time);
+ g_date_time_unref (expiration_time);
connect_alarm_signals (self);
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]