Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > linux.kernel > #1644929 > unrolled thread

Re: [RFC 04/11] ima: add support to namespace securityfs file

Started byTycho Andersen <tycho@docker.com>
First post2017-05-18 23:40 +0200
Last post2017-05-18 23:40 +0200
Articles 1 — 1 participant

Back to article view | Back to linux.kernel

This discussion starts older than the indexed window; earlier articles aren't shown. The article labeled Started by below is the oldest one visible, not the original post.


Contents

  Re: [RFC 04/11] ima: add support to namespace securityfs file Tycho Andersen <tycho@docker.com> - 2017-05-18 23:40 +0200

#1644929 — Re: [RFC 04/11] ima: add support to namespace securityfs file

FromTycho Andersen <tycho@docker.com>
Date2017-05-18 23:40 +0200
SubjectRe: [RFC 04/11] ima: add support to namespace securityfs file
Message-ID<tIBjj-Bq-3@gated-at.bofh.it>
Hi Guilherme,

On Thu, May 11, 2017 at 10:59:56AM -0300, Guilherme Magalhaes wrote:
> +static int ima_open_namespaces(struct inode *inode, struct file *filp)
> +{
> +	if (!(filp->f_flags & O_WRONLY))
> +		return -EACCES;
> +
> +	if (!capable(CAP_SYS_ADMIN))
> +		return -EPERM;
> +
> +	if (test_and_set_bit(IMA_FS_BUSY, &ima_fs_flags))
> +		return -EBUSY;

It probably makes sense to do something like:

if (!(ima_appraise & IMA_APPRAISE_NAMESPACE))
	return -EINVAL;

here.

I'll keep playing around with this patchset and see if I have any
other feedback.

Cheers,

Tycho

[toc] | [standalone]


Back to top | Article view | linux.kernel


csiph-web