"AndrC) S." <andresto...@gmail.com> wrote: >> Indeed, it seems v. 1.251 of session.c got it wrong. >> >> Can you see if this helps? >> >> /Alexander >> >> >> Index: session.c >> =================================================================== >> RCS file: /cvs/src/usr.bin/ssh/session.c,v >> retrieving revision 1.259 >> diff -u -p -r1.259 session.c >> --- session.c 24 Oct 2011 02:13:13 -0000 1.259 >> +++ session.c 14 Mar 2012 22:21:57 -0000 >> @@ -1108,7 +1108,7 @@ do_nologin(struct passwd *pw) >> char buf[1024], *nl, *def_nl = _PATH_NOLOGIN; >> struct stat sb; >> >> - if (login_getcapbool(lc, "ignorenologin", 0) && pw->pw_uid) >> + if (login_getcapbool(lc, "ignorenologin", 0) || pw->pw_uid == 0) >> return; >> nl = login_getcapstr(lc, "nologin", def_nl, def_nl); > >Hey Alexander, > >thanks a lot, this works for me now, no side-effects. :-) > >Regards >Andre
Philip Guenther also came up with an identical diff, which has been committed. Thanks for bringing this regression since 2010 to our attention! /Alexander
