Re: Xserver crash with cgoban

2022-11-06 Thread Matthieu Herrb
On Sat, Nov 05, 2022 at 08:55:46PM +0100, bau...@pestilenz.org wrote: Hi, Thanks for the report. You're right that this was overlooked when I wrote this code. I'd suggest the more complete (and paranoid) patch below: Index: os/client.c ==

Re: Xserver crash with cgoban

2022-11-06 Thread Todd C . Miller
On Sun, 06 Nov 2022 10:44:10 +, Matthieu Herrb wrote: > Thanks for the report. You're right that this was overlooked when I > wrote this code. > > I'd suggest the more complete (and paranoid) patch below: I don't think you can set client->clientIds->cmdname to a constant string such as "" sin

Re: Xserver crash with cgoban

2022-11-06 Thread Matthieu Herrb
On Sun, Nov 06, 2022 at 07:07:06AM -0700, Todd C. Miller wrote: > On Sun, 06 Nov 2022 10:44:10 +, Matthieu Herrb wrote: > > > Thanks for the report. You're right that this was overlooked when I > > wrote this code. > > > > I'd suggest the more complete (and paranoid) patch below: > > I don't