]> git.ipfire.org Git - thirdparty/systemd.git/commitdiff
login/pam_systemd: use isatty_safe()
authorMike Yuan <me@yhndnzj.com>
Sun, 2 Mar 2025 15:00:05 +0000 (16:00 +0100)
committerMike Yuan <me@yhndnzj.com>
Sun, 2 Mar 2025 15:21:59 +0000 (16:21 +0100)
Follow-up for d8069b8add9a2290d7ed85012f8459fccfc632ed

src/login/pam_systemd.c

index 4c0f1a58b877e9080286d1d4532d87ab59695b52..74e1406bc7a9338d62fa7e99955625e79f9336da 100644 (file)
@@ -1604,7 +1604,7 @@ static int open_osc_context(pam_handle_t *handle, const char *session_type, User
          * high-level for us, as it suffixes the output with a newline, expecting a full blown text message
          * as prompt string, not just an ANSI sequence. Note that PAM's conv_misc() actually goes to stdout
          * anyway, hence let's do so here too, but only after careful validation. */
-        if (!isatty(STDOUT_FILENO))
+        if (!isatty_safe(STDOUT_FILENO))
                 return PAM_SUCCESS;
 
         /* Keep a reference to the TTY we are operating on, so that we can issue the OSC close sequence also
@@ -1688,7 +1688,7 @@ static int close_osc_context(pam_handle_t *handle) {
         fd = fd_move_above_stdio(fd);
 
         /* Safety check, let's verify this is a valid TTY we just opened */
-        if (!isatty(fd))
+        if (!isatty_safe(fd))
                 return PAM_SUCCESS;
 
         _cleanup_free_ char *osc = NULL;