lpd: small shrink
lpd_main 761 747 -14
diff --git a/printutils/lpd.c b/printutils/lpd.c
index f4c902c..9533244 100644
--- a/printutils/lpd.c
+++ b/printutils/lpd.c
@@ -87,43 +87,6 @@
return str;
}
-// we can use leaky setenv since we are about to exec or exit
-static void exec_helper(char **filenames, char **argv) ATTRIBUTE_NORETURN;
-static void exec_helper(char **filenames, char **argv)
-{
- char *p, *q;
- char var[2];
-
- var[1] = '\0';
-
- // read and delete ctrlfile
- q = xmalloc_open_read_close(filenames[0], NULL);
- unlink(filenames[0]);
- // provide datafile name
- xsetenv("DATAFILE", filenames[1]);
- // parse control file by "\n"
- while ((p = strchr(q, '\n')) != NULL
- && isalpha(*q)
- ) {
- *p++ = '\0';
- // q is a line of <SYM><VALUE>,
- // we are setting environment string <SYM>=<VALUE>.
- // Ignoring "l<datafile>", exporting others:
- if (*q != 'l') {
- var[0] = *q++;
- xsetenv(var, q);
- }
- // next line, plz!
- q = p;
- }
- // we are the helper, we wanna be silent.
- // this call reopens stdio fds to "/dev/null"
- // (no daemonization is done)
- bb_daemonize_or_rexec(DAEMON_DEVNULL_STDIO | DAEMON_ONLY_SANITIZE, NULL);
- BB_EXECVP(*argv, argv);
- exit(127); // it IS error if helper cannot be executed!
-}
-
static char *xmalloc_read_stdin(void)
{
// SECURITY:
@@ -184,12 +147,42 @@
// N.B. we bail out on any error
s = xmalloc_read_stdin();
if (!s) { // (probably) EOF
- if (spooling /* && 7 != spooling - always true */) {
- // we didn't see both ctrlfile & datafile!
- goto err_exit;
+ char *p, *q, var[2];
+
+ // non-spooling mode or no spool helper specified
+ if (!spooling || !*argv)
+ return EXIT_SUCCESS; // the only non-error exit
+ // spooling mode but we didn't see both ctrlfile & datafile
+ if (spooling != 7)
+ goto err_exit; // reject job
+
+ // spooling mode and spool helper specified -> exec spool helper
+ // (we exit 127 if helper cannot be executed)
+ var[1] = '\0';
+ // read and delete ctrlfile
+ q = xmalloc_open_read_close(filenames[0], NULL);
+ unlink(filenames[0]);
+ // provide datafile name
+ // we can use leaky setenv since we are about to exec or exit
+ xsetenv("DATAFILE", filenames[1]);
+ // parse control file by "\n"
+ while ((p = strchr(q, '\n')) != NULL && isalpha(*q)) {
+ *p++ = '\0';
+ // q is a line of <SYM><VALUE>,
+ // we are setting environment string <SYM>=<VALUE>.
+ // Ignoring "l<datafile>", exporting others:
+ if (*q != 'l') {
+ var[0] = *q++;
+ xsetenv(var, q);
+ }
+ q = p; // next line
}
- // one of only two non-error exits
- return EXIT_SUCCESS;
+ // helper should not talk over network.
+ // this call reopens stdio fds to "/dev/null"
+ // (no daemonization is done)
+ bb_daemonize_or_rexec(DAEMON_DEVNULL_STDIO | DAEMON_ONLY_SANITIZE, NULL);
+ BB_EXECVP(*argv, argv);
+ exit(127);
}
// validate input.
@@ -269,14 +262,6 @@
free(s);
close(fd); // NB: can do close(-1). Who cares?
-
- // spawn spool helper and exit if all files are dumped
- if (7 == spooling && *argv) {
- // signal OK
- safe_write(STDOUT_FILENO, "", 1);
- // does not return (exits 0)
- exec_helper(filenames, argv);
- }
} // while (1)
err_exit: