diff options
| author | Filipe Manana <fdmanana@suse.com> | 2025-02-05 14:54:07 +0300 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com> | 2025-03-18 22:35:45 +0300 |
| commit | c727371879488449a71d8ee7e199b30290ad0953 (patch) | |
| tree | 65f21e0f6a965afe220b6ea12fc3396b514d4a7c | |
| parent | 147ff868609b99b48fb67cad54b41372cac23f3b (diff) | |
| download | linux-c727371879488449a71d8ee7e199b30290ad0953.tar.xz | |
btrfs: send: simplify return logic from fs_path_add()
There is no need to have an 'out' label and jump into it since there are
no resource cleanups to perform (release locks, free memory, etc), so
make this simpler by removing the label and goto and instead return
directly.
Signed-off-by: Filipe Manana <fdmanana@suse.com>
Reviewed-by: David Sterba <dsterba@suse.com>
Signed-off-by: David Sterba <dsterba@suse.com>
| -rw-r--r-- | fs/btrfs/send.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/fs/btrfs/send.c b/fs/btrfs/send.c index 535384028cb8..2203745569e0 100644 --- a/fs/btrfs/send.c +++ b/fs/btrfs/send.c @@ -560,11 +560,10 @@ static int fs_path_add(struct fs_path *p, const char *name, int name_len) ret = fs_path_prepare_for_add(p, name_len, &prepared); if (ret < 0) - goto out; + return ret; memcpy(prepared, name, name_len); -out: - return ret; + return 0; } static int fs_path_add_path(struct fs_path *p, struct fs_path *p2) |