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

Handle autofs entries in /etc/mtab #1329

Merged
merged 2 commits into from May 20, 2019
Merged

Conversation

jan-cerny
Copy link
Member

Some file systems can be mounted using autofs, which should be
considered during analysis of /etc/mtab. For more details,
please see the comment introduced in this patch.

Some file systems can be mounted using autofs, which should be
considered during analysis of /etc/mtab.F or more details,
please see the comment introduced in this patch.
@jan-cerny jan-cerny added this to the 1.2.18 milestone May 17, 2019
Adds a simple unit test that checks whether autofs entries in
/etc/mtab are not considered local.
@matejak
Copy link
Contributor

matejak commented May 20, 2019

Looks great, thank you for the test!

@matejak matejak merged commit c4c4e3c into OpenSCAP:maint-1.2 May 20, 2019
@jan-cerny jan-cerny deleted the mtab branch May 20, 2019 14:53
jan-cerny added a commit to jan-cerny/openscap that referenced this pull request May 23, 2019
The test uses a fake `mtab` file which contains 1 entry for a local
filesystem, 1 entry for a direct autofs map and 1 entry for a NFS
system mounted using autofs. By parsing the `mtab` file only 1 local
filesystem should be found. It will help us to test
OpenSCAP#1329
jan-cerny added a commit to jan-cerny/openscap that referenced this pull request May 23, 2019
The test uses a fake `mtab` file which contains 1 entry for a local
filesystem, 1 entry for a direct autofs map and 1 entry for a NFS
system mounted using autofs. By parsing the `mtab` file only 1 local
filesystem should be found. It will help us to test
OpenSCAP#1329
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.

None yet

2 participants