[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: Checking for getenv("SESSION_MANAGER") (Was: Bug#3008: dcopserver does not compile under Solaris
From:       Matthias Ettrich <ettrich () trolltech ! com>
Date:       2000-04-13 12:06:47
[Download RAW message or body]

Am Don, 13 Apr 2000 schrieb Waldo Bastian:
> On Wed, 12 Apr 2000, Stephan Kulow wrote:
> > > Hm. almost.
> > >
> > > putenv("SESSION_MANAGER=") clears the env variable but it doesn't
> > > undefine it. So getenv("SESSION_MANAGER") still returns a non-null
> > > pointer. Now dcopserver gives a Qt generated "Session management error".
> >
> > Hmm ;(
> > looking at the unsetenv implementation I'm afraid it's not portable at
> > all.
> > Our fake sets it to null, so could you please check not for non-null
> > pointer, but for non-null value?
> 
> The check for SESSION_MANAGER is in qapplication_x11.cpp. The trolls need to 
> change that. (CC'ed to qt-bugs@troll.no)

Actually, the bug seems to be in SmcOpenConnection (this is where
SESSION_MANAGER is read). I agree, Qt should work around that. An empty string
is for sure no valid network address.

We'll change it for the next release.

  Matthias

[prev in list] [next in list] [prev in thread] [next in thread]