Skip to content

squid: client: ll_walk will process absolute paths as relative#62499

Merged
joscollin merged 2 commits intoceph:squidfrom
batrick:wip-70663-squid
Sep 23, 2025
Merged

squid: client: ll_walk will process absolute paths as relative#62499
joscollin merged 2 commits intoceph:squidfrom
batrick:wip-70663-squid

Conversation

@batrick
Copy link
Member

@batrick batrick commented Mar 25, 2025

backport tracker: https://tracker.ceph.com/issues/70663


backport of #62406
parent tracker: https://tracker.ceph.com/issues/70573

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/main/src/script/ceph-backport.sh

@batrick batrick added this to the v16.2.15 milestone Mar 25, 2025
@batrick batrick added the cephfs Ceph File System label Mar 25, 2025
@batrick
Copy link
Member Author

batrick commented Apr 17, 2025

jenkins test api

@batrick
Copy link
Member Author

batrick commented Apr 17, 2025

jenkins test windows

@batrick
Copy link
Member Author

batrick commented Apr 17, 2025

jenkins test make check

batrick and others added 2 commits April 17, 2025 12:23
Fixes: https://tracker.ceph.com/issues/70573
Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
(cherry picked from commit 0306789)
ll_walk was wrongly coded to make the input file path relative to ino=0. That inode
does not exist and isn't considered the root inode.

Fixes: https://tracker.ceph.com/issues/70573
Fixes: e08210d
Signed-off-by: Patrick Donnelly <pdonnell@ibm.com>
(cherry picked from commit 25eb99d)
@joscollin
Copy link
Member

This PR is under test in https://tracker.ceph.com/issues/72883.

Copy link
Member

@joscollin joscollin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@joscollin joscollin merged commit b4e1687 into ceph:squid Sep 23, 2025
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants