github-mirror/fish-shell
2
0
Fork 0
mirror of https://github.com/fish-shell/fish-shell.git synced 2025-01-19 18:22:45 +08:00
Code Issues Actions Packages Projects Releases Wiki Activity

Suppress uvar error messages due to permissions or file not found

Browse Source 
su does not reset XDG_RUNTIME_DIR, which means that XDG_RUNTIME_DIR
may point to directories that the user does not have permission
to access. Similarly there is no guarantee that XDG_RUNTIME_DIR
points to a directory that actually exists. Rather than try to
handle these issues, we simply ignore them, effectively disabling
realtime uvar notifications. Fixes #1955.
This commit is contained in:
ridiculousfish 2015-06-01 19:20:25 -07:00
parent 3a190bbe49
commit d31eceddaf
1 changed files with 6 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
env_universal_common.cpp
View File

@ -1376,7 +1376,12 @@ class universal_notifier_named_pipe_t : public universal_notifier_t
{
// Maybe open failed, maybe mkfifo failed
int err = errno;
report_error(err, L"Unable to make or open a FIFO for universal variables with path '%ls'", vars_path.c_str());
// We explicitly do NOT report an error for ENOENT or EACCESS
// This works around #1955, where $XDG_RUNTIME_DIR may get a bogus value under suc
if (err != ENOENT && err != EPERM) {
report_error(err, L"Unable to make or open a FIFO for universal variables with path '%ls'", vars_path.c_str());
}
pipe_fd= -1;
}
else
{