c7b90dc4d
Denys Vlasenko
2021-12-12
Subject
uudecode: special-case "/dev/stdout", closes 14241
Binary
No binary (build failed)
Architecture & Compiler Flags
—
Build Log
(log yok)
Commit Diff GitHub →
coreutils/uudecode.c+10 -1
| @@ -155,7 +155,16 @@ int uudecode_main(int argc UNUSED_PARAM, char **argv) | |
| break; | |
| } | |
| dst_stream = stdout; | |
| - | if (NOT_LONE_DASH(outname)) { |
| + | if (NOT_LONE_DASH(outname) |
| + | /* https://pubs.opengroup.org/onlinepubs/9699919799/utilities/uudecode.html |
| + | * https://pubs.opengroup.org/onlinepubs/9699919799/utilities/uuencode.html |
| + | * The above says that output file name specified in input file |
| + | * or overridden by -o OUTFILE can be special "/dev/stdout" string. |
| + | * This usually works "implicitly": many systems have /dev/stdout. |
| + | * If ENABLE_DESKTOP, support that explicitly: |
| + | */ |
| + | && (!ENABLE_DESKTOP || strcmp(outname, "/dev/stdout") != 0) |
| + | ) { |
| dst_stream = xfopen_for_write(outname); | |
| fchmod(fileno(dst_stream), mode & (S_IRWXU | S_IRWXG | S_IRWXO)); | |
| } | |
docs/posix_conformance.txt+1 -1
| @@ -690,7 +690,7 @@ uniq Busybox specific options: | |
| uudecode POSIX options | |
| option | exists | compliant | remarks | |
| - | -o outfile | no | no | |
| + | -o outfile | yes | no | |
| uudecode Busybox specific options: None | |
| uuencode POSIX options | |