NX Window not closing when Logging out

[GoogleCodeExporter]

What steps will reproduce the problem?
1.connect to the neatx server using nx client(as usual)
2.try to log out with the top right button and choose "Log out"
3.Top and bottom Panel tabs "go away"

What is the expected output?
NX window should close itself and the session should be terminated

What do you see instead?
Panel tabs are removed but session is still alive. I have to click on the
top right X and choose the "terminate" button.
I don't if it has something to deal with, but I could see in
/var/log/syslog this:
gnome-session[7195]: GLib-GObject-CRITICAL: g_object_unref: assertion
`G_IS_OBJECT (object)' failed

What version of the product are you using? On what operating system?
Server is neatx rev. 24 on Ubuntu 9.04 desktop x86_64
Clients are win32/ubuntu 9.04 desktop (both 3.3.0-6)

Please provide any additional information below.
OK, that's not critical, but it could be a cause of trouble for users in
fullscreen mode 

Original issue reported on code.google.com by afeni...@gmail.com on 23 Jul 2009 at 1:49

[GoogleCodeExporter]

Ok, it looks like there's a bug in lib/daemon.py somewhere. When the session 
command 
exits (e.g. gnome-session), neatx should notice, and shutdown the rest of the 
session. 
However, __CheckExit() checks to make sure all i/o to the process is finished, 
and 
that check is failing. I'm not sure why yet.

Original comment by kormat on 23 Jul 2009 at 6:44

• Changed state: Accepted

[GoogleCodeExporter]

Found the issue, patch out for review to fix it.

Original comment by kormat on 26 Jul 2009 at 1:02

• Changed state: Started

[GoogleCodeExporter]

hmmmm... updated to rev29 and still the same problem! (not kidding...)
the neatx.data is still to "running" and window not closing.
Did, I made something wrong?

Original comment by afeni...@gmail.com on 27 Jul 2009 at 9:12

[GoogleCodeExporter]

The patch has only just been commited to svn, as rev30

Original comment by kormat on 27 Jul 2009 at 9:24

[GoogleCodeExporter]

patched to rev32, this is working as expected!
nice job ;)
thank you

Original comment by afeni...@gmail.com on 27 Jul 2009 at 9:37

[GoogleCodeExporter]

Great, thanks for testing (and for reporting the issue in the first place)! 
Marking as 
verified.

Original comment by kormat on 27 Jul 2009 at 11:31

• Changed state: Verified