diff options
author | Colin Watson <cjwatson@debian.org> | 2020-10-20 14:12:31 +0100 |
---|---|---|
committer | Colin Watson <cjwatson@debian.org> | 2020-10-20 14:12:31 +0100 |
commit | e371906fbbbbc11b0dced8fd4e0d258eb489d7c1 (patch) | |
tree | 4d0d8d2afd52572deb7910e29ff5a334b2bcf702 /auth-pam.c | |
parent | e429009cde648a41479cd1b60ce972760a2bdabc (diff) | |
parent | 3728919292c05983372954d27426f7d966813139 (diff) |
New upstream release (8.4p1)
Diffstat (limited to 'auth-pam.c')
-rw-r--r-- | auth-pam.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/auth-pam.c b/auth-pam.c index 5a3ba09b4..832382151 100644 --- a/auth-pam.c +++ b/auth-pam.c | |||
@@ -375,7 +375,11 @@ import_environments(struct sshbuf *b) | |||
375 | error("PAM: pam_putenv: %s", | 375 | error("PAM: pam_putenv: %s", |
376 | pam_strerror(sshpam_handle, r)); | 376 | pam_strerror(sshpam_handle, r)); |
377 | } | 377 | } |
378 | /* XXX leak env? */ | 378 | /* |
379 | * XXX this possibly leaks env because it is not documented | ||
380 | * what pam_putenv() does with it. Does it copy it? Does it | ||
381 | * take ownweship? We don't know, so it's safest just to leak. | ||
382 | */ | ||
379 | } | 383 | } |
380 | #endif | 384 | #endif |
381 | } | 385 | } |