registry team mailing list archive

[Bug Watch Updater <74946@xxxxxxxxxxxxxxxxxx>]

Launchpad has imported 14 comments from the remote bug at
http://bugs.freedesktop.org/show_bug.cgi?id=8235.

If you reply to an imported comment from within Launchpad, your comment
will be sent to the remote bug automatically. Read more about
Launchpad's inter-bugtracker facilities at
https://help.launchpad.net/InterBugTracking.

------------------------------------------------------------------------
On 2006-09-12T04:55:30+00:00 Kimmo Hämäläinen wrote:

Here is a patch that works against dbus-glib-0.71. It adds missing checks to
dbus-gproxy.c because it assumes too much about received NameOwnerChanged signals.

Also one memory leak patch that came from you guys, I think...

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/0

------------------------------------------------------------------------
On 2006-09-12T04:56:00+00:00 Kimmo Hämäläinen wrote:

Created an attachment (id=6920)
proposed patch

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/1

------------------------------------------------------------------------
On 2006-10-12T00:26:41+00:00 Kimmo Hämäläinen wrote:

(From update of attachment 6920)
This turned out to be a buggy patch. I'll attach a new one when it's been
tested.

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/2

------------------------------------------------------------------------
On 2006-10-13T03:17:06+00:00 Kimmo Hämäläinen wrote:

Created an attachment (id=7401)
proposed patch

Here is the tested, fixed patch.

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/3

------------------------------------------------------------------------
On 2006-10-25T13:31:12+00:00 Rob Taylor wrote:

I'd like to look into the 'neater' option mentioned in the patch.

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/4

------------------------------------------------------------------------
On 2006-11-03T05:26:42+00:00 Kimmo Hämäläinen wrote:

It seems that unassociate_proxies() should also remove the proxy from the proxy
list before adding it to the data->destroyed list, because the caller destroys
the proxy.

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/5

------------------------------------------------------------------------
On 2006-11-16T02:41:55+00:00 Kimmo Hämäläinen wrote:

Created an attachment (id=7809)
proposed patch

Here is a modified patch that applies against 0.72.

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/6

------------------------------------------------------------------------
On 2006-11-28T09:21:33+00:00 Matthias Clasen wrote:

Don't you want to create a new info in the else part ?

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/7

------------------------------------------------------------------------
On 2006-11-28T09:25:54+00:00 Rstrode wrote:

Red Hat downstream report:
https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=216034

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/8

------------------------------------------------------------------------
On 2006-11-28T09:32:01+00:00 Matthias Clasen wrote:

The RH bug has my alternative patch

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/9

------------------------------------------------------------------------
On 2006-11-29T00:28:12+00:00 Kimmo Hämäläinen wrote:

(In reply to comment #7)
> Don't you want to create a new info in the else part ?

I'm not sure, because of the comment "Name owner changed or deleted" in the
code: if the info was not found, doesn't that mean that there was not changing
or deletion? I.e. If the info is not found, the name owner is not related to
GProxies. Or am I confused?

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/10

------------------------------------------------------------------------
On 2006-12-06T12:47:05+00:00 Otaylor-redhat wrote:

Just wanted to note here that this is regularly crashing gnome-session,
since it blanket listens to all NameOwnerChanged signals; I don't see
a GNOME bugzilla issue for this, but there are probably some dups there
if you looked hard enough.

I think that Kimmo is right and you don't want to insert a new info 
object, the message was simply not related to any GProxy objects that
the library was tracking.

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/11

------------------------------------------------------------------------
On 2006-12-08T05:35:19+00:00 Rob Taylor wrote:

Yep, that patch looks right to me, I'll prep a release with it in asap.

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/12

------------------------------------------------------------------------
On 2007-02-08T18:07:38+00:00 Rob Taylor wrote:

Fixed in git head, will be in dbus-glib 0.73

Reply at: https://bugs.launchpad.net/dbus/+bug/74946/comments/17


** Changed in: dbus
   Importance: Unknown => Medium

-- 
sometime something happens on dbus which makes apps using it crashing
https://bugs.launchpad.net/bugs/74946
You received this bug notification because you are a member of Registry
Administrators, which is the registrant for D-Bus.