On Fri, Sep 14, 2012 at 08:59, Antoine Jacoutot wrote: > > Anyone? >> + /* lock the account */ >> + if (strncmp(pwp->pw_shell+strlen(pwp->pw_shell) - 1, > acctlock_str+strlen(acctlock_str) - 1, sizeof(acctlock_str) - 1) != 0) {
This looks like a horifically complicated way to check if the last character is -. Can we simplify some?