diff -rup libqb-1.0.1.orig/lib/ipc_int.h libqb-1.0.1/lib/ipc_int.h --- libqb-1.0.1.orig/lib/ipc_int.h 2019-05-29 11:17:36.156450341 +0100 +++ libqb-1.0.1/lib/ipc_int.h 2019-05-29 11:20:19.561840335 +0100 @@ -161,7 +161,7 @@ enum qb_ipcs_connection_state { QB_IPCS_CONNECTION_SHUTTING_DOWN, }; -#define CONNECTION_DESCRIPTION (34) /* INT_MAX length + 3 */ +#define CONNECTION_DESCRIPTION NAME_MAX struct qb_ipcs_connection_auth { uid_t uid; @@ -208,4 +208,6 @@ int32_t qb_ipc_us_sock_error_is_disconne int use_filesystem_sockets(void); +void remove_tempdir(const char *name); + #endif /* QB_IPC_INT_H_DEFINED */ Only in libqb-1.0.1/lib: ipc_int.h.orig diff -rup libqb-1.0.1.orig/lib/ipcs.c libqb-1.0.1/lib/ipcs.c --- libqb-1.0.1.orig/lib/ipcs.c 2016-11-08 10:10:23.000000000 +0000 +++ libqb-1.0.1/lib/ipcs.c 2019-05-29 11:20:19.563840340 +0100 @@ -642,12 +642,13 @@ qb_ipcs_disconnect(struct qb_ipcs_connec scheduled_retry = 1; } } - + remove_tempdir(c->description); if (scheduled_retry == 0) { /* This removes the initial alloc ref */ qb_ipcs_connection_unref(c); } } + } static void diff -rup libqb-1.0.1.orig/lib/ipc_setup.c libqb-1.0.1/lib/ipc_setup.c --- libqb-1.0.1.orig/lib/ipc_setup.c 2019-05-29 11:17:36.156450341 +0100 +++ libqb-1.0.1/lib/ipc_setup.c 2019-05-29 11:20:19.562840338 +0100 @@ -621,6 +621,8 @@ handle_new_connection(struct qb_ipcs_ser int32_t res2 = 0; uint32_t max_buffer_size = QB_MAX(req->max_msg_size, s->max_buffer_size); struct qb_ipc_connection_response response; + const char suffix[] = "/qb"; + int desc_len; c = qb_ipcs_connection_alloc(s); if (c == NULL) { @@ -643,8 +645,45 @@ handle_new_connection(struct qb_ipcs_ser c->auth.gid = c->egid = ugp->gid; c->auth.mode = 0600; c->stats.client_pid = ugp->pid; - snprintf(c->description, CONNECTION_DESCRIPTION, - "%d-%d-%d", s->pid, ugp->pid, c->setup.u.us.sock); + +#if defined(QB_LINUX) || defined(QB_CYGWIN) + desc_len = snprintf(c->description, CONNECTION_DESCRIPTION - sizeof suffix, + "/dev/shm/qb-%d-%d-%d-XXXXXX", s->pid, ugp->pid, c->setup.u.us.sock); + if (desc_len < 0) { + res = -errno; + goto send_response; + } + if (desc_len >= CONNECTION_DESCRIPTION - sizeof suffix) { + res = -ENAMETOOLONG; + goto send_response; + } + if (mkdtemp(c->description) == NULL) { + res = -errno; + goto send_response; + } + if (chmod(c->description, 0770)) { + res = -errno; + goto send_response; + } + /* chown can fail because we might not be root */ + (void)chown(c->description, c->auth.uid, c->auth.gid); + + /* We can't pass just a directory spec to the clients */ + memcpy(c->description + desc_len, suffix, sizeof suffix); +#else + desc_len = snprintf(c->description, CONNECTION_DESCRIPTION, + "%d-%d-%d", s->pid, ugp->pid, c->setup.u.us.sock); + if (desc_len < 0) { + res = -errno; + goto send_response; + } + if (desc_len >= CONNECTION_DESCRIPTION) { + res = -ENAMETOOLONG; + goto send_response; + } +#endif + + if (auth_result == 0 && c->service->serv_fns.connection_accept) { res = c->service->serv_fns.connection_accept(c, @@ -865,3 +904,21 @@ retry_accept: qb_ipcs_uc_recv_and_auth(new_fd, s); return 0; } + +void remove_tempdir(const char *name) +{ +#if defined(QB_LINUX) || defined(QB_CYGWIN) + char dirname[PATH_MAX]; + char *slash = strrchr(name, '/'); + + if (slash && slash - name < sizeof dirname) { + memcpy(dirname, name, slash - name); + dirname[slash - name] = '\0'; + /* This gets called more than it needs to be really, so we don't check + * the return code. It's more of a desperate attempt to clean up after ourself + * in either the server or client. + */ + (void)rmdir(dirname); + } +#endif +} Only in libqb-1.0.1/lib: ipc_setup.c.orig diff -rup libqb-1.0.1.orig/lib/ipc_shm.c libqb-1.0.1/lib/ipc_shm.c --- libqb-1.0.1.orig/lib/ipc_shm.c 2019-05-29 11:17:36.156450341 +0100 +++ libqb-1.0.1/lib/ipc_shm.c 2019-05-29 11:21:38.135028190 +0100 @@ -267,6 +267,7 @@ qb_ipcs_shm_disconnect(struct qb_ipcs_co } end_disconnect: sigaction(SIGBUS, &old_sa, NULL); + remove_tempdir(c->description); } static int32_t @@ -313,11 +314,11 @@ qb_ipcs_shm_connect(struct qb_ipcs_servi qb_util_log(LOG_DEBUG, "connecting to client [%d]", c->pid); snprintf(r->request, NAME_MAX, "%s-request-%s", - s->name, c->description); + c->description, s->name); snprintf(r->response, NAME_MAX, "%s-response-%s", - s->name, c->description); + c->description, s->name); snprintf(r->event, NAME_MAX, "%s-event-%s", - s->name, c->description); + c->description, s->name); res = qb_ipcs_shm_rb_open(c, &c->request, r->request); Only in libqb-1.0.1/lib: ipc_shm.c~ Only in libqb-1.0.1/lib: ipc_shm.c.orig Only in libqb-1.0.1/lib: ipc_shm.c.rej diff -rup libqb-1.0.1.orig/lib/ipc_socket.c libqb-1.0.1/lib/ipc_socket.c --- libqb-1.0.1.orig/lib/ipc_socket.c 2019-05-29 11:17:36.150450327 +0100 +++ libqb-1.0.1/lib/ipc_socket.c 2019-05-29 11:20:19.563840340 +0100 @@ -364,6 +364,10 @@ qb_ipcc_us_disconnect(struct qb_ipcc_con unlink(sock_name); } } + + /* Last-ditch attempt to tidy up after ourself */ + remove_tempdir(c->request.u.us.shared_file_name); + qb_ipcc_us_sock_close(c->event.u.us.sock); qb_ipcc_us_sock_close(c->request.u.us.sock); qb_ipcc_us_sock_close(c->setup.u.us.sock); @@ -742,7 +746,10 @@ qb_ipcs_us_disconnect(struct qb_ipcs_con c->state == QB_IPCS_CONNECTION_ACTIVE) { munmap(c->request.u.us.shared_data, SHM_CONTROL_SIZE); unlink(c->request.u.us.shared_file_name); + + } + remove_tempdir(c->description); } static int32_t @@ -761,13 +768,13 @@ qb_ipcs_us_connect(struct qb_ipcs_servic c->request.u.us.sock = c->setup.u.us.sock; c->response.u.us.sock = c->setup.u.us.sock; - snprintf(r->request, NAME_MAX, "qb-%s-control-%s", - s->name, c->description); - snprintf(r->response, NAME_MAX, "qb-%s-%s", s->name, c->description); + snprintf(r->request, NAME_MAX, "%s-control-%s", + c->description, s->name); + snprintf(r->response, NAME_MAX, "%s-%s", c->description, s->name); fd_hdr = qb_sys_mmap_file_open(path, r->request, SHM_CONTROL_SIZE, - O_CREAT | O_TRUNC | O_RDWR); + O_CREAT | O_TRUNC | O_RDWR | O_EXCL); if (fd_hdr < 0) { res = fd_hdr; errno = -fd_hdr; Only in libqb-1.0.1/lib: ipc_socket.c.orig diff -rup libqb-1.0.1.orig/lib/ringbuffer.c libqb-1.0.1/lib/ringbuffer.c --- libqb-1.0.1.orig/lib/ringbuffer.c 2019-05-29 11:17:36.147450320 +0100 +++ libqb-1.0.1/lib/ringbuffer.c 2019-05-29 11:20:19.563840340 +0100 @@ -155,7 +155,7 @@ qb_rb_open_2(const char *name, size_t si sizeof(struct qb_ringbuffer_shared_s) + shared_user_data_size; if (flags & QB_RB_FLAG_CREATE) { - file_flags |= O_CREAT | O_TRUNC; + file_flags |= O_CREAT | O_TRUNC | O_EXCL; } rb = calloc(1, sizeof(struct qb_ringbuffer_s)); @@ -166,7 +166,7 @@ qb_rb_open_2(const char *name, size_t si /* * Create a shared_hdr memory segment for the header. */ - snprintf(filename, PATH_MAX, "qb-%s-header", name); + snprintf(filename, PATH_MAX, "%s-header", name); fd_hdr = qb_sys_mmap_file_open(path, filename, shared_size, file_flags); if (fd_hdr < 0) { @@ -217,7 +217,7 @@ qb_rb_open_2(const char *name, size_t si * They have to be separate. */ if (flags & QB_RB_FLAG_CREATE) { - snprintf(filename, PATH_MAX, "qb-%s-data", name); + snprintf(filename, PATH_MAX, "%s-data", name); fd_data = qb_sys_mmap_file_open(path, filename, real_size, file_flags); diff -rup libqb-1.0.1.orig/lib/unix.c libqb-1.0.1/lib/unix.c --- libqb-1.0.1.orig/lib/unix.c 2016-11-08 11:15:16.000000000 +0000 +++ libqb-1.0.1/lib/unix.c 2019-05-29 11:20:19.564840342 +0100 @@ -81,7 +81,9 @@ qb_sys_mmap_file_open(char *path, const (void)strlcpy(path, file, PATH_MAX); } else { #if defined(QB_LINUX) || defined(QB_CYGWIN) - snprintf(path, PATH_MAX, "/dev/shm/%s", file); + /* This is only now called when talking to an old libqb + where we need to add qb- to the name */ + snprintf(path, PATH_MAX, "/dev/shm/qb-%s", file); #else snprintf(path, PATH_MAX, "%s/%s", SOCKETDIR, file); is_absolute = path;