CommandFilter does not allow validating full path command
Bug #1956606 reported by
David Vallee Delisle
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
oslo.rootwrap |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
If we use this configuration:
sftp_server: CommandFilter, /usr/libexec/
rootwrap can't validate full paths
~~~
# /usr/bin/
/usr/bin/
~~~
Changed in oslo.rootwrap: | |
status: | New → In Progress |
To post a comment you must log in.
Reviewed: https:/ /review. opendev. org/c/openstack /oslo.rootwrap/ +/823571 /opendev. org/openstack/ oslo.rootwrap/ commit/ 1b1b960d0d6b0dd f63c5b0f9c6aa7c 10a1914073
Committed: https:/
Submitter: "Zuul (22348)"
Branch: master
commit 1b1b960d0d6b0dd f63c5b0f9c6aa7c 10a1914073
Author: David Vallee Delisle <email address hidden>
Date: Wed Jan 5 12:36:04 2022 -0500
CommandFilter should allow exec from full path
The current logic prevents from using a full path as argument.
We can't just compare basename to basename as it would allow passing
bogus paths. We need to make sure that passing a full path will compare
to the config's full path.
Closes-Bug: #1956606
Change-Id: I76094065de5b37 f59a2500fbce7f5 00ada9915da