Re: [Linuxwacom-devel] [PATCH 3/4] Keep an enabled flag in the tool to avoid racy startup.

2011-03-11 Thread Ping Cheng
On Thu, Mar 10, 2011 at 7:45 PM, Peter Hutterer peter.hutte...@who-t.netwrote: On Thu, Mar 10, 2011 at 05:51:34PM -0800, Ping Cheng wrote: On Wed, Mar 2, 2011 at 10:06 PM, Peter Hutterer peter.hutte...@who-t.netwrote: There is a small time window where a device may try to send an event

Re: [Linuxwacom-devel] [PATCH 3/4] Keep an enabled flag in the tool to avoid racy startup.

2011-03-10 Thread Ping Cheng
On Wed, Mar 2, 2011 at 10:06 PM, Peter Hutterer peter.hutte...@who-t.netwrote: There is a small time window where a device may try to send an event even though it is not fully setup to send events yet, causing a server crash. This window opens when the tool is added to the list of devices in

Re: [Linuxwacom-devel] [PATCH 3/4] Keep an enabled flag in the tool to avoid racy startup.

2011-03-10 Thread Peter Hutterer
On Thu, Mar 10, 2011 at 05:51:34PM -0800, Ping Cheng wrote: On Wed, Mar 2, 2011 at 10:06 PM, Peter Hutterer peter.hutte...@who-t.netwrote: There is a small time window where a device may try to send an event even though it is not fully setup to send events yet, causing a server crash.

Re: [Linuxwacom-devel] [PATCH 3/4] Keep an enabled flag in the tool to avoid racy startup.

2011-03-03 Thread Ping Cheng
On Thu, Mar 3, 2011 at 2:35 AM, Peter Hutterer peter.hutte...@who-t.netwrote: On 3/03/11 17:07 , Ping Cheng wrote: On Wed, Mar 2, 2011 at 10:06 PM, Peter Hutterer peter.hutte...@who-t.net mailto:peter.hutte...@who-t.net wrote: There is a small time window where a device may try to send

[Linuxwacom-devel] [PATCH 3/4] Keep an enabled flag in the tool to avoid racy startup.

2011-03-02 Thread Peter Hutterer
There is a small time window where a device may try to send an event even though it is not fully setup to send events yet, causing a server crash. This window opens when the tool is added to the list of devices in wcmParseOptions() and closes with the server calling xf86ActivateDevice(). If an