summaryrefslogtreecommitdiff
path: root/os
diff options
context:
space:
mode:
authorAntoine Martin <antoine@nagafix.co.uk>2018-03-28 15:16:02 -0400
committerAdam Jackson <ajax@redhat.com>2018-03-28 15:16:02 -0400
commitc95361465ee98dd97a99778d8a31be88fe9e4c4f (patch)
tree1d0b1a3eee31119560163623f0c6f9bad88ff15a /os
parentb355e55f500c18d55d0ac241e5f7e826ed62f555 (diff)
downloadxserver-c95361465ee98dd97a99778d8a31be88fe9e4c4f.tar.gz
os: Fix -logfile when used with -displayfd
Trivial way to reproduce the bug: $ Xorg -logfile /tmp/mylog -config /etc/xpra/xorg.conf -displayfd 2 The server then moans: Failed to rename log file "/tmp/mylog" to "/tmp/mylog": No such file or directory And the log file is created but immediately renamed to "/tmp/mylog.old". This is caused by the changes to the log file handling introduced by this commit: https://cgit.freedesktop.org/xorg/xserver/commit/?id=edcb6426f20c3be5dd5f50b76a686754aef2f64e To fix this, only rename the logfile if the log filename contains the magic substitution string "%s". Signed-off-by: Antoine Martin <antoine@nagafix.co.uk> Reviewed-by: Alan Coopersmith <alan.coopersmith@oracle.com>
Diffstat (limited to 'os')
-rw-r--r--os/log.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/os/log.c b/os/log.c
index 91e55a532..a3b28ccb4 100644
--- a/os/log.c
+++ b/os/log.c
@@ -296,7 +296,7 @@ LogInit(const char *fname, const char *backup)
void
LogSetDisplay(void)
{
- if (saved_log_fname) {
+ if (saved_log_fname && strstr(saved_log_fname, "%s")) {
char *logFileName;
logFileName = LogFilePrep(saved_log_fname, saved_log_backup, display);