From aefb9a5bc41f92227e4bffa050caca0270b450ba Mon Sep 17 00:00:00 2001 From: Junegunn Choi Date: Mon, 10 Mar 2025 18:16:20 +0900 Subject: [PATCH] Nullify unwanted FZF_DEFAULT_* variables in tmux popup Fix #4298 --- src/proxy.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/src/proxy.go b/src/proxy.go index 660b0f94..53a3c265 100644 --- a/src/proxy.go +++ b/src/proxy.go @@ -90,9 +90,9 @@ func runProxy(commandPrefix string, cmdBuilder func(temp string, needBash bool) } } - // To ensure that the options are processed by a POSIX-compliant shell, - // we need to write the command to a temporary file and execute it with sh. - var exports []string + // * Write the command to a temporary file and run it with sh to ensure POSIX compliance. + // * Nullify FZF_DEFAULT_* variables as tmux popup may inject them even when undefined. + exports := []string{"FZF_DEFAULT_COMMAND=", "FZF_DEFAULT_OPTS=", "FZF_DEFAULT_OPTS_FILE="} needBash := false if withExports { validIdentifier := regexp.MustCompile(`^[a-zA-Z_][a-zA-Z0-9_]*$`)