]> git.ipfire.org Git - thirdparty/coreutils.git/commitdiff
maint: mkfifo: reduce variable scope
authorCollin Funk <collin.funk1@gmail.com>
Sun, 4 Jan 2026 21:46:41 +0000 (13:46 -0800)
committerCollin Funk <collin.funk1@gmail.com>
Sun, 4 Jan 2026 21:46:41 +0000 (13:46 -0800)
* src/mkfifo.c (main): Declare variables where they are used instead of
at the start of a block.

src/mkfifo.c

index 4a39b387faf02fa5cdfdb4c583317d7dce974bd1..4c434bc07b6b2cf42edcdde40e82f9f2e3f6ffce 100644 (file)
@@ -74,10 +74,7 @@ Create named pipes (FIFOs) with the given NAMEs.\n\
 int
 main (int argc, char **argv)
 {
-  mode_t newmode;
   char const *specified_mode = nullptr;
-  int exit_status = EXIT_SUCCESS;
-  int optc;
   char const *scontext = nullptr;
   struct selabel_handle *set_security_context = nullptr;
 
@@ -89,6 +86,7 @@ main (int argc, char **argv)
 
   atexit (close_stdout);
 
+  int optc;
   while ((optc = getopt_long (argc, argv, "m:Z", longopts, nullptr)) != -1)
     {
       switch (optc)
@@ -148,14 +146,13 @@ main (int argc, char **argv)
                quote (scontext));
     }
 
-  newmode = MODE_RW_UGO;
+  mode_t newmode = MODE_RW_UGO;
   if (specified_mode)
     {
-      mode_t umask_value;
       struct mode_change *change = mode_compile (specified_mode);
       if (!change)
         error (EXIT_FAILURE, 0, _("invalid mode"));
-      umask_value = umask (0);
+      mode_t umask_value = umask (0);
       umask (umask_value);
       newmode = mode_adjust (newmode, false, umask_value, change, nullptr);
       free (change);
@@ -164,6 +161,7 @@ main (int argc, char **argv)
                _("mode must specify only file permission bits"));
     }
 
+  int exit_status = EXIT_SUCCESS;
   for (; optind < argc; ++optind)
     {
       if (set_security_context)