Move Xauth to just after env(). Stop creating ~/.xsession-errors.
authorAuke Kok <auke-jan.h.kok@intel.com>
Mon, 2 Apr 2012 18:02:34 +0000 (11:02 -0700)
committerAuke Kok <auke-jan.h.kok@intel.com>
Mon, 2 Apr 2012 18:02:34 +0000 (11:02 -0700)
All log output just goes into the journal.

src/main.c
src/user.c
src/xserver.c

index 19161c1..9471653 100644 (file)
@@ -108,8 +108,6 @@ int main(int argc, char **argv)
        
        set_tty();
 
-       setup_xauth();
-
        setup_pam_session();
 
        switch_to_user();
index 69fb089..8d06414 100644 (file)
@@ -143,6 +143,8 @@ void switch_to_user(void)
 
        ret = chdir(pass->pw_dir);
 
+       setup_xauth();
+
        fp = fopen(user_xauth_path, "w");
        if (fp) {
                if (XauWriteAuth(fp, &x_auth) != 1)
@@ -150,18 +152,6 @@ void switch_to_user(void)
                fclose(fp);
        }
 
-       /* redirect further IO to .xsession-errors */
-       snprintf(fn, PATH_MAX, "%s/.xsession-errors", pass->pw_dir);
-       fp = fopen(fn, "w");
-       if (fp) {
-               fclose(fp);
-               /* xserver.c already truncates this file, so append */
-               fp = freopen(fn, "a", stdout);
-               fp = freopen(fn, "a", stderr);
-       } else {
-               lprintf("Unable to open \"%s\n\" for writing", fn);
-       }
-
        d_out();
 }
 
index 9ed7fc8..32b7fe5 100644 (file)
@@ -304,17 +304,6 @@ void start_X_server(void)
        }
        lprintf("starting X server with: \"%s\"", all);
 
-       /* redirect further IO to .xsession-errors */
-       snprintf(fn, PATH_MAX, "%s/.xsession-errors", pass->pw_dir);
-       fp = fopen(fn, "w");
-       if (fp) {
-               fclose(fp);
-               fp = freopen(fn, "w", stdout);
-               fp = freopen(fn, "w", stderr);
-       } else {
-               lprintf("Unable to open \"%s\n\" for writing", fn);
-       }
-
        execv(ptrs[0], ptrs);
 
        d_out();