strlen(3) returns size_t, which is not strictly equivalent to int an 64-bit

archs, hence printf(3) warning.  Fix this.

Noticed by:	tinderbox
Approved by:	rse

1 files changed, 4 insertions, 3 deletions

diff --git a/sbin/dump/optr.c b/sbin/dump/optr.c
index 0d70b9e..4bdc172 100644
--- a/sbin/dump/optr.c
+++ b/sbin/dump/optr.c
@@ -209,14 +209,15 @@ timeest(void)
 		mins = (deltat % 3600) / 60;
 
 		tdone_str = ctime(&tdone);
+		tdone_str[(strlen(tdone_str) - 1)] = '\0';
 		setproctitle(
-		    "%s: pass %d: %3.2f%% done, finished in %d:%02d at %.*s",
-		    disk, passno, percent, hours, mins, strlen(tdone_str) - 1, tdone_str);
+		    "%s: pass %d: %3.2f%% done, finished in %d:%02d at %s",
+		    disk, passno, percent, hours, mins, tdone_str);
 		if (tnow >= tschedule) {
 			tschedule = tnow + 300;
 			if (blockswritten < 500)
 				return;
-			msg("%3.2f%% done, finished in %d:%02d at %s", percent,
+			msg("%3.2f%% done, finished in %d:%02d at %s\n", percent,
 			    hours, mins, tdone_str);
 		}
 	}