From da975fa809f98afb103bd1f55a59bbdaae5b727a Mon Sep 17 00:00:00 2001 From: guenther Date: Tue, 30 Aug 2016 14:52:09 +0000 Subject: [PATCH] Use a constant format string and output the variable part with %s ok krw@ millert@ --- lib/libutil/uucplock.c | 22 +++++++++++----------- 1 file changed, 11 insertions(+), 11 deletions(-) diff --git a/lib/libutil/uucplock.c b/lib/libutil/uucplock.c index bf63f775b34..f38382282ba 100644 --- a/lib/libutil/uucplock.c +++ b/lib/libutil/uucplock.c @@ -1,4 +1,4 @@ -/* $OpenBSD: uucplock.c,v 1.18 2016/08/30 14:44:45 guenther Exp $ */ +/* $OpenBSD: uucplock.c,v 1.19 2016/08/30 14:52:09 guenther Exp $ */ /* * Copyright (c) 1988, 1993 * The Regents of the University of California. All rights reserved. @@ -153,7 +153,7 @@ const char * uu_lockerr(int uu_lockresult) { static char errbuf[128]; - char *fmt; + const char *err; switch (uu_lockresult) { case UU_LOCK_INUSE: @@ -161,32 +161,32 @@ uu_lockerr(int uu_lockresult) case UU_LOCK_OK: return ""; case UU_LOCK_OPEN_ERR: - fmt = "open error: %s"; + err = "open error"; break; case UU_LOCK_READ_ERR: - fmt = "read error: %s"; + err = "read error"; break; case UU_LOCK_CREAT_ERR: - fmt = "creat error: %s"; + err = "creat error"; break; case UU_LOCK_WRITE_ERR: - fmt = "write error: %s"; + err = "write error"; break; case UU_LOCK_LINK_ERR: - fmt = "link error: %s"; + err = "link error"; break; case UU_LOCK_TRY_ERR: - fmt = "too many tries: %s"; + err = "too many tries"; break; case UU_LOCK_OWNER_ERR: - fmt = "not locking process: %s"; + err = "not locking process"; break; default: - fmt = "undefined error: %s"; + err = "undefined error"; break; } - (void)snprintf(errbuf, sizeof(errbuf), fmt, strerror(errno)); + (void)snprintf(errbuf, sizeof(errbuf), "%s: %s", err, strerror(errno)); return errbuf; } -- 2.20.1