From ad34400ba150b159c6893b44f817646419ea4e8e Mon Sep 17 00:00:00 2001 From: Richard Yao Date: Fri, 16 Sep 2022 17:02:54 -0400 Subject: [PATCH] Fix null pointer dereferences in PAM Coverity caught these. Reviewed-by: Brian Behlendorf Signed-off-by: Richard Yao Closes #13889 --- contrib/pam_zfs_key/pam_zfs_key.c | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/contrib/pam_zfs_key/pam_zfs_key.c b/contrib/pam_zfs_key/pam_zfs_key.c index 6f95d468074f..c1001e6b81c2 100644 --- a/contrib/pam_zfs_key/pam_zfs_key.c +++ b/contrib/pam_zfs_key/pam_zfs_key.c @@ -531,7 +531,6 @@ zfs_key_config_get_dataset(zfs_key_config_t *config) if (zhp == NULL) { pam_syslog(NULL, LOG_ERR, "dataset %s not found", config->homes_prefix); - zfs_close(zhp); return (NULL); } @@ -543,6 +542,10 @@ zfs_key_config_get_dataset(zfs_key_config_t *config) return (dsname); } + if (config->homes_prefix == NULL) { + return (NULL); + } + size_t len = ZFS_MAX_DATASET_NAME_LEN; size_t total_len = strlen(config->homes_prefix) + 1 + strlen(config->username);