From ed457e07b9bdc0acda9dc52307d8bc2118f32f06 Mon Sep 17 00:00:00 2001 From: Jeremy Allison Date: Tue, 12 Jan 2010 21:18:36 -0800 Subject: Fix two uses of strncat -> strlcat. Ensure proper use of strncpy when setting socket name. Jeremy. --- source3/modules/vfs_scannedonly.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'source3') diff --git a/source3/modules/vfs_scannedonly.c b/source3/modules/vfs_scannedonly.c index 69958988ac..ff16d78c3f 100644 --- a/source3/modules/vfs_scannedonly.c +++ b/source3/modules/vfs_scannedonly.c @@ -200,7 +200,7 @@ static int connect_to_scanner(vfs_handle_struct * handle) } saun.sun_family = AF_UNIX; strncpy(saun.sun_path, so->socketname, - strlen(so->socketname) + 1); + sizeof(saun.sun_path) - 1); if (connect(so->socket, (struct sockaddr *)(void *)&saun, SUN_LEN(&saun)) < 0) { DEBUG(2, ("failed to connect to socket %s\n", @@ -323,8 +323,8 @@ static void notify_scanner(vfs_handle_struct * handle, const char *scanfile) if (gsendlen + tmplen >= SENDBUFFERSIZE) { flush_sendbuffer(handle); } - strncat(so->gsendbuffer, tmp, tmplen); - strncat(so->gsendbuffer, "\n", 1); + strlcat(so->gsendbuffer, tmp, SENDBUFFERSIZE + 1); + strlcat(so->gsendbuffer, "\n", SENDBUFFERSIZE + 1); } static bool is_scannedonly_file(struct Tscannedonly *so, const char *shortname) -- cgit