On 07/ 9/12 06:02 PM, Daniel Stone wrote:
> Sync is designed to let you add system counters before the extension has
> been initialised, which means the system counter list may well be full
> of bees. Make sure it's initialised before we add to it, to avoid the
> risk of fatal injury.
>
> Signed-off-by: Daniel Stone <[email protected]>
> ---
> Xext/sync.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/Xext/sync.c b/Xext/sync.c
> index 91968e4..8c3b1a3 100644
> --- a/Xext/sync.c
> +++ b/Xext/sync.c
> @@ -948,6 +948,7 @@ SyncCreateSystemCounter(const char *name,
> if (RTCounter == 0) {
> return NULL;
> }
> + xorg_list_init(&SysCounterList);
> }
>
> pCounter = SyncCreateCounter(NULL, FakeClientID(0), initial);
Should the xorg_list_init in SyncExtensionInit be moved inside its
if (RTCounter == 0) check then, so it doesn't reset the list back to
empty after you've done this?
--
-Alan Coopersmith- [email protected]
Oracle Solaris Engineering - http://blogs.oracle.com/alanc
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
