openssh bugs - Aug 2016 - [Bug 2604] New: Remove orphaned do_pam_set_tty() or make use of it

https://bugzilla.mindrot.org/show_bug.cgi?id=2604

            Bug ID: 2604
           Summary: Remove orphaned do_pam_set_tty() or make use of it
           Product: Portable OpenSSH
           Version: 7.3p1
          Hardware: Other
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: P5
         Component: PAM support
          Assignee: unassigned-bugs at mindrot.org
          Reporter: jjelen at redhat.com

today I ran into problems with PAM and bumped into the function
do_pam_set_tty(), which is defined, but never used, if I see right:

$ git grep do_pam_set_tty
auth-pam.c:do_pam_set_tty(const char *tty)
auth-pam.h:void do_pam_set_tty(const char *);

There is basically duplicate code in sshpam_init(). In the git, last
usage of this function was removed in 2008 [1]. I don't think it makes
sense to keep it.


[1] https://github.com/openssh/openssh-portable/commit/52358d6d

-- 
You are receiving this mail because:
You are watching the assignee of the bug.

https://bugzilla.mindrot.org/show_bug.cgi?id=2604

Darren Tucker <dtucker at zip.com.au> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
                 CC|                            |dtucker at zip.com.au
         Resolution|---                         |FIXED
             Blocks|                            |2594

--- Comment #1 from Darren Tucker <dtucker at zip.com.au> ---
Done, thanks.


Referenced Bugs:

https://bugzilla.mindrot.org/show_bug.cgi?id=2594
[Bug 2594] Tracking bug for OpenSSH 7.4 release
-- 
You are receiving this mail because:
You are watching the assignee of the bug.
You are watching someone on the CC list of the bug.

https://bugzilla.mindrot.org/show_bug.cgi?id=2604

Damien Miller <djm at mindrot.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #2 from Damien Miller <djm at mindrot.org> ---
closing resolved bugs as of 8.6p1 release

-- 
You are receiving this mail because:
You are watching the assignee of the bug.
You are watching someone on the CC list of the bug.