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

auparse: do not interpret fE as a capability field #19

Conversation

rgbriggs
Copy link
Member

The file effective capability is a boolean. It is being interpreted as the
capability "chown" by auparse. Just print its raw value.

An example from an execve syscall:
type=BPRM_FCAPS msg=audit(03/07/2017 17:29:56.494:969) : fver=2 fp=sys_admin fi=none fe=chown old_pp=none old_pi=none old_pe=none new_pp=sys_admin new_pi=none new_pe=sys_admin

Fixed:
type=BPRM_FCAPS msg=audit(03/07/2017 17:29:56.494:969) : fver=2 fp=sys_admin fi=none fe=1 old_pp=none old_pi=none old_pe=none new_pp=sys_admin new_pi=none new_pe=sys_admin

See: #17
Signed-off-by: Richard Guy Briggs rgb@redhat.com

@rgbriggs
Copy link
Member Author

Correction, See: #18

The file effective capability is a boolean.  It is being interpreted as the
capability "chown" by auparse.  Just print its raw value.

An example from an execve syscall:
type=BPRM_FCAPS msg=audit(03/07/2017 17:29:56.494:969) : fver=2 fp=sys_admin fi=none fe=chown old_pp=none old_pi=none old_pe=none new_pp=sys_admin new_pi=none new_pe=sys_admin

Fixed:
type=BPRM_FCAPS msg=audit(03/07/2017 17:29:56.494:969) : fver=2 fp=sys_admin fi=none fe=1 old_pp=none old_pi=none old_pe=none new_pp=sys_admin new_pi=none new_pe=sys_admin

See: linux-audit#18
Signed-off-by: Richard Guy Briggs <rgb@redhat.com>
@rgbriggs rgbriggs force-pushed the ghau18-do-not-interpret-boolean-cap-fe branch from 74c9e3a to 9bcd211 Compare April 20, 2017 14:21
@rgbriggs rgbriggs self-assigned this Apr 20, 2017
@rgbriggs rgbriggs added this to the audit-2.7 milestone Apr 20, 2017
@rgbriggs rgbriggs requested a review from stevegrubb May 6, 2017 14:22
@rgbriggs
Copy link
Member Author

Patch merged upstream via mailing list.

@rgbriggs rgbriggs closed this Jun 13, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant