Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pam_usb causes infinite loop in tty #115

Closed
TheScientist101 opened this issue Mar 28, 2022 · 15 comments · Fixed by #117
Closed

pam_usb causes infinite loop in tty #115

TheScientist101 opened this issue Mar 28, 2022 · 15 comments · Fixed by #117
Assignees
Labels
bug Something isn't working question Further information is requested

Comments

@TheScientist101
Copy link

Hello,
I installed pam_usb and it was working fine with lightdm, but when I tried to login to tty to fix something I found that pam_usb was causing an infinite loop. I could login when I removed the line, but any form of pam_usb was asking me to login again. I also use arch btw, if that might be the problem.
Thanks,
Urjith Mishra

@mcdope mcdope self-assigned this Mar 29, 2022
@mcdope
Copy link
Owner

mcdope commented Mar 29, 2022

Could you provide your authentication log for such a failed login? On ubuntu that would be /var/log/auth.log, not sure what it is on arch.

Also, how have you installed? From source, AUR, something else? Using current version I assume?

@mcdope mcdope added bug Something isn't working question Further information is requested labels Mar 29, 2022
@TheScientist101
Copy link
Author

I compiled it about 2 weeks ago, but I can't find any "releases". I don't know what the commit was. I will try to install the latest commit and try again.

@mcdope
Copy link
Owner

mcdope commented Apr 2, 2022

2 weeks ago should have been the latest commit i guess. the log is more important :)

@mcdope
Copy link
Owner

mcdope commented Apr 2, 2022

Hmmm, actually I can reproduce that on my main machine. But the log is showing a successful authentication... Interesting...

I can even briefly see the MOTD and a prompt, but session is closed again. If I disable pamusb this doesnt happen, but its really weird since the whole PAM stuff is done at that time...

@TheScientist101
Copy link
Author

TheScientist101 commented Apr 2, 2022

The log says this, let me know if I should hide something in the logs.

Apr 02 10:20:50 iusearchbtw login[141626]: pam_unix(login:session): session opened for user urjith(uid=1000) by LOGIN(uid=0)
Apr 02 10:20:50 iusearchbtw login[141626]: pam_env(login:session): deprecated reading of user environment enabled
Apr 02 10:20:50 iusearchbtw login[141626]: pam_unix(login:session): session closed for user urjith

I ran journalctl SYSLOG_IDENTIFIER=login

@mcdope
Copy link
Owner

mcdope commented Apr 30, 2022

Sorry for the silence, got some problems keeping up with my life 😅

(Workaround tip -->) What I can say so far is that it works if you disable deny_remote. (<-- Workaround tip)

So it's for sure related to local check, but haven't found the actual source.

@mcdope
Copy link
Owner

mcdope commented Apr 30, 2022

Please give https://www.dropbox.com/s/9hkro5bmhxtk9r6/libpam-usb_0.8.1_issue115_ttylogin_amd64.deb?dl=0 a try

This build circumvents the problem by whitelisting tty logins as local.

@TheScientist101
Copy link
Author

TheScientist101 commented May 1, 2022 via email

@mcdope
Copy link
Owner

mcdope commented May 2, 2022

How can I install that on Arch? Sorry for the beginner question.

On Apr 30, 2022, at 9:27 AM, McDope @.***> wrote:  Please give https://www.dropbox.com/s/9hkro5bmhxtk9r6/libpam-usb_0.8.1_issue115_ttylogin_amd64.deb?dl=0 a try This build circumvents the problem by whitelisting tty logins as local. — Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you authored the thread.

Sorry, my bad - forgot that you're on arch. You can built that version from branch issue-115-ttylogin

@mcdope
Copy link
Owner

mcdope commented May 7, 2022

If you don't say anything indicating otherwise I consider this fixed and will merge it tomorrow.

@TheScientist101
Copy link
Author

TheScientist101 commented May 7, 2022 via email

@TheScientist101
Copy link
Author

TheScientist101 commented May 7, 2022

The PR fixed my issue. Unless you have anything to add, should I close the issue?

@mcdope
Copy link
Owner

mcdope commented May 7, 2022 via email

@AlexOConnorHub
Copy link

I'll chime in and say that, when using the current release on arch, I had the same problem, and that the issue-115-ttylogin branch resolves the issue. (Just to say that the event is not isolated to just and that this solution works!)

mcdope added a commit that referenced this issue Aug 18, 2022
mcdope added a commit that referenced this issue Aug 18, 2022
mcdope added a commit that referenced this issue Aug 18, 2022
mcdope added a commit that referenced this issue Aug 20, 2022
…149)

* #148: Re-format Makefile

* #148: Add editorconfig and gitblameignorerevs

* #148: Move debug CFLAG to CFLAGS

* #148: Use tabs again for Makefile - doh!

* #148: Add new commit to ignorelist

* #148: [conf.c] Reformat

* #148: Update ignore-revs

* #148: Update .editorconfig

* #148: [device.c] Reformat

* #148: [local.c] Reformat

* #148: [log.c] Reformat

* #148: [pad.c] Reformat

* #148: [pam.c] Reformat

* #149: Update .git-blame-ignore-revs

* #148: [pamusb-check.c] Reformat

* #148: [volume.c] Reformat

* #148: [xpath.c] Reformat

* #148: Update .git-blame-ignore-revs

* #148: [pamusb-agent] Reformat

* #148: [pamusb-conf] Reformat

* #148: Update .git-blame-ignore-revs

* #148: if braces

* #148: Update .editorconfig

* #148: while braces

* #148: for braces

* [Makefile] Install config as .dist if already configured

* #115: [Packaging] [Fedora] Spec formatting

* #148: Update ignore file
@mcdope
Copy link
Owner

mcdope commented Oct 11, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants