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

Fix clam scanning path issues when using mod_vroot #21

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

offsides
Copy link
Contributor

When using mod_vroot, the chroot path gets prepended to the scan path when it isn't needed. This fixes that problem. (I can't remember exactly why it was that this is needed because it's been so long since I did my investigation, and I had forgotten about submitting a PR for it, but without it doing non-stream scans with mod_vroot doesn't work because the path passed to clamd looks like /vroot/path/vroot/path/jailed/path rather than just /vroot/path/jailed/path).

When using mod_vroot, the chroot path gets prepended to the scan path when it isn't needed.  This fixes that problem.
@offsides
Copy link
Contributor Author

offsides commented Apr 4, 2024

I looked at the new FSIO handler for the real path mentioned in proftpd #1780, but I'm not sure how to apply it here instead of this fix. But it doesn't break this, so I'll leave this here as a solution to the mod_vroot pathing issues unless/until someone comes up with a better way to do it.

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.

1 participant