From fdae41eb6368a4ba8c70cc42cb81dfc1f0d64317 Mon Sep 17 00:00:00 2001
From: nekral-guest <nekral-guest@5a98b0ae-9ef6-0310-add3-de5d479b70d7>
Date: Sun, 3 Feb 2008 21:30:47 +0000
Subject: [PATCH] Use a function to convert the dates from /etc/shadow to human
 readable dates.

---
 ChangeLog     |  5 +++++
 src/usermod.c | 60 +++++++++++++++++++++------------------------------
 2 files changed, 30 insertions(+), 35 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 7907ad78..48ee8d4a 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,8 @@
+2008-02-03  Nicolas François  <nicolas.francois@centraliens.net>
+
+	* src/usermod.c: Use a function to convert the dates from
+	/etc/shadow to human readable dates.
+
 2008-02-03  Nicolas François  <nicolas.francois@centraliens.net>
 
 	* src/login.c: Do not translate the fromhost variable. It is
diff --git a/src/usermod.c b/src/usermod.c
index d7ff6fdf..f1ed892d 100644
--- a/src/usermod.c
+++ b/src/usermod.c
@@ -133,6 +133,8 @@ static int sgr_locked = 0;
 
 
 /* local function prototypes */
+static void date_to_str (char *buf, size_t maxsize,
+                         long int date, const char *negativ);
 static int get_groups (char *);
 static void usage (void);
 static void new_pwent (struct passwd *);
@@ -159,6 +161,25 @@ static void update_files (void);
 static void move_mailbox (void);
 #endif
 
+static void date_to_str (char *buf, size_t maxsize,
+                         long int date, const char *negativ)
+{
+	struct tm *tp;
+
+	if ((negatif != NULL) && (date < 0)) {
+		strncpy (buf, negativ, maxsize);
+	} else {
+		time_t t = date;
+		tp = gmtime (&t);
+#ifdef HAVE_STRFTIME
+		strftime (buf, maxsize, "%Y-%m-%d", tp);
+#else
+		snprintf (buf, maxsize, "%04d-%02d-%02d",
+		          tp->tm_year + 1900, tp->tm_mon + 1, tp->tm_mday);
+#endif				/* HAVE_STRFTIME */
+	}
+	buf[maxsize - 1] = '\0';
+}
 /*
  * Had to move this over from useradd.c since we have groups named
  * "56k-family"... ergh.
@@ -450,43 +471,12 @@ static void new_spent (struct spwd *spent)
 		/* XXX - dates might be better than numbers of days.  --marekm */
 #ifdef WITH_AUDIT
 		if (audit_fd >= 0) {
-			time_t exp_t;
-			struct tm *exp_tm;
 			char new_exp[16], old_exp[16];
+			date_to_str (new_exp, sizeof(16),
+			             user_newexpire * DAY, "never");
+			date_to_str (old_exp, sizeof(16),
+			             user_expire * DAY, "never");
 
-			if (user_newexpire == -1)
-				new_exp[0] = '\0';
-			else {
-				exp_t = user_newexpire * DAY;
-				exp_tm = gmtime (&exp_t);
-#ifdef HAVE_STRFTIME
-				strftime (new_exp, sizeof (new_exp), "%Y-%m-%d",
-					  exp_tm);
-#else
-				memset (new_exp, 0, sizeof (new_exp));
-				snprintf (new_exp, sizeof (new_exp) - 1,
-					  "%04i-%02i-%02i",
-					  exp_tm->tm_year + 1900,
-					  exp_tm->tm_mon + 1, exp_tm->tm_mday);
-#endif
-			}
-
-			if (user_expire == -1)
-				old_exp[0] = '\0';
-			else {
-				exp_t = user_expire * DAY;
-				exp_tm = gmtime (&exp_t);
-#ifdef HAVE_STRFTIME
-				strftime (old_exp, sizeof (old_exp), "%Y-%m-%d",
-					  exp_tm);
-#else
-				memset (old_exp, 0, sizeof (old_exp));
-				snprintf (old_exp, sizeof (old_exp) - 1,
-					  "%04i-%02i-%02i",
-					  exp_tm->tm_year + 1900,
-					  exp_tm->tm_mon + 1, exp_tm->tm_mday);
-#endif
-			}
 			audit_logger (AUDIT_USER_CHAUTHTOK, Prog,
 				      "changing expiration date", user_newname,
 				      user_newid, 1);