Am 08.11.2012 14:41, schrieb Jon TURNEY: > If we are not backing up logfiles, remove the old logfile before trying to > write > a new logfile, as otherwise the operation may fail if the previous logfile was > created by a different user. >
what are the permissions of that file ? normally you can not remove any file that is not worn by your. I would print an advive to use rm /logfile if the creation failed. just my 2 cents re, wh > This change is useful when: > - The DDX doesn't use the logfile backup mechanism (i.e. not Xorg) > - The DDX is run by a non-root user, and then by a different non-root user > - The logfile directory doesn't have the restricted-deletion flag set > > Signed-off-by: Jon TURNEY <jon.tur...@dronecode.org.uk> > Acked-by: Yaakov Selkowitz <yselkow...@users.sourceforge.net> > Reviewed-by: Yaakov Selkowitz <yselkow...@users.sourceforge.net> > --- > os/log.c | 3 +++ > 1 files changed, 3 insertions(+), 0 deletions(-) > > diff --git a/os/log.c b/os/log.c > index 4820e9a..15271eb 100644 > --- a/os/log.c > +++ b/os/log.c > @@ -216,6 +216,9 @@ LogInit(const char *fname, const char *backup) > free(oldLog); > } > } > + else { > + unlink(logFileName); > + } > if ((logFile = fopen(logFileName, "w")) == NULL) > FatalError("Cannot open log file \"%s\"\n", logFileName); > setvbuf(logFile, NULL, _IONBF, 0); _______________________________________________ xorg-devel@lists.x.org: X.Org development Archives: http://lists.x.org/archives/xorg-devel Info: http://lists.x.org/mailman/listinfo/xorg-devel