From: Jim Meyering <jim@meyering.net>
Date: Fri, 18 Jul 2003 08:38:43 +0000 (+0000)
Subject: (usage): Don't call close_stdout here.
X-Git-Tag: v2.22-rc1~347^2~10^2~14
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=f7a168d629faf801308792129d9f0bcf78066117;p=thirdparty%2Futil-linux.git

(usage): Don't call close_stdout here.
(main): Use close_stdout via atexit.
Now `su --version > /dev/full' fails, as it should.
Somehow, the change of 2000-05-07 that purports to fix this
was not checked in.
---

diff --git a/login-utils/su.c b/login-utils/su.c
index 03dc7d8d51..2321039214 100644
--- a/login-utils/su.c
+++ b/login-utils/su.c
@@ -452,7 +452,6 @@ Change the effective user id and group id to that of USER.\n\
 A mere - implies -l.   If USER not given, assume root.\n\
 "), stdout);
       printf (_("\nReport bugs to <%s>.\n"), PACKAGE_BUGREPORT);
-      close_stdout ();
     }
   exit (status);
 }
@@ -474,6 +473,8 @@ main (int argc, char **argv)
   bindtextdomain (PACKAGE, LOCALEDIR);
   textdomain (PACKAGE);
 
+  atexit (close_stdout);
+
   fast_startup = 0;
   simulate_login = 0;
   change_environment = 1;