Skip to content
This repository was archived by the owner on Sep 18, 2020. It is now read-only.

Conversation

@glevand
Copy link
Contributor

@glevand glevand commented May 18, 2018

Fixes the audit rules for both boards and fixes run-time errors on arm64.

# For info on configuring audit rules see the auditctl man page and
# https://github.com/torvalds/linux/blob/master/include/uapi/linux/audit.h

-a exclude,always -F msgtype<1400 -F msgtype>1499 -F msgtype!=CONFIG_CHANGE
Copy link
Contributor

Choose a reason for hiding this comment

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

Doesn't this mean msgtype<1400 && msgtype>1499 which will never match?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@dm0- Well, yes and no...
It seems the > and < operators don't work with the msgtype field, so auditctl will add the rule successfully, but it doesn't seem to have any effect. I was seeing the AVC messages in my tests so thought things were OK. I'll push out an updated patch.

@glevand glevand force-pushed the for-merge-audit branch 2 times, most recently from 5454fd6 to 73825f0 Compare May 22, 2018 00:00
glevand added 3 commits May 23, 2018 10:08
The existing rules were not giving the correct message filter.  Fix that
and add some comments for users wishing to modify the default rules.

Signed-off-by: Geoff Levand <geoff@infradead.org>
The --with-arm and --with-aarch64 configure script options need to be
set when building for the arm64-usr target.

Fixes runtime errors like these when adding machine/arch specific rules:

    # auditctl -a exit,always -S fchmod
    Error detecting machine type

    # auditctl -a exit,always -F arch=aarch64 -S fchmod
    arch elf mapping not found

Signed-off-by: Geoff Levand <geoff@infradead.org>
@glevand glevand force-pushed the for-merge-audit branch from 73825f0 to 36692e0 Compare May 23, 2018 17:08
@glevand
Copy link
Contributor Author

glevand commented May 23, 2018

Rebased to latest.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants