summaryrefslogtreecommitdiff
path: root/daemon
diff options
context:
space:
mode:
authorAlessandro Bono <alessandro.bono369@gmail.com>2022-08-31 18:19:29 +0200
committerAlessandro Bono <alessandro.bono369@gmail.com>2022-10-29 12:39:39 +0200
commit9e038b65ffc57afb36435e87eceb2ae0c8a8d175 (patch)
tree76fdf9b8d960184e7a7ae41b6f39417880a5b34b /daemon
parent296f16036c90e8555f6c786adebd7ae195d8138e (diff)
downloadgdm-9e038b65ffc57afb36435e87eceb2ae0c8a8d175.tar.gz
gdm-display: Add missing guards in public functions
Diffstat (limited to 'daemon')
-rw-r--r--daemon/gdm-display.c13
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);