Reset the umask right after fopen to avoid leaving it changed on error, noticed

by deraadt@.
This commit is contained in:
Nicholas Marriott 2009-10-26 21:13:06 +00:00
parent 6dc6333323
commit 539c73bdb1

View File

@ -70,6 +70,7 @@ cmd_save_buffer_exec(struct cmd *self, struct cmd_ctx *ctx)
f = fopen(data->arg, "ab");
else
f = fopen(data->arg, "wb");
umask(mask);
if (f == NULL) {
ctx->error(ctx, "%s: %s", data->arg, strerror(errno));
return (-1);
@ -82,7 +83,6 @@ cmd_save_buffer_exec(struct cmd *self, struct cmd_ctx *ctx)
}
fclose(f);
umask(mask);
return (0);
}