Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Document better path resolution code introduced in PR #90 #188

Merged
merged 4 commits into from
Jun 28, 2022

Conversation

tpapagian
Copy link
Member

@tpapagian tpapagian commented Jun 23, 2022

Please review commit by commit.

FIXES #187

Signed-off-by: Anastasios Papagiannis [email protected]

@tpapagian tpapagian requested a review from kkourt June 23, 2022 13:47
@tpapagian tpapagian requested a review from a team as a code owner June 23, 2022 13:47
@tpapagian tpapagian linked an issue Jun 23, 2022 that may be closed by this pull request
@tpapagian tpapagian marked this pull request as draft June 23, 2022 13:50
@tpapagian tpapagian force-pushed the pr/apapag/doc-path-resolution branch 2 times, most recently from ba7e00e to 81b817e Compare June 23, 2022 15:46
@tpapagian tpapagian marked this pull request as ready for review June 23, 2022 17:45
If the buffer does not have the required free space for the dentry name
now we return an error.

This commit copies the part of the dentry name that fits and sets the error
accordingly.

Signed-off-by: Anastasios Papagiannis <[email protected]>
Tested on 5.13, 5.10, 5.4, and 4.19.

Signed-off-by: Anastasios Papagiannis <[email protected]>
Tested on 5.13, 5.10, 5.4, and 4.19.

Signed-off-by: Anastasios Papagiannis <[email protected]>
@tpapagian tpapagian force-pushed the pr/apapag/doc-path-resolution branch from 81b817e to 2438f62 Compare June 27, 2022 06:17
Copy link
Contributor

@kkourt kkourt left a comment

Choose a reason for hiding this comment

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

Thanks!

@kkourt kkourt merged commit a797a06 into main Jun 28, 2022
@kkourt kkourt deleted the pr/apapag/doc-path-resolution branch June 28, 2022 15:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Document path resolution code
3 participants