diff --git a/cmd-break-pane.c b/cmd-break-pane.c index 4be989c3..0d8d83fd 100644 --- a/cmd-break-pane.c +++ b/cmd-break-pane.c @@ -58,9 +58,14 @@ cmd_break_pane_exec(struct cmd *self, struct cmdq_item *item) struct session *dst_s = target->s; struct window_pane *wp = source->wp; struct window *w = wl->window; - char *name, *cause, *cp; + char *newname, *cause, *cp; int idx = target->idx, before; - const char *template; + const char *template, *name = args_get(args, 'n'); + + if (name != NULL && !check_name(name, WINDOW_NAME_FORBID)) { + cmdq_error(item, "invalid window name: %s", name); + return (CMD_RETURN_ERROR); + } before = args_has(args, 'b'); if (args_has(args, 'a') || before) { @@ -80,8 +85,8 @@ cmd_break_pane_exec(struct cmd *self, struct cmdq_item *item) free(cause); return (CMD_RETURN_ERROR); } - if (args_has(args, 'n')) { - window_set_name(w, args_get(args, 'n')); + if (name != NULL) { + window_set_name(w, name, WINDOW_NAME_FORBID); options_set_number(w->options, "automatic-rename", 0); } server_unlink_window(src_s, wl); @@ -109,12 +114,12 @@ cmd_break_pane_exec(struct cmd *self, struct cmdq_item *item) w->active = wp; w->latest = tc; - if (!args_has(args, 'n')) { - name = default_window_name(w); - window_set_name(w, name); - free(name); + if (name != NULL) { + newname = default_window_name(w); + window_set_name(w, newname, WINDOW_NAME_FORBID); + free(newname); } else { - window_set_name(w, args_get(args, 'n')); + window_set_name(w, name, 0); options_set_number(w->options, "automatic-rename", 0); } diff --git a/cmd-display-panes.c b/cmd-display-panes.c index 054c3c35..53e7989b 100644 --- a/cmd-display-panes.c +++ b/cmd-display-panes.c @@ -264,7 +264,7 @@ cmd_display_panes_draw(struct client *c, __unused void *data, log_debug("%s: %s @%u", __func__, c->name, w->id); TAILQ_FOREACH(wp, &w->panes, entry) { - if (window_pane_visible(wp)) + if (window_pane_is_visible(wp)) cmd_display_panes_draw_pane(ctx, wp); } } diff --git a/cmd-new-session.c b/cmd-new-session.c index a397ccf9..dbee4e55 100644 --- a/cmd-new-session.c +++ b/cmd-new-session.c @@ -77,8 +77,8 @@ cmd_new_session_exec(struct cmd *self, struct cmdq_item *item) struct termios tio, *tiop; struct session_group *sg = NULL; const char *errstr, *template, *group, *tmp; - char *cause, *cwd = NULL, *cp, *newname = NULL; - char *name, *prefix = NULL; + char *cause, *cwd = NULL, *cp, *ename; + char *wname = NULL, *sname = NULL, *prefix = NULL; int detached, already_attached, is_control = 0; u_int sx, sy, dsx, dsy, count = args_count(args); struct spawn_context sc = { 0 }; @@ -99,20 +99,29 @@ cmd_new_session_exec(struct cmd *self, struct cmdq_item *item) return (CMD_RETURN_ERROR); } - tmp = args_get(args, 's'); - if (tmp != NULL) { - name = format_single(item, tmp, c, NULL, NULL, NULL); - newname = clean_name(name, "#:."); - if (newname == NULL) { - cmdq_error(item, "invalid session: %s", name); - free(name); + if ((tmp = args_get(args, 'n')) != NULL) { + ename = format_single(item, tmp, c, NULL, NULL, NULL); + if (!check_name(ename, WINDOW_NAME_FORBID)) { + cmdq_error(item, "invalid window name: %s", ename); + free(ename); return (CMD_RETURN_ERROR); } - free(name); + wname = clean_name(ename, WINDOW_NAME_FORBID); + free(ename); + } + if ((tmp = args_get(args, 's')) != NULL) { + ename = format_single(item, tmp, c, NULL, NULL, NULL); + if (!check_name(ename, SESSION_NAME_FORBID)) { + cmdq_error(item, "invalid session name: %s", ename); + free(ename); + goto fail; + } + sname = clean_name(ename, SESSION_NAME_FORBID); + free(ename); } if (args_has(args, 'A')) { - if (newname != NULL) - as = session_find(newname); + if (sname != NULL) + as = session_find(sname); else as = target->s; if (as != NULL) { @@ -120,12 +129,13 @@ cmd_new_session_exec(struct cmd *self, struct cmdq_item *item) args_has(args, 'D'), args_has(args, 'X'), 0, args_get(args, 'c'), args_has(args, 'E'), args_get(args, 'f')); - free(newname); + free(wname); + free(sname); return (retval); } } - if (newname != NULL && session_find(newname) != NULL) { - cmdq_error(item, "duplicate session: %s", newname); + if (sname != NULL && session_find(sname) != NULL) { + cmdq_error(item, "duplicate session: %s", sname); goto fail; } @@ -142,12 +152,12 @@ cmd_new_session_exec(struct cmd *self, struct cmdq_item *item) else if (groupwith != NULL) prefix = xstrdup(groupwith->name); else { - prefix = clean_name(group, "#:."); - if (prefix == NULL) { - cmdq_error(item, "invalid session group: %s", - group); + if (!check_name(group, SESSION_NAME_FORBID)) { + cmdq_error(item, + "invalid session group name: %s", group); goto fail; } + prefix = clean_name(group, SESSION_NAME_FORBID); } } @@ -276,7 +286,7 @@ cmd_new_session_exec(struct cmd *self, struct cmdq_item *item) environ_put(env, av->string, 0); av = args_next_value(av); } - s = session_create(prefix, newname, cwd, env, oo, tiop); + s = session_create(prefix, sname, cwd, env, oo, tiop); /* Spawn the initial window. */ sc.item = item; @@ -284,7 +294,7 @@ cmd_new_session_exec(struct cmd *self, struct cmdq_item *item) if (!detached) sc.tc = c; - sc.name = args_get(args, 'n'); + sc.name = wname; args_to_vector(args, &sc.argc, &sc.argv); sc.idx = -1; @@ -357,7 +367,8 @@ cmd_new_session_exec(struct cmd *self, struct cmdq_item *item) if (sc.argv != NULL) cmd_free_argv(sc.argc, sc.argv); free(cwd); - free(newname); + free(wname); + free(sname); free(prefix); return (CMD_RETURN_NORMAL); @@ -365,7 +376,8 @@ fail: if (sc.argv != NULL) cmd_free_argv(sc.argc, sc.argv); free(cwd); - free(newname); + free(wname); + free(sname); free(prefix); return (CMD_RETURN_ERROR); } diff --git a/cmd-new-window.c b/cmd-new-window.c index dd64baab..c7a203e2 100644 --- a/cmd-new-window.c +++ b/cmd-new-window.c @@ -61,7 +61,7 @@ cmd_new_window_exec(struct cmd *self, struct cmdq_item *item) struct session *s = target->s; struct winlink *wl = target->wl, *new_wl = NULL; int idx = target->idx, before; - char *cause = NULL, *cp, *expanded; + char *cause = NULL, *cp, *expanded, *wname; const char *template, *name; struct cmd_find_state fs; struct args_value *av; @@ -71,8 +71,18 @@ cmd_new_window_exec(struct cmd *self, struct cmdq_item *item) * name already exists, select it. */ name = args_get(args, 'n'); - if (args_has(args, 'S') && name != NULL && target->idx == -1) { + if (name != NULL) { expanded = format_single(item, name, c, s, NULL, NULL); + if (!check_name(expanded, WINDOW_NAME_FORBID)) { + cmdq_error(item, "invalid window name: %s", expanded); + free(expanded); + return (CMD_RETURN_ERROR); + } + wname = clean_name(expanded, WINDOW_NAME_FORBID); + free(expanded); + } + if (args_has(args, 'S') && wname != NULL && target->idx == -1) { + expanded = format_single(item, wname, c, s, NULL, NULL); RB_FOREACH(wl, winlinks, &s->windows) { if (strcmp(wl->window->name, expanded) != 0) continue; @@ -80,12 +90,14 @@ cmd_new_window_exec(struct cmd *self, struct cmdq_item *item) new_wl = wl; continue; } - cmdq_error(item, "multiple windows named %s", name); + cmdq_error(item, "multiple windows named %s", wname); + free(wname); free(expanded); return (CMD_RETURN_ERROR); } free(expanded); if (new_wl != NULL) { + free(wname); if (args_has(args, 'd')) return (CMD_RETURN_NORMAL); if (session_set_current(s, new_wl) == 0) @@ -108,7 +120,7 @@ cmd_new_window_exec(struct cmd *self, struct cmdq_item *item) sc.s = s; sc.tc = tc; - sc.name = args_get(args, 'n'); + sc.name = wname; args_to_vector(args, &sc.argc, &sc.argv); sc.environ = environ_create(); @@ -130,10 +142,7 @@ cmd_new_window_exec(struct cmd *self, struct cmdq_item *item) if ((new_wl = spawn_window(&sc, &cause)) == NULL) { cmdq_error(item, "create window failed: %s", cause); free(cause); - if (sc.argv != NULL) - cmd_free_argv(sc.argc, sc.argv); - environ_free(sc.environ); - return (CMD_RETURN_ERROR); + goto fail; } if (!args_has(args, 'd') || new_wl == s->curw) { cmd_find_from_winlink(current, new_wl, 0); @@ -156,5 +165,13 @@ cmd_new_window_exec(struct cmd *self, struct cmdq_item *item) if (sc.argv != NULL) cmd_free_argv(sc.argc, sc.argv); environ_free(sc.environ); + free(wname); return (CMD_RETURN_NORMAL); + +fail: + if (sc.argv != NULL) + cmd_free_argv(sc.argc, sc.argv); + environ_free(sc.environ); + free(wname); + return (CMD_RETURN_ERROR); } diff --git a/cmd-rename-session.c b/cmd-rename-session.c index 37b26100..ca6a2eac 100644 --- a/cmd-rename-session.c +++ b/cmd-rename-session.c @@ -52,12 +52,12 @@ cmd_rename_session_exec(struct cmd *self, struct cmdq_item *item) char *newname, *tmp; tmp = format_single_from_target(item, args_string(args, 0)); - newname = clean_name(tmp, "#:."); - if (newname == NULL) { - cmdq_error(item, "invalid session: %s", tmp); + if (!check_name(tmp, SESSION_NAME_FORBID)) { + cmdq_error(item, "invalid session name: %s", tmp); free(tmp); return (CMD_RETURN_ERROR); } + newname = clean_name(tmp, SESSION_NAME_FORBID); free(tmp); if (strcmp(newname, s->name) == 0) { free(newname); diff --git a/cmd-rename-window.c b/cmd-rename-window.c index 472b571b..8428e23f 100644 --- a/cmd-rename-window.c +++ b/cmd-rename-window.c @@ -48,15 +48,21 @@ cmd_rename_window_exec(struct cmd *self, struct cmdq_item *item) struct args *args = cmd_get_args(self); struct cmd_find_state *target = cmdq_get_target(item); struct winlink *wl = target->wl; - char *newname; + char *name; - newname = format_single_from_target(item, args_string(args, 0)); - window_set_name(wl->window, newname); + name = format_single_from_target(item, args_string(args, 0)); + if (!check_name(name, WINDOW_NAME_FORBID)) { + cmdq_error(item, "invalid window name: %s", name); + free(name); + return (CMD_RETURN_ERROR); + } + + window_set_name(wl->window, name, WINDOW_NAME_FORBID); options_set_number(wl->window->options, "automatic-rename", 0); + free(name); server_redraw_window_borders(wl->window); server_status_window(wl->window); - free(newname); return (CMD_RETURN_NORMAL); } diff --git a/cmd-select-pane.c b/cmd-select-pane.c index e7fee1d6..d146ea6c 100644 --- a/cmd-select-pane.c +++ b/cmd-select-pane.c @@ -135,7 +135,7 @@ cmd_select_pane_exec(struct cmd *self, struct cmdq_item *item) } if (args_has(args, 'm') || args_has(args, 'M')) { - if (args_has(args, 'm') && !window_pane_visible(wp)) + if (args_has(args, 'm') && !window_pane_is_visible(wp)) return (CMD_RETURN_NORMAL); if (server_check_marked()) lastwp = marked_pane.wp; @@ -217,7 +217,7 @@ cmd_select_pane_exec(struct cmd *self, struct cmdq_item *item) if (args_has(args, 'T')) { title = format_single_from_target(item, args_get(args, 'T')); - if (screen_set_title(&wp->base, title)) { + if (screen_set_title(&wp->base, title, 0)) { notify_pane("pane-title-changed", wp); server_redraw_window_borders(wp->window); server_status_window(wp->window); diff --git a/cmd-split-window.c b/cmd-split-window.c index 2b9a4542..8ff52a04 100644 --- a/cmd-split-window.c +++ b/cmd-split-window.c @@ -213,7 +213,7 @@ cmd_split_window_exec(struct cmd *self, struct cmdq_item *item) } if (args_has(args, 'T')) { title = format_single_from_target(item, args_get(args, 'T')); - screen_set_title(&new_wp->base, title); + screen_set_title(&new_wp->base, title, 0); notify_pane("pane-title-changed", new_wp); free(title); } diff --git a/input-keys.c b/input-keys.c index 662b4214..39692232 100644 --- a/input-keys.c +++ b/input-keys.c @@ -806,7 +806,7 @@ input_key_mouse(struct window_pane *wp, struct mouse_event *m) return; if (cmd_mouse_at(wp, m, &x, &y, 0) != 0) return; - if (!window_pane_visible(wp)) + if (!window_pane_is_visible(wp)) return; if (!input_key_get_mouse(s, m, x, y, &buf, &len)) return; diff --git a/input.c b/input.c index 21920064..aa7b0c0c 100644 --- a/input.c +++ b/input.c @@ -2701,7 +2701,7 @@ input_exit_osc(struct input_ctx *ictx) case 2: if (wp != NULL && options_get_number(wp->options, "allow-set-title") && - screen_set_title(sctx->s, p)) { + screen_set_title(sctx->s, p, 1)) { notify_pane("pane-title-changed", wp); server_redraw_window_borders(wp->window); server_status_window(wp->window); @@ -2711,7 +2711,7 @@ input_exit_osc(struct input_ctx *ictx) input_osc_4(ictx, p); break; case 7: - if (screen_set_path(sctx->s, p) && wp != NULL) { + if (wp != NULL && screen_set_path(sctx->s, p, 1)) { server_redraw_window_borders(wp->window); server_status_window(wp->window); } @@ -2779,7 +2779,7 @@ input_exit_apc(struct input_ctx *ictx) if (wp != NULL && options_get_number(wp->options, "allow-set-title") && - screen_set_title(sctx->s, ictx->input_buf)) { + screen_set_title(sctx->s, ictx->input_buf, 1)) { notify_pane("pane-title-changed", wp); server_redraw_window_borders(wp->window); server_status_window(wp->window); @@ -2822,10 +2822,10 @@ input_exit_rename(struct input_ctx *ictx) if (o != NULL) options_remove_or_default(o, -1, NULL); if (!options_get_number(w->options, "automatic-rename")) - window_set_name(w, ""); + window_set_name(w, "", WINDOW_NAME_FORBID_EXT); } else { options_set_number(w->options, "automatic-rename", 0); - window_set_name(w, ictx->input_buf); + window_set_name(w, ictx->input_buf, WINDOW_NAME_FORBID_EXT); } server_redraw_window_borders(w); server_status_window(w); diff --git a/names.c b/names.c index 8a04df73..756af5b0 100644 --- a/names.c +++ b/names.c @@ -95,7 +95,7 @@ check_window_name(struct window *w) name = format_window_name(w); if (strcmp(name, w->name) != 0) { log_debug("@%u new name %s (was %s)", w->id, name, w->name); - window_set_name(w, name); + window_set_name(w, name, WINDOW_NAME_FORBID_EXT); server_redraw_window_borders(w); server_status_window(w); } else @@ -166,7 +166,7 @@ parse_window_name(const char *in) if (*name == '/') name = basename(name); - name = clean_name(name, "#"); + name = clean_name(name, WINDOW_NAME_FORBID); free(copy); if (name == NULL) return (xstrdup("")); diff --git a/paste.c b/paste.c index 6461d9ba..081e3dbe 100644 --- a/paste.c +++ b/paste.c @@ -176,7 +176,6 @@ paste_add(const char *prefix, char *data, size_t size) } pb = xmalloc(sizeof *pb); - pb->name = NULL; do { free(pb->name); @@ -296,7 +295,6 @@ paste_set(char *data, size_t size, const char *name, char **cause) } pb = xmalloc(sizeof *pb); - pb->name = newname; pb->data = data; diff --git a/popup.c b/popup.c index ed2f4e63..999e4066 100644 --- a/popup.c +++ b/popup.c @@ -437,7 +437,7 @@ popup_make_pane(struct popup_data *pd, enum layout_type type) pd->job = NULL; } - screen_set_title(&pd->s, new_wp->base.title); + screen_set_title(&pd->s, new_wp->base.title, 0); screen_free(&new_wp->base); memcpy(&new_wp->base, &pd->s, sizeof wp->base); screen_resize(&new_wp->base, new_wp->sx, new_wp->sy, 1); diff --git a/screen-redraw.c b/screen-redraw.c index 3bec3852..91c79c16 100644 --- a/screen-redraw.c +++ b/screen-redraw.c @@ -316,7 +316,7 @@ screen_redraw_cell_border(struct screen_redraw_ctx *ctx, struct window_pane *wp, * single z-index. */ TAILQ_FOREACH(wp2, &w->z_index, zentry) { - if (!window_pane_visible(wp2) || window_pane_is_floating(wp2)) + if (!window_pane_is_visible(wp2) || window_pane_is_floating(wp2)) continue; n = screen_redraw_cell_border1(ctx, sb_pos, sb_w, wp2, px, py); if (n != -1) @@ -493,7 +493,7 @@ screen_redraw_check_cell(struct screen_redraw_ctx *ctx, int px, int py, if (!window_pane_is_floating(wp)) tiled_only = 1; do { /* loop until back to wp == start */ - if (!window_pane_visible(wp)) + if (!window_pane_is_visible(wp)) goto next; if (tiled_only && window_pane_is_floating(wp)) goto next; @@ -692,7 +692,7 @@ screen_redraw_draw_pane_status(struct screen_redraw_ctx *ctx) log_debug("%s: %s @%u", __func__, c->name, w->id); TAILQ_FOREACH(wp, &w->panes, entry) { - if (!window_pane_visible(wp)) + if (!window_pane_is_visible(wp)) continue; s = &wp->status_screen; @@ -866,7 +866,7 @@ screen_redraw_pane(struct client *c, struct window_pane *wp, { struct screen_redraw_ctx ctx; - if (!window_pane_visible(wp)) + if (!window_pane_is_visible(wp)) return; screen_redraw_set_context(c, &ctx); @@ -1101,7 +1101,7 @@ screen_redraw_draw_panes(struct screen_redraw_ctx *ctx) log_debug("%s: %s @%u", __func__, c->name, w->id); TAILQ_FOREACH(wp, &w->panes, entry) { - if (window_pane_visible(wp)) + if (window_pane_is_visible(wp)) screen_redraw_draw_pane(ctx, wp); } } @@ -1221,7 +1221,7 @@ screen_redraw_get_visible_ranges(struct window_pane *base_wp, int px, bb = wp->yoff + (int)wp->sy; } if (!found_self || - !window_pane_visible(wp) || + !window_pane_is_visible(wp) || py < tb || py > bb) continue; @@ -1451,7 +1451,7 @@ screen_redraw_draw_pane_scrollbars(struct screen_redraw_ctx *ctx) TAILQ_FOREACH(wp, &w->panes, entry) { if (window_pane_show_scrollbar(wp, ctx->pane_scrollbars) && - window_pane_visible(wp)) + window_pane_is_visible(wp)) screen_redraw_draw_pane_scrollbar(ctx, wp); } } diff --git a/screen.c b/screen.c index 233f2d06..0a761660 100644 --- a/screen.c +++ b/screen.c @@ -245,11 +245,14 @@ screen_set_cursor_colour(struct screen *s, int colour) /* Set screen title. */ int -screen_set_title(struct screen *s, const char *title) +screen_set_title(struct screen *s, const char *title, int untrusted) { char *new_title; - new_title = clean_name(title, "#"); + if (untrusted) + new_title = clean_name(title, "#"); + else + new_title = clean_name(title, ""); if (new_title == NULL) return (0); free(s->title); @@ -259,11 +262,14 @@ screen_set_title(struct screen *s, const char *title) /* Set screen path. */ int -screen_set_path(struct screen *s, const char *path) +screen_set_path(struct screen *s, const char *path, int untrusted) { char *new_path; - new_path = clean_name(path, "#"); + if (untrusted) + new_path = clean_name(path, "#"); + else + new_path = clean_name(path, ""); if (new_path == NULL) return (0); free(s->path); diff --git a/session.c b/session.c index 1e558977..445897e2 100644 --- a/session.c +++ b/session.c @@ -116,7 +116,6 @@ session_create(const char *prefix, const char *name, const char *cwd, s = xcalloc(1, sizeof *s); s->references = 1; s->flags = 0; - s->cwd = xstrdup(cwd); TAILQ_INIT(&s->lastw); diff --git a/spawn.c b/spawn.c index b0a4f7df..ddeca0d3 100644 --- a/spawn.c +++ b/spawn.c @@ -74,15 +74,12 @@ spawn_log(const char *from, struct spawn_context *sc) struct winlink * spawn_window(struct spawn_context *sc, char **cause) { - struct cmdq_item *item = sc->item; - struct client *c = cmdq_get_client(item); struct session *s = sc->s; struct window *w; struct window_pane *wp; struct winlink *wl; int idx = sc->idx; u_int sx, sy, xpixel, ypixel; - char *name; spawn_log(__func__, sc); @@ -180,15 +177,12 @@ spawn_window(struct spawn_context *sc, char **cause) /* Set the name of the new window. */ if (~sc->flags & SPAWN_RESPAWN) { free(w->name); - if (sc->name != NULL) { - name = format_single(item, sc->name, c, s, NULL, NULL); - w->name = clean_name(name, "#"); - free(name); - if (w->name == NULL) - w->name = xstrdup(""); - options_set_number(w->options, "automatic-rename", 0); - } else + if (sc->name == NULL || *sc->name == '\0') w->name = default_window_name(w); + else { + w->name = xstrdup(sc->name); + options_set_number(w->options, "automatic-rename", 0); + } } /* Switch to the new window if required. */ diff --git a/tmux.c b/tmux.c index 251b0336..c06a2435 100644 --- a/tmux.c +++ b/tmux.c @@ -298,6 +298,25 @@ clean_name(const char *name, const char* forbid) return (new_name); } +/* + * Check a name given by a command: reject it if it is empty, not valid UTF-8, + * or contains a forbidden character. Other characters that clean_name would + * change (for example with utf8_stravis) are allowed and fixed silently. + */ +int +check_name(const char *name, const char *forbid) +{ + const char *cp; + + if (*name == '\0' || !utf8_isvalid(name)) + return (0); + for (cp = name; *cp != '\0'; cp++) { + if (strchr(forbid, *cp) != NULL) + return (0); + } + return (1); +} + const char * sig2name(int signo) { diff --git a/tmux.h b/tmux.h index 39677edb..206ee4e1 100644 --- a/tmux.h +++ b/tmux.h @@ -96,6 +96,12 @@ struct winlink; #define TMUX_LOCK_CMD "lock -np" #endif +/* Forbidden characters in names. */ +#define WINDOW_NAME_FORBID ":." +#define WINDOW_NAME_FORBID_EXT ":.#" +#define SESSION_NAME_FORBID ":." +#define SESSION_NAME_FORBID_EXT ":.#" + /* Minimum and maximum layout cell size, NOT including border lines. */ #define PANE_MINIMUM 1 #define PANE_MAXIMUM 10000 @@ -2423,6 +2429,7 @@ void setblocking(int, int); char *shell_argv0(const char *, int); uint64_t get_timer(void); char *clean_name(const char *, const char *); +int check_name(const char *, const char *); const char *sig2name(int); const char *find_cwd(void); const char *find_home(void); @@ -3394,8 +3401,8 @@ void screen_reset_hyperlinks(struct screen *); void screen_set_default_cursor(struct screen *, struct options *); void screen_set_cursor_style(u_int, enum screen_cursor_style *, int *); void screen_set_cursor_colour(struct screen *, int); -int screen_set_title(struct screen *, const char *); -int screen_set_path(struct screen *, const char *); +int screen_set_title(struct screen *, const char *, int); +int screen_set_path(struct screen *, const char *, int); void screen_push_title(struct screen *); void screen_pop_title(struct screen *); void screen_set_progress_bar(struct screen *, enum progress_bar_state, int); @@ -3488,7 +3495,7 @@ int window_pane_key(struct window_pane *, struct client *, struct mouse_event *); void window_pane_paste(struct window_pane *, key_code, char *, size_t); -int window_pane_visible(struct window_pane *); +int window_pane_is_visible(struct window_pane *); int window_pane_exited(struct window_pane *); u_int window_pane_search(struct window_pane *, const char *, int, int); @@ -3502,7 +3509,7 @@ void window_pane_stack_push(struct window_panes *, struct window_pane *); void window_pane_stack_remove(struct window_panes *, struct window_pane *); -void window_set_name(struct window *, const char *); +void window_set_name(struct window *, const char *, const char *); void window_add_ref(struct window *, const char *); void window_remove_ref(struct window *, const char *); void winlink_clear_flags(struct winlink *); diff --git a/window.c b/window.c index 76830c80..33ac6e1f 100644 --- a/window.c +++ b/window.c @@ -409,11 +409,11 @@ window_remove_ref(struct window *w, const char *from) } void -window_set_name(struct window *w, const char *new_name) +window_set_name(struct window *w, const char *new_name, const char *forbid) { char *name; - name = clean_name(new_name, "#"); + name = clean_name(new_name, forbid); if (name != NULL) { free(w->name); w->name = name; @@ -642,7 +642,7 @@ window_get_active_at(struct window *w, u_int x, u_int y) * bottom border. */ TAILQ_FOREACH(wp, &w->z_index, zentry) { - if (!window_pane_visible(wp) || + if (!window_pane_is_visible(wp) || window_pane_is_floating(wp)) continue; @@ -656,7 +656,7 @@ window_get_active_at(struct window *w, u_int x, u_int y) } TAILQ_FOREACH(wp, &w->z_index, zentry) { - if (!window_pane_visible(wp)) + if (!window_pane_is_visible(wp)) continue; window_pane_full_size_offset(wp, &xoff, &yoff, &sx, &sy); if (!window_pane_is_floating(wp)) { @@ -1096,7 +1096,7 @@ window_pane_create(struct window *w, u_int sx, u_int sy, u_int hlimit) style_ranges_init(&wp->border_status_line.ranges); if (gethostname(host, sizeof host) == 0) - screen_set_title(&wp->base, host); + screen_set_title(&wp->base, host, 0); return (wp); } @@ -1363,7 +1363,7 @@ window_pane_copy_paste(struct window_pane *wp, char *buf, size_t len) TAILQ_EMPTY(&loop->modes) && loop->fd != -1 && (~loop->flags & PANE_INPUTOFF) && - window_pane_visible(loop) && + window_pane_is_visible(loop) && options_get_number(loop->options, "synchronize-panes")) { log_debug("%s: %.*s", __func__, (int)len, buf); bufferevent_write(loop->event, buf, len); @@ -1381,7 +1381,7 @@ window_pane_copy_key(struct window_pane *wp, key_code key) TAILQ_EMPTY(&loop->modes) && loop->fd != -1 && (~loop->flags & PANE_INPUTOFF) && - window_pane_visible(loop) && + window_pane_is_visible(loop) && options_get_number(loop->options, "synchronize-panes")) input_key_pane(loop, key, NULL); } @@ -1438,7 +1438,7 @@ window_pane_key(struct window_pane *wp, struct client *c, struct session *s, } int -window_pane_visible(struct window_pane *wp) +window_pane_is_visible(struct window_pane *wp) { if (~wp->window->flags & WINDOW_ZOOMED) return (1);