interfaces: add /proc/version_signature to appamor template #1612

Closed
wants to merge 1 commit into
from

Conversation

Projects
None yet
4 participants
Contributor

arges commented Aug 1, 2016

Allow access to /proc/version_signature to be able to easily read kernel version
and build number.

Signed-off-by: Chris J Arges chris.j.arges@canonical.com

Contributor

arges commented Aug 1, 2016

I have no idea why autopkgtest failed:

# cd .; git clone https://github.com/kisielk/gotool /tmp/adt-run.rNJcbF/integrationtests-artifacts/build/src/github.com/kisielk/gotool
Cloning into '/tmp/adt-run.rNJcbF/integrationtests-artifacts/build/src/github.com/kisielk/gotool'...
fatal: unable to access 'https://github.com/kisielk/gotool/': Received HTTP code 504 from proxy after CONNECT
package github.com/kisielk/gotool: exit status 128

Looks like some sort of connectivity issue? This wasn't added by my patch.

interfaces: add /proc/version_signature to appamor template
Allow access to /proc/version_signature to be able to easily read kernel version
and build number. Add this to the apparmor template.

Signed-off-by: Chris J Arges <chris.j.arges@canonical.com>
Collaborator

mvo5 commented Aug 3, 2016

👍

Contributor

niemeyer commented Aug 3, 2016

Why is it a good idea to expose this to every single application?

Contributor

niemeyer commented Aug 3, 2016

Chris, to avoid much back and forth on this, please merge both this and the lspci on the kernel-modules interface itself. We'll probably need a few tweaks on the latter such as naming/etc, but this puts you much closer to finalizing your needs, and it's totally fine for an app that has access to loading arbitrary kernel modules to also have access to these details, obviously.

@niemeyer niemeyer closed this Aug 3, 2016

Contributor

jdstrand commented Aug 3, 2016

This could also go in system-observe.

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