[PATCH xserver] config/dbus: Initialize dbus fd to -1 so teardown doesn't use fd 0

Peter Hutterer peter.hutterer at who-t.net
Thu Sep 15 20:48:01 UTC 2016


On Wed, Sep 14, 2016 at 10:44:15PM -0700, Keith Packard wrote:
> The dbus teardown code is called when the server fatal errors even if
> that is before dbus has ever been initialized.  By statically
> initializing the value of bus_info.fd, we avoid calling RemoveNotifyFd
> on stdin.
> 
> Signed-off-by: Keith Packard <keithp at keithp.com>

Reviewed-by: Peter Hutterer <peter.hutterer at who-t.net>

Cheers,
   Peter
> ---
>  config/dbus-core.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/config/dbus-core.c b/config/dbus-core.c
> index 3c85ad7..6d9a3f9 100644
> --- a/config/dbus-core.c
> +++ b/config/dbus-core.c
> @@ -43,7 +43,7 @@ struct dbus_core_info {
>      OsTimerPtr timer;
>      struct dbus_core_hook *hooks;
>  };
> -static struct dbus_core_info bus_info;
> +static struct dbus_core_info bus_info = { .fd = -1 };
>  
>  static CARD32 reconnect_timer(OsTimerPtr timer, CARD32 time, void *arg);
>  
> -- 
> 2.8.1
> 


More information about the xorg-devel mailing list