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

interfaces: use security tags to index security snippets. #1332

Merged
merged 6 commits into from Jun 21, 2016

Conversation

kyrofa
Copy link
Contributor

@kyrofa kyrofa commented Jun 15, 2016

This allows the same data structure to be used for both apps and hooks, since security tags need to be unique between them already. Also update the apparmor, dbus, seccomp, udev, and mount security systems to work with the new snippets. This makes progress on LP: #1586465.

@kyrofa kyrofa force-pushed the feature/1586465/security_tag_snippets branch from 1145484 to 7c54b37 Compare June 15, 2016 14:18
Updates: #1586465

Signed-off-by: Kyle Fazzari <kyle@canonical.com>
This allows the same data structure to be used for both apps and hooks,
since security tags need to be unique between them already. Also update
the apparmor, dbus, seccomp, and udev security systems to work with the
new snippets.

Updates: #1586465

Signed-off-by: Kyle Fazzari <kyle@canonical.com>
@kyrofa kyrofa force-pushed the feature/1586465/security_tag_snippets branch from 7c54b37 to 79e1222 Compare June 17, 2016 20:11
@zyga
Copy link
Collaborator

zyga commented Jun 20, 2016

This looks good. +1 To land it

@niemeyer
Copy link
Contributor

LGTM once tests are fixed.

@kyrofa kyrofa merged commit f59264c into snapcore:master Jun 21, 2016
@kyrofa kyrofa deleted the feature/1586465/security_tag_snippets branch June 21, 2016 14:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants