diff options
author | Rob Landley <rob@landley.net> | 2006-08-04 23:16:31 +0400 |
---|---|---|
committer | Rob Landley <rob@landley.net> | 2006-08-04 23:16:31 +0400 |
commit | 27d07c9ce2f07cf85116455e356ba6c4b57f86fe (patch) | |
tree | 92c46c98529619d8cfc8d4762671041b4fb69258 /archival/unzip.c | |
parent | daf58ef61a5f78fa54958c0f7a53d13d792e0bb9 (diff) |
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.
Diffstat (limited to 'archival/unzip.c')
-rw-r--r-- | archival/unzip.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/archival/unzip.c b/archival/unzip.c index 68083e987..2585cbf3f 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); |