I have a workaround fpor 0.6.0, just in case folks can't install the fix right away.
It's just to add enough 0's in front of the PID in the PID file, until
there are 5 digits. It's not pretty, but it works.
See attached modified init.d/xrdp.sh script.
(See attached file: xrdp.sh)
Thanks again!!!!!!
Steven Saxe
LINUX SA
Raytheon
BLDG S77 3M-74
16800 E. CentreTech Parkway
Aurora, Colorado 80011
Desk: (720)216-8101
Cell (303)960-6308
From: Jay Sorg <jay.s...@gmail.com>
To: Steven M Saxe <steven.m.s...@raytheon.com>
Cc: "xrdp-devel@lists.sourceforge.net"
<xrdp-devel@lists.sourceforge.net>
Date: 06/19/2013 12:39 PM
Subject: Re: [Xrdp-devel] Possible error in xrdp-sesman
Thanks, I think this should fix it. I'll push it into master.
Jay
diff --git a/sesman/sesman.c b/sesman/sesman.c
index 4a294a5..72561ca 100644
--- a/sesman/sesman.c
+++ b/sesman/sesman.c
@@ -202,6 +202,7 @@ main(int argc, char **argv)
return 1;
}
+ g_memset(pid_s, 0, sizeof(pid_s));
error = g_file_read(fd, pid_s, 7);
if (-1 == error)
On Wed, Jun 19, 2013 at 9:56 AM, Steven M Saxe
<steven.m.s...@raytheon.com> wrote:
> All,
> Thanks for all your hard work. xrdp is a great product.
>
> We're using xrdp 0.6.0. We noticed sometime when we did
> a............................
> /usr/local/sbin/xrdp-sesman --kill
>
> We'd get the error......................
> error killing sesman: invalid pid
>
> We did an strace and found that when the PID has less that 5 digits,
> the PID used in the signal call was incorrect.
>
> I've attached an strace of it failing, and it working.
>
> I'm not a great c programmer, so I'm not sure what the fix would be.
>
> Anyway, I've attached some info.
>
> Thanks.(See attached file: info.tgz)
> Steven Saxe
> LINUX SA
> Raytheon
> BLDG S77 3M-74
> 16800 E. CentreTech Parkway
> Aurora, Colorado 80011
> Desk: (720)216-8101
> Cell (303)960-6308
>
>
>
------------------------------------------------------------------------------
> This SF.net email is sponsored by Windows:
>
> Build for Windows Store.
>
> http://p.sf.net/sfu/windows-dev2dev
> _______________________________________________
> xrdp-devel mailing list
> xrdp-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/xrdp-devel
>
<<inline: graycol.gif>>
xrdp.sh
Description: Binary data
------------------------------------------------------------------------------ This SF.net email is sponsored by Windows: Build for Windows Store. http://p.sf.net/sfu/windows-dev2dev
_______________________________________________ xrdp-devel mailing list xrdp-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/xrdp-devel
