diff options
author | Alessandro Bono <alessandro.bono369@gmail.com> | 2022-08-31 18:19:29 +0200 |
---|---|---|
committer | Alessandro Bono <alessandro.bono369@gmail.com> | 2022-10-29 12:39:39 +0200 |
commit | 9e038b65ffc57afb36435e87eceb2ae0c8a8d175 (patch) | |
tree | 76fdf9b8d960184e7a7ae41b6f39417880a5b34b /daemon | |
parent | 296f16036c90e8555f6c786adebd7ae195d8138e (diff) | |
download | gdm-9e038b65ffc57afb36435e87eceb2ae0c8a8d175.tar.gz |
gdm-display: Add missing guards in public functions
Diffstat (limited to 'daemon')
-rw-r--r-- | daemon/gdm-display.c | 13 |
1 files changed, 13 insertions, 0 deletions
diff --git a/daemon/gdm-display.c b/daemon/gdm-display.c index 42762fd8..dd617227 100644 --- a/daemon/gdm-display.c +++ b/daemon/gdm-display.c @@ -168,6 +168,8 @@ gdm_display_get_session_id (GdmDisplay *self) { GdmDisplayPrivate *priv; + g_return_val_if_fail (GDM_IS_DISPLAY (self), NULL); + priv = gdm_display_get_instance_private (self); return priv->session_id; } @@ -261,6 +263,8 @@ gdm_display_add_user_authorization (GdmDisplay *self, xcb_void_cookie_t cookies[3]; g_return_val_if_fail (GDM_IS_DISPLAY (self), FALSE); + g_return_val_if_fail (username != NULL, FALSE); + g_return_val_if_fail (filename != NULL, FALSE); priv = gdm_display_get_instance_private (self); @@ -340,6 +344,7 @@ gdm_display_remove_user_authorization (GdmDisplay *self, GdmDisplayPrivate *priv; g_return_val_if_fail (GDM_IS_DISPLAY (self), FALSE); + g_return_val_if_fail (username != NULL, FALSE); priv = gdm_display_get_instance_private (self); @@ -1470,6 +1475,8 @@ gdm_display_get_object_skeleton (GdmDisplay *self) { GdmDisplayPrivate *priv; + g_return_val_if_fail (GDM_IS_DISPLAY (self), NULL); + priv = gdm_display_get_instance_private (self); return priv->object_skeleton; } @@ -1695,6 +1702,8 @@ gdm_display_start_greeter_session (GdmDisplay *self) char *hostname; char *auth_file = NULL; + g_return_if_fail (GDM_IS_DISPLAY (self)); + priv = gdm_display_get_instance_private (self); g_return_if_fail (g_strcmp0 (priv->session_class, "greeter") == 0); @@ -1761,6 +1770,8 @@ gdm_display_stop_greeter_session (GdmDisplay *self) { GdmDisplayPrivate *priv; + g_return_if_fail (GDM_IS_DISPLAY (self)); + priv = gdm_display_get_instance_private (self); if (priv->launch_environment != NULL) { @@ -1876,6 +1887,8 @@ gdm_display_connect (GdmDisplay *self) xcb_auth_info_t *auth_info = NULL; gboolean ret; + g_return_val_if_fail (GDM_IS_DISPLAY (self), FALSE); + priv = gdm_display_get_instance_private (self); g_debug ("GdmDisplay: Server is ready - opening display %s", priv->x11_display_name); |