HomeFreeBSD

PAM: Fix unchecked return value from zfs_key_config_load()

Description

PAM: Fix unchecked return value from zfs_key_config_load()

9a49c6b782443ba6e627f2261c45f082ad843094 was intended to fix this issue,
but I had missed the case in pam_sm_open_session(). Clang's static
analyzer had not reported it and I forgot to look for other cases.

Interestingly, GCC gcc-12.1.1_p20220625's static analyzer had caught
this as multiple double-free bugs, since another failure after the
failure in zfs_key_config_load() will cause us to attempt to free the
memory that zfs_key_config_load() was supposed to allocate, but had
cleaned up upon failure.

Reviewed-by: Brian Behlendorf <behlendorf1@llnl.gov>
Signed-off-by: Richard Yao <richard.yao@alumni.stonybrook.edu>
Closes #13978

Details

Provenance
Richard Yao <richard.yao@alumni.stonybrook.edu>Authored on Oct 6 2022, 12:09 AM
GitHub <noreply@github.com>Committed on Oct 6 2022, 12:09 AM
Parents
rG4b629d04a5b8: Avoid calling rw_destroy() on uninitialized rwlock
Branches
Unknown
Tags
Unknown

Event Timeline

GitHub <noreply@github.com> committed rG2ba240f3583e: PAM: Fix unchecked return value from zfs_key_config_load() (authored by Richard Yao <richard.yao@alumni.stonybrook.edu>).Oct 6 2022, 12:09 AM