From 10ae5480527ac964306aa42f25e4394bc6878b36 Mon Sep 17 00:00:00 2001 From: Willy Tarreau Date: Thu, 18 Oct 2007 16:15:52 +0200 Subject: [PATCH] [BUG] fix off-by-one in path length in destroy_uxst_socket() An off-by-one error was left in the computation of the unix socket path. --- src/proto_uxst.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/proto_uxst.c b/src/proto_uxst.c index d8c0a9090..24c435ff6 100644 --- a/src/proto_uxst.c +++ b/src/proto_uxst.c @@ -198,7 +198,7 @@ static void destroy_uxst_socket(const char *path) addr.sun_family = AF_UNIX; strncpy(addr.sun_path, path, sizeof(addr.sun_path)); - addr.sun_path[sizeof(addr.sun_path)] = 0; + addr.sun_path[sizeof(addr.sun_path) - 1] = 0; ret = connect(sock, (struct sockaddr *)&addr, sizeof(addr)); if (ret < 0 && errno == ECONNREFUSED) { /* Connect failed: the socket still exists but is not used