alex/bcachefs-tools
1
0
Fork 0
mirror of https://github.com/koverstreet/bcachefs-tools.git synced 2025-01-23 00:07:07 +03:00
Code Issues Actions Packages Projects Releases Wiki Activity

fix(subvol): canonicalize requires path to exist

Browse Source 
Which is probably not the case when you want to create a subvolume in
that path.

Signed-off-by: Thomas Mühlbacher <tmuehlbacher@posteo.net>
This commit is contained in:
Thomas Mühlbacher 2024-06-10 23:07:27 +02:00
parent c2354f0326
commit c20f323fdb
1 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/commands/subvolume.rs
View File

@ -1,4 +1,4 @@
use std::path::PathBuf;
use std::{env, path::PathBuf};
use bch_bindgen::c::BCH_SUBVOL_SNAPSHOT_RO;
use clap::{Parser, Subcommand};
@ -42,9 +42,13 @@ pub fn subvolume(argv: Vec<String>) -> i32 {
match cli.subcommands {
Subcommands::Create { targets } => {
for target in targets {
let target = target
.canonicalize()
.expect("unable to canonicalize a target path");
let target = if target.is_absolute() {
target
} else {
env::current_dir()
.map(|p| p.join(target))
.expect("unable to get current directory")
};
if let Some(dirname) = target.parent() {
let fs = unsafe { BcachefsHandle::open(dirname) };
@ -56,7 +60,7 @@ pub fn subvolume(argv: Vec<String>) -> i32 {
Subcommands::Delete { target } => {
let target = target
.canonicalize()
.expect("unable to canonicalize a target path");
.expect("subvolume path does not exist or can not be canonicalized");
if let Some(dirname) = target.parent() {
let fs = unsafe { BcachefsHandle::open(dirname) };