diff options
| author | Alessandro Bono <alessandro.bono369@gmail.com> | 2022-08-31 17:55:08 +0200 |
|---|---|---|
| committer | Alessandro Bono <alessandro.bono369@gmail.com> | 2022-10-29 12:39:26 +0200 |
| commit | 9b30c002f8a8293353dafcb4df413a65d0b91393 (patch) | |
| tree | 8095b07563a514677d7ce092e58bc12ade55fa6b /daemon | |
| parent | 88983214c31dc6a2063a4b588766591d541e6cca (diff) | |
| download | gdm-9b30c002f8a8293353dafcb4df413a65d0b91393.tar.gz | |
gdm-display-access-file: Add missing guards in public functions
Diffstat (limited to 'daemon')
| -rw-r--r-- | daemon/gdm-display-access-file.c | 16 |
1 files changed, 10 insertions, 6 deletions
diff --git a/daemon/gdm-display-access-file.c b/daemon/gdm-display-access-file.c index ed0a146d..d0dd5cc8 100644 --- a/daemon/gdm-display-access-file.c +++ b/daemon/gdm-display-access-file.c @@ -410,7 +410,7 @@ gdm_display_access_file_open (GdmDisplayAccessFile *file, { GError *create_error; - g_return_val_if_fail (file != NULL, FALSE); + g_return_val_if_fail (GDM_IS_DISPLAY_ACCESS_FILE (file), FALSE); g_return_val_if_fail (file->fp == NULL, FALSE); g_return_val_if_fail (file->path == NULL, FALSE); @@ -480,8 +480,9 @@ gdm_display_access_file_add_display (GdmDisplayAccessFile *file, GError *add_error; gboolean display_added; - g_return_val_if_fail (file != NULL, FALSE); + g_return_val_if_fail (GDM_IS_DISPLAY_ACCESS_FILE (file), FALSE); g_return_val_if_fail (file->path != NULL, FALSE); + g_return_val_if_fail (display != NULL, FALSE); g_return_val_if_fail (cookie != NULL, FALSE); add_error = NULL; @@ -519,8 +520,9 @@ gdm_display_access_file_add_display_with_cookie (GdmDisplayAccessFile *file, Xauth auth_entry; gboolean display_added; - g_return_val_if_fail (file != NULL, FALSE); + g_return_val_if_fail (GDM_IS_DISPLAY_ACCESS_FILE (file), FALSE); g_return_val_if_fail (file->path != NULL, FALSE); + g_return_val_if_fail (display != NULL, FALSE); g_return_val_if_fail (cookie != NULL, FALSE); _get_auth_info_for_display (file, display, @@ -574,7 +576,7 @@ gdm_display_access_file_close (GdmDisplayAccessFile *file) { char *auth_dir; - g_return_if_fail (file != NULL); + g_return_if_fail (GDM_IS_DISPLAY_ACCESS_FILE (file)); g_return_if_fail (file->fp != NULL); g_return_if_fail (file->path != NULL); @@ -608,7 +610,9 @@ gdm_display_access_file_close (GdmDisplayAccessFile *file) } char * -gdm_display_access_file_get_path (GdmDisplayAccessFile *access_file) +gdm_display_access_file_get_path (GdmDisplayAccessFile *file) { - return g_strdup (access_file->path); + g_return_val_if_fail (GDM_IS_DISPLAY_ACCESS_FILE (file), NULL); + + return g_strdup (file->path); } |