summaryrefslogtreecommitdiffhomepage
path: root/src/nxt_controller.c
diff options
context:
space:
mode:
authorAndrew Clayton <a.clayton@nginx.com>2022-08-07 00:19:24 +0100
committerAndrew Clayton <andrew@digital-domain.net>2022-10-03 14:26:45 +0100
commit57fc9201cb91e3d8901a64e3daaaf31684ee5bf5 (patch)
treeeff38400658ae0dd62046f1e426734cc4753c9f9 /src/nxt_controller.c
parent2e69b7eb5712e03a198caa66240c2002175c5ed9 (diff)
downloadunit-57fc9201cb91e3d8901a64e3daaaf31684ee5bf5.tar.gz
unit-57fc9201cb91e3d8901a64e3daaaf31684ee5bf5.tar.bz2
Socket: Created control socket & pid file directories.
@alejandro-colomar reported an issue on GitHub whereby Unit would fail to start due to not being able to create the control socket (a Unix Domain Socket) 2022/08/05 20:12:22 [alert] 21613#21613 bind(6, unix:/opt/local/unit/var/run/unit/control.unit.sock.tmp) failed (2: No such file or directory) This could happen if the control socket was set to a directory that doesn't exist. A common place to put the control socket would be under /run/unit, and while /run will exist, /run/unit may well not (/run is/should be cleared on each boot). The pid file would also generally go under /run/unit, though this is created after the control socket, however it could go someplace else so we should also ensure its directory exists. This commit will try to create the pid file and control sockets parent directory. In some cases the user will need to ensure that the rest of the path already exists. This adds a new nxt_fs_mkdir_parent() function that given a full path to a file (or directory), strips the last component off before passing the remaining directory path to nxt_fs_mkdir(). Cc: Konstantin Pavlov <thresh@nginx.com> Closes: <https://github.com/nginx/unit/issues/742> Reported-by: Alejandro Colomar <alx@nginx.com> Reviewed-by: Alejandro Colomar <alx@nginx.com> Tested-by: Alejandro Colomar <alx@nginx.com> Signed-off-by: Andrew Clayton <a.clayton@nginx.com>
Diffstat (limited to 'src/nxt_controller.c')
-rw-r--r--src/nxt_controller.c8
1 files changed, 8 insertions, 0 deletions
diff --git a/src/nxt_controller.c b/src/nxt_controller.c
index 09168821..99c7ca10 100644
--- a/src/nxt_controller.c
+++ b/src/nxt_controller.c
@@ -666,6 +666,14 @@ nxt_runtime_controller_socket(nxt_task_t *task, nxt_runtime_t *rt)
#endif
ls->handler = nxt_controller_conn_init;
+#if (NXT_HAVE_UNIX_DOMAIN)
+ if (ls->sockaddr->u.sockaddr.sa_family == AF_UNIX) {
+ const char *path = ls->sockaddr->u.sockaddr_un.sun_path;
+
+ nxt_fs_mkdir_parent((const u_char *) path, 0755);
+ }
+#endif
+
if (nxt_listen_socket_create(task, rt->mem_pool, ls) != NXT_OK) {
return NXT_ERROR;
}