diff options
| author |  David Sterba <dsterba@suse.com>
| 2024-01-24 22:29:46 +0100 |
|---|---|---|
| committer | David Sterba <dsterba@suse.com>
| 2024-03-04 16:24:50 +0100 |
| commit | 5b9579893a216de3f09018cbc9369a849a74bf41 (patch) | |
| tree | 19ec03c6e9bfe394b58de925340e5e483a957472 /fs/btrfs/backref.c | |
| parent | btrfs: push errors up from add_async_extent() (diff) | |
| download | linux-5b9579893a216de3f09018cbc9369a849a74bf41.tar.gz linux-5b9579893a216de3f09018cbc9369a849a74bf41.tar.bz2 linux-5b9579893a216de3f09018cbc9369a849a74bf41.zip | |
btrfs: update comment and drop assertion in extent item lookup in find_parent_nodes()
Same comment was added to this type of error, unify that and drop the
assertion as we'd find out quickly that something is wrong after
returning -EUCLEAN.
Signed-off-by: David Sterba <dsterba@suse.com>
Diffstat (limited to 'fs/btrfs/backref.c')
| -rw-r--r-- | fs/btrfs/backref.c | 6 |
1 files changed, 4 insertions, 2 deletions
diff --git a/fs/btrfs/backref.c b/fs/btrfs/backref.c index beed7e459dab..0fa27ed802f6 100644 --- a/fs/btrfs/backref.c +++ b/fs/btrfs/backref.c @@ -1435,8 +1435,10 @@ again: if (ret < 0) goto out; if (ret == 0) { - /* This shouldn't happen, indicates a bug or fs corruption. */ - ASSERT(ret != 0); + /* + * Key with offset -1 found, there would have to exist an extent + * item with such offset, but this is out of the valid range. + */ ret = -EUCLEAN; goto out; } |