From 8a8763b7cf131f2cec6ead11b87e4d476483ffca Mon Sep 17 00:00:00 2001 From: Pawel Jakub Dawidek Date: Mon, 21 Mar 2011 08:37:50 +0000 Subject: [PATCH] Use snprlcat() instead of two strlcat(3)s. MFC after: 1 week --- sbin/hastd/hooks.c | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/sbin/hastd/hooks.c b/sbin/hastd/hooks.c index 7bc9913c463..5692565f748 100644 --- a/sbin/hastd/hooks.c +++ b/sbin/hastd/hooks.c @@ -52,6 +52,7 @@ __FBSDID("$FreeBSD$"); #include #include "hooks.h" +#include "subr.h" #include "synch.h" /* Report processes that are running for too long not often than this value. */ @@ -184,8 +185,8 @@ hook_alloc(const char *path, char **args) (void)strlcpy(hp->hp_comm, path, sizeof(hp->hp_comm)); /* We start at 2nd argument as we don't want to have exec name twice. */ for (ii = 1; args[ii] != NULL; ii++) { - (void)strlcat(hp->hp_comm, " ", sizeof(hp->hp_comm)); - (void)strlcat(hp->hp_comm, args[ii], sizeof(hp->hp_comm)); + (void)snprlcat(hp->hp_comm, sizeof(hp->hp_comm), " %s", + args[ii]); } if (strlen(hp->hp_comm) >= sizeof(hp->hp_comm) - 1) { pjdlog_error("Exec path too long, correct configuration file.");