diff --git a/src/ops.rs b/src/ops.rs index 3dc457aa64..44c1f87027 100644 --- a/src/ops.rs +++ b/src/ops.rs @@ -594,21 +594,12 @@ fn op_open( open_options.read(true); } "r+" => { - if let Err(e) = state.check_write(&filename_str) { - return odd_future(e); - } open_options.read(true).write(true); } "w" => { - if let Err(e) = state.check_write(&filename_str) { - return odd_future(e); - } open_options.create(true).write(true).truncate(true); } "w+" => { - if let Err(e) = state.check_write(&filename_str) { - return odd_future(e); - } open_options .read(true) .create(true) @@ -616,27 +607,15 @@ fn op_open( .truncate(true); } "a" => { - if let Err(e) = state.check_write(&filename_str) { - return odd_future(e); - } open_options.create(true).append(true); } "a+" => { - if let Err(e) = state.check_write(&filename_str) { - return odd_future(e); - } open_options.read(true).create(true).append(true); } "x" => { - if let Err(e) = state.check_write(&filename_str) { - return odd_future(e); - } open_options.create_new(true).write(true); } "x+" => { - if let Err(e) = state.check_write(&filename_str) { - return odd_future(e); - } open_options.create_new(true).read(true).write(true); } &_ => { @@ -644,6 +623,13 @@ fn op_open( } } + if mode != "r" { + // Write permission is needed except "r" mode + if let Err(e) = state.check_write(&filename_str) { + return odd_future(e); + } + } + let op = open_options .open(filename) .map_err(DenoError::from)