Module Name: src Committed By: christos Date: Tue Jan 3 19:02:55 UTC 2012
Modified Files: src/lib/libpam/modules/pam_chroot: pam_chroot.c src/lib/libpam/modules/pam_exec: pam_exec.c src/lib/libpam/modules/pam_ftpusers: pam_ftpusers.c src/lib/libpam/modules/pam_lastlog: pam_lastlog.c src/lib/libpam/modules/pam_login_access: login_access.c src/lib/libpam/modules/pam_ssh: pam_ssh.c Log Message: avoid using %m in format. To generate a diff of this commit: cvs rdiff -u -r1.4 -r1.5 src/lib/libpam/modules/pam_chroot/pam_chroot.c cvs rdiff -u -r1.5 -r1.6 src/lib/libpam/modules/pam_exec/pam_exec.c cvs rdiff -u -r1.5 -r1.6 src/lib/libpam/modules/pam_ftpusers/pam_ftpusers.c cvs rdiff -u -r1.13 -r1.14 src/lib/libpam/modules/pam_lastlog/pam_lastlog.c cvs rdiff -u -r1.5 -r1.6 \ src/lib/libpam/modules/pam_login_access/login_access.c cvs rdiff -u -r1.20 -r1.21 src/lib/libpam/modules/pam_ssh/pam_ssh.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/lib/libpam/modules/pam_chroot/pam_chroot.c diff -u src/lib/libpam/modules/pam_chroot/pam_chroot.c:1.4 src/lib/libpam/modules/pam_chroot/pam_chroot.c:1.5 --- src/lib/libpam/modules/pam_chroot/pam_chroot.c:1.4 Mon Apr 18 23:15:34 2005 +++ src/lib/libpam/modules/pam_chroot/pam_chroot.c Tue Jan 3 14:02:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: pam_chroot.c,v 1.4 2005/04/19 03:15:34 christos Exp $ */ +/* $NetBSD: pam_chroot.c,v 1.5 2012/01/03 19:02:54 christos Exp $ */ /*- * Copyright (c) 2003 Networks Associates Technology, Inc. @@ -38,7 +38,7 @@ #ifdef __FreeBSD__ __FBSDID("$FreeBSD: src/lib/libpam/modules/pam_chroot/pam_chroot.c,v 1.3 2003/04/30 00:40:24 des Exp $"); #else -__RCSID("$NetBSD: pam_chroot.c,v 1.4 2005/04/19 03:15:34 christos Exp $"); +__RCSID("$NetBSD: pam_chroot.c,v 1.5 2012/01/03 19:02:54 christos Exp $"); #endif #include <sys/param.h> @@ -46,6 +46,7 @@ __RCSID("$NetBSD: pam_chroot.c,v 1.4 200 #include <pwd.h> #include <stdio.h> #include <string.h> +#include <errno.h> #include <unistd.h> #define PAM_SM_SESSION @@ -96,11 +97,11 @@ pam_sm_open_session(pam_handle_t *pamh, openpam_log(PAM_LOG_DEBUG, "chrooting %s to %s", dir, user); if (chroot(dir) == -1) { - openpam_log(PAM_LOG_ERROR, "chroot(): %m"); + openpam_log(PAM_LOG_ERROR, "chroot(): %s", strerror(errno)); return (PAM_SESSION_ERR); } if (chdir(cwd) == -1) { - openpam_log(PAM_LOG_ERROR, "chdir(): %m"); + openpam_log(PAM_LOG_ERROR, "chdir(): %s", strerror(errno)); return (PAM_SESSION_ERR); } pam_setenv(pamh, "HOME", cwd, 1); Index: src/lib/libpam/modules/pam_exec/pam_exec.c diff -u src/lib/libpam/modules/pam_exec/pam_exec.c:1.5 src/lib/libpam/modules/pam_exec/pam_exec.c:1.6 --- src/lib/libpam/modules/pam_exec/pam_exec.c:1.5 Wed Feb 2 21:05:59 2011 +++ src/lib/libpam/modules/pam_exec/pam_exec.c Tue Jan 3 14:02:54 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: pam_exec.c,v 1.5 2011/02/03 02:05:59 christos Exp $ */ +/* $NetBSD: pam_exec.c,v 1.6 2012/01/03 19:02:54 christos Exp $ */ /*- * Copyright (c) 2001,2003 Networks Associates Technology, Inc. @@ -38,7 +38,7 @@ #ifdef __FreeBSD__ __FBSDID("$FreeBSD: src/lib/libpam/modules/pam_exec/pam_exec.c,v 1.4 2005/02/01 10:37:07 des Exp $"); #else -__RCSID("$NetBSD: pam_exec.c,v 1.5 2011/02/03 02:05:59 christos Exp $"); +__RCSID("$NetBSD: pam_exec.c,v 1.6 2012/01/03 19:02:54 christos Exp $"); #endif #include <sys/types.h> @@ -128,15 +128,15 @@ _pam_exec(pam_handle_t *pamh __unused, i } openpam_free_envlist(envlist); if (pid == -1) { - openpam_log(PAM_LOG_ERROR, "vfork(): %m"); + openpam_log(PAM_LOG_ERROR, "vfork(): %s", strerror(errno)); return (PAM_SYSTEM_ERR); } if (waitpid(pid, &status, 0) == -1) { - openpam_log(PAM_LOG_ERROR, "waitpid(): %m"); + openpam_log(PAM_LOG_ERROR, "waitpid(): %s", strerror(errno)); return (PAM_SYSTEM_ERR); } if (childerr != 0) { - openpam_log(PAM_LOG_ERROR, "execve(): %m"); + openpam_log(PAM_LOG_ERROR, "execve(): %s", strerror(errno)); return (PAM_SYSTEM_ERR); } if (WIFSIGNALED(status)) { Index: src/lib/libpam/modules/pam_ftpusers/pam_ftpusers.c diff -u src/lib/libpam/modules/pam_ftpusers/pam_ftpusers.c:1.5 src/lib/libpam/modules/pam_ftpusers/pam_ftpusers.c:1.6 --- src/lib/libpam/modules/pam_ftpusers/pam_ftpusers.c:1.5 Tue Apr 19 09:04:38 2005 +++ src/lib/libpam/modules/pam_ftpusers/pam_ftpusers.c Tue Jan 3 14:02:55 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: pam_ftpusers.c,v 1.5 2005/04/19 13:04:38 christos Exp $ */ +/* $NetBSD: pam_ftpusers.c,v 1.6 2012/01/03 19:02:55 christos Exp $ */ /*- * Copyright (c) 2001 Networks Associates Technology, Inc. @@ -38,7 +38,7 @@ #ifdef __FreeBSD__ __FBSDID("$FreeBSD: src/lib/libpam/modules/pam_ftpusers/pam_ftpusers.c,v 1.1 2002/05/08 00:30:10 des Exp $"); #else -__RCSID("$NetBSD: pam_ftpusers.c,v 1.5 2005/04/19 13:04:38 christos Exp $"); +__RCSID("$NetBSD: pam_ftpusers.c,v 1.6 2012/01/03 19:02:55 christos Exp $"); #endif #include <ctype.h> @@ -46,6 +46,7 @@ __RCSID("$NetBSD: pam_ftpusers.c,v 1.5 2 #include <paths.h> #include <pwd.h> #include <stdio.h> +#include <errno.h> #include <stdlib.h> #include <string.h> @@ -80,7 +81,7 @@ pam_sm_acct_mgmt(pam_handle_t *pamh, int found = 0; ulen = strlen(user); if ((f = fopen(_PATH_FTPUSERS, "r")) == NULL) { - PAM_LOG("%s: %m", _PATH_FTPUSERS); + PAM_LOG("%s: %s", _PATH_FTPUSERS, strerror(errno)); goto done; } while (!found && (line = fgetln(f, &len)) != NULL) { Index: src/lib/libpam/modules/pam_lastlog/pam_lastlog.c diff -u src/lib/libpam/modules/pam_lastlog/pam_lastlog.c:1.13 src/lib/libpam/modules/pam_lastlog/pam_lastlog.c:1.14 --- src/lib/libpam/modules/pam_lastlog/pam_lastlog.c:1.13 Sun Jan 25 23:01:14 2009 +++ src/lib/libpam/modules/pam_lastlog/pam_lastlog.c Tue Jan 3 14:02:55 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: pam_lastlog.c,v 1.13 2009/01/26 04:01:14 lukem Exp $ */ +/* $NetBSD: pam_lastlog.c,v 1.14 2012/01/03 19:02:55 christos Exp $ */ /*- * Copyright (c) 1980, 1987, 1988, 1991, 1993, 1994 @@ -47,7 +47,7 @@ #ifdef __FreeBSD__ __FBSDID("$FreeBSD: src/lib/libpam/modules/pam_lastlog/pam_lastlog.c,v 1.20 2004/01/26 19:28:37 des Exp $"); #else -__RCSID("$NetBSD: pam_lastlog.c,v 1.13 2009/01/26 04:01:14 lukem Exp $"); +__RCSID("$NetBSD: pam_lastlog.c,v 1.14 2012/01/03 19:02:55 christos Exp $"); #endif #include <sys/param.h> @@ -60,6 +60,7 @@ __RCSID("$NetBSD: pam_lastlog.c,v 1.13 2 #include <stdlib.h> #include <string.h> #include <syslog.h> +#include <errno.h> #include <time.h> #include <unistd.h> #include <stdarg.h> @@ -291,10 +292,10 @@ doutmpx(const char *username, const char (void)strncpy(utmpx.ut_id, tty, sizeof(utmpx.ut_id)); } if (pututxline(&utmpx) == NULL) - logit(LOG_NOTICE, "Cannot update utmpx %m"); + logit(LOG_NOTICE, "Cannot update utmpx: %s", strerror(errno)); endutxent(); if (updwtmpx(_PATH_WTMPX, &utmpx) != 0) - logit(LOG_NOTICE, "Cannot update wtmpx %m"); + logit(LOG_NOTICE, "Cannot update wtmpx: %s", strerror(errno)); } static void @@ -323,7 +324,7 @@ dolastlogx(pam_handle_t *pamh, int quiet (void)memset(&ll.ll_ss, 0, sizeof(ll.ll_ss)); if (updlastlogx(_PATH_LASTLOGX, pwd->pw_uid, &ll) != 0) - logit(LOG_NOTICE, "Cannot update lastlogx %m"); + logit(LOG_NOTICE, "Cannot update lastlogx: %s", strerror(errno)); PAM_LOG("Login recorded in %s", _PATH_LASTLOGX); } #endif @@ -352,7 +353,8 @@ dolastlog(pam_handle_t *pamh, int quiet, int fd; if ((fd = open(_PATH_LASTLOG, O_RDWR, 0)) == -1) { - logit(LOG_NOTICE, "Cannot open `%s' %m", _PATH_LASTLOG); + logit(LOG_NOTICE, "Cannot open `%s': %s", _PATH_LASTLOG, + strerror(errno)); return; } (void)lseek(fd, (off_t)(pwd->pw_uid * sizeof(ll)), SEEK_SET); Index: src/lib/libpam/modules/pam_login_access/login_access.c diff -u src/lib/libpam/modules/pam_login_access/login_access.c:1.5 src/lib/libpam/modules/pam_login_access/login_access.c:1.6 --- src/lib/libpam/modules/pam_login_access/login_access.c:1.5 Fri Nov 3 13:55:40 2006 +++ src/lib/libpam/modules/pam_login_access/login_access.c Tue Jan 3 14:02:55 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: login_access.c,v 1.5 2006/11/03 18:55:40 christos Exp $ */ +/* $NetBSD: login_access.c,v 1.6 2012/01/03 19:02:55 christos Exp $ */ /* * This module implements a simple but effective form of login access @@ -19,7 +19,7 @@ static char sccsid[] = "%Z% %M% %I% %E% #ifdef __FreeBSD__ __FBSDID("$FreeBSD: src/lib/libpam/modules/pam_login_access/login_access.c,v 1.12 2004/03/05 08:10:18 markm Exp $"); #else -__RCSID("$NetBSD: login_access.c,v 1.5 2006/11/03 18:55:40 christos Exp $"); +__RCSID("$NetBSD: login_access.c,v 1.6 2012/01/03 19:02:55 christos Exp $"); #endif #include <sys/types.h> @@ -122,7 +122,7 @@ login_access(const char *user, const cha } (void) fclose(fp); } else if (errno != ENOENT) { - logit(LOG_ERR, "cannot open %s: %m", _PATH_LOGACCESS); + logit(LOG_ERR, "cannot open %s: %s", _PATH_LOGACCESS, strerror(errno)); } return (match == 0 || (line[0] == '+')); } Index: src/lib/libpam/modules/pam_ssh/pam_ssh.c diff -u src/lib/libpam/modules/pam_ssh/pam_ssh.c:1.20 src/lib/libpam/modules/pam_ssh/pam_ssh.c:1.21 --- src/lib/libpam/modules/pam_ssh/pam_ssh.c:1.20 Fri Dec 16 12:37:14 2011 +++ src/lib/libpam/modules/pam_ssh/pam_ssh.c Tue Jan 3 14:02:55 2012 @@ -1,4 +1,4 @@ -/* $NetBSD: pam_ssh.c,v 1.20 2011/12/16 17:37:14 drochner Exp $ */ +/* $NetBSD: pam_ssh.c,v 1.21 2012/01/03 19:02:55 christos Exp $ */ /*- * Copyright (c) 2003 Networks Associates Technology, Inc. @@ -38,7 +38,7 @@ #ifdef __FreeBSD__ __FBSDID("$FreeBSD: src/lib/libpam/modules/pam_ssh/pam_ssh.c,v 1.40 2004/02/10 10:13:21 des Exp $"); #else -__RCSID("$NetBSD: pam_ssh.c,v 1.20 2011/12/16 17:37:14 drochner Exp $"); +__RCSID("$NetBSD: pam_ssh.c,v 1.21 2012/01/03 19:02:55 christos Exp $"); #endif #include <sys/param.h> @@ -306,19 +306,19 @@ pam_ssh_start_agent(pam_handle_t *pamh, #endif /* child: drop privs, close fds and start agent */ if (setgid(pwd->pw_gid) == -1) { - openpam_log(PAM_LOG_DEBUG, "%s: Cannot setgid %d (%m)", - __func__, (int)pwd->pw_gid); + openpam_log(PAM_LOG_DEBUG, "%s: Cannot setgid %d (%s)", + __func__, (int)pwd->pw_gid, strerror(errno)); goto done; } if (initgroups(pwd->pw_name, pwd->pw_gid) == -1) { openpam_log(PAM_LOG_DEBUG, - "%s: Cannot initgroups for %s (%m)", - __func__, pwd->pw_name); + "%s: Cannot initgroups for %s (%s)", + __func__, pwd->pw_name, strerror(errno)); goto done; } if (setuid(pwd->pw_uid) == -1) { - openpam_log(PAM_LOG_DEBUG, "%s: Cannot setuid %d (%m)", - __func__, (int)pwd->pw_uid); + openpam_log(PAM_LOG_DEBUG, "%s: Cannot setuid %d (%s)", + __func__, (int)pwd->pw_uid, strerror(errno)); goto done; } (void)close(STDIN_FILENO);