emo/busybox
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

uudecode: special-case "/dev/stdout", closes 14241

Browse Source 
function                                             old     new   delta
uudecode_main                                        295     322     +27

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2021-12-12 00:34:15 +01:00
parent 27df6aeef2
commit c7b90dc4d1
2 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
coreutils/uudecode.c
View File

@ -155,7 +155,16 @@ int uudecode_main(int argc UNUSED_PARAM, char **argv)
break; break;
} }
dst_stream = stdout; 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); dst_stream = xfopen_for_write(outname);
fchmod(fileno(dst_stream), mode & (S_IRWXU | S_IRWXG | S_IRWXO)); fchmod(fileno(dst_stream), mode & (S_IRWXU | S_IRWXG | S_IRWXO));
} }

2
docs/posix_conformance.txt
View File

@ -690,7 +690,7 @@ uniq Busybox specific options:
uudecode POSIX options uudecode POSIX options
option | exists | compliant | remarks option | exists | compliant | remarks
-o outfile | no | no | -o outfile | yes | no |
uudecode Busybox specific options: None uudecode Busybox specific options: None
uuencode POSIX options uuencode POSIX options