All of lore.kernel.org
 help / color / mirror / Atom feed
* [PATCH] dbus: free previous sender string in _dbus_message_set_sender
@ 2016-03-14 14:49 Andrew Zaborowski
  2016-03-14 15:41 ` Denis Kenzior
  0 siblings, 1 reply; 2+ messages in thread
From: Andrew Zaborowski @ 2016-03-14 14:49 UTC (permalink / raw)
  To: ell

[-- Attachment #1: Type: text/plain, Size: 845 bytes --]

Allow _dbus_message_set_sender and _dbus_message_set_destination to be
called multiple times.
---
 ell/dbus-message.c | 4 ++++
 1 file changed, 4 insertions(+)

diff --git a/ell/dbus-message.c b/ell/dbus-message.c
index a123a8a..84d42d4 100644
--- a/ell/dbus-message.c
+++ b/ell/dbus-message.c
@@ -1390,6 +1390,8 @@ void _dbus_message_set_sender(struct l_dbus_message *message,
 	if (!_dbus_message_is_gvariant(message))
 		return;
 
+	l_free(message->sender);
+
 	message->sender = l_strdup(sender);
 	message->kdbus_sender = true;
 }
@@ -1400,6 +1402,8 @@ void _dbus_message_set_destination(struct l_dbus_message *message,
 	if (!_dbus_message_is_gvariant(message))
 		return;
 
+	l_free(message->destination);
+
 	message->destination = l_strdup(destination);
 	message->kdbus_destination = true;
 }
-- 
2.5.0


^ permalink raw reply related	[flat|nested] 2+ messages in thread

* Re: [PATCH] dbus: free previous sender string in _dbus_message_set_sender
  2016-03-14 14:49 [PATCH] dbus: free previous sender string in _dbus_message_set_sender Andrew Zaborowski
@ 2016-03-14 15:41 ` Denis Kenzior
  0 siblings, 0 replies; 2+ messages in thread
From: Denis Kenzior @ 2016-03-14 15:41 UTC (permalink / raw)
  To: ell

[-- Attachment #1: Type: text/plain, Size: 286 bytes --]

Hi Andrew,

On 03/14/2016 09:49 AM, Andrew Zaborowski wrote:
> Allow _dbus_message_set_sender and _dbus_message_set_destination to be
> called multiple times.
> ---
>   ell/dbus-message.c | 4 ++++
>   1 file changed, 4 insertions(+)
>

Applied, thanks.

Regards,
-Denis


^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2016-03-14 15:41 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-03-14 14:49 [PATCH] dbus: free previous sender string in _dbus_message_set_sender Andrew Zaborowski
2016-03-14 15:41 ` Denis Kenzior

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.