summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorRob Landley2006-08-04 19:16:31 +0000
committerRob Landley2006-08-04 19:16:31 +0000
commit27d07c9ce2f07cf85116455e356ba6c4b57f86fe (patch)
tree92c46c98529619d8cfc8d4762671041b4fb69258
parentdaf58ef61a5f78fa54958c0f7a53d13d792e0bb9 (diff)
downloadbusybox-27d07c9ce2f07cf85116455e356ba6c4b57f86fe.zip
busybox-27d07c9ce2f07cf85116455e356ba6c4b57f86fe.tar.gz
Now that we have xopen3(), it's just plain unclean to have xopen() with
O_CREAT. Two users are still doing it (and thus getting permissions 777), I'm not sure what permissions they should be using but here they are changed to xopen3(). This costs us a dozen or so bytes, but removing the 777 from xopen() should get some of that back.
-rw-r--r--archival/unzip.c2
-rw-r--r--networking/ftpgetput.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/archival/unzip.c b/archival/unzip.c
index 68083e9..2585cbf 100644
--- a/archival/unzip.c
+++ b/archival/unzip.c
@@ -333,7 +333,7 @@ int unzip_main(int argc, char **argv)
overwrite = o_always;
case 'y': /* Open file and fall into unzip */
unzip_create_leading_dirs(dst_fn);
- dst_fd = xopen(dst_fn, O_WRONLY | O_CREAT);
+ dst_fd = xopen(dst_fn, O_WRONLY | O_CREAT, 777);
case -1: /* Unzip */
if (verbosity == v_normal) {
printf(" inflating: %s\n", dst_fn);
diff --git a/networking/ftpgetput.c b/networking/ftpgetput.c
index 767ace9..978601d 100644
--- a/networking/ftpgetput.c
+++ b/networking/ftpgetput.c
@@ -166,7 +166,7 @@ static int ftp_receive(ftp_host_info_t *server, FILE *control_stream,
if (do_continue) {
fd_local = xopen(local_path, O_APPEND | O_WRONLY);
} else {
- fd_local = xopen(local_path, O_CREAT | O_TRUNC | O_WRONLY);
+ fd_local = xopen3(local_path, O_CREAT | O_TRUNC | O_WRONLY, 777);
}
}