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.
This commit is contained in:
@@ -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);
|
||||
}
|
||||
}
|
||||
|
||||
|
Reference in New Issue
Block a user