Re: [PATCH v4 04/19] SELinux: Remove cred security blob poisoning

2018-09-27 Thread James Morris
fic code has > to go. The poisioning could be introduced into the infrastructure > at some later date. If it's useful, it should be incorporated into core LSM, otherwise that's a regression for SELinux. -- James Morris

Re: [PATCH v5 5/5] sidechannel: Linux Security Module for sidechannel

2018-09-27 Thread James Morris
gt; + return -EACCES; > + if (task_active_pid_ns(current) != task_active_pid_ns(p)) > + return -EACCES; > + return 0; I really don't like the idea of hard-coding namespace security semantics in an LSM. Also, I'm not sure if these semantics make any sense. It least make it user configurable. -- James Morris

Re: [PATCH v5 5/5] sidechannel: Linux Security Module for sidechannel

2018-09-27 Thread James Morris
gt; + return -EACCES; > + if (task_active_pid_ns(current) != task_active_pid_ns(p)) > + return -EACCES; > + return 0; I really don't like the idea of hard-coding namespace security semantics in an LSM. Also, I'm not sure if these semantics make any sense. It least make it user configurable. -- James Morris

Re: Leaking Path in XFS's ioctl interface(missing LSM check)

2018-09-27 Thread James Morris
mediated by LSM at a high level (security_file_ioctl). Some problematic ones are singled out at that point by LSMs for special handling. -- James Morris

Re: Leaking Path in XFS's ioctl interface(missing LSM check)

2018-09-27 Thread James Morris
mediated by LSM at a high level (security_file_ioctl). Some problematic ones are singled out at that point by LSMs for special handling. -- James Morris

Re: [GIT PULL] Keys: fix C++ reserved word issue

2018-09-25 Thread James Morris
> change that needs reverting. > > Yes, as that's obviously correct. I think you should wait for your > patch 2 here until 4.20-rc1, as this isn't a "fix" that is required at > this point in time. Ok, I'll queue this for the merge window. -- James Morris

Re: [GIT PULL] Keys: fix C++ reserved word issue

2018-09-25 Thread James Morris
> change that needs reverting. > > Yes, as that's obviously correct. I think you should wait for your > patch 2 here until 4.20-rc1, as this isn't a "fix" that is required at > this point in time. Ok, I'll queue this for the merge window. -- James Morris

[GIT PULL] Keys: fix C++ reserved word issue

2018-09-24 Thread James Morris
Please pull this revert and update, from David Howells: "Here's a pair of fixes that need to go upstream asap, please: (1) Revert an incorrect fix to the keyrings UAPI for a C++ reserved word used as a struct member name. This change being reverted breaks existing userspace code and

[GIT PULL] Keys: fix C++ reserved word issue

2018-09-24 Thread James Morris
Please pull this revert and update, from David Howells: "Here's a pair of fixes that need to go upstream asap, please: (1) Revert an incorrect fix to the keyrings UAPI for a C++ reserved word used as a struct member name. This change being reverted breaks existing userspace code and

Re: [PATCH v2 00/10] LSM: Module stacking in support of S.A.R.A and Landlock

2018-09-11 Thread James Morris
On Tue, 11 Sep 2018, Casey Schaufler wrote: > LSM: Module stacking in support of S.A.R.A and Landlock Please help prevent RSI and shorten this to SARA. -- James Morris

Re: [PATCH v2 00/10] LSM: Module stacking in support of S.A.R.A and Landlock

2018-09-11 Thread James Morris
On Tue, 11 Sep 2018, Casey Schaufler wrote: > LSM: Module stacking in support of S.A.R.A and Landlock Please help prevent RSI and shorten this to SARA. -- James Morris

Re: [PATCH 00/22] KEYS: Support TPM-wrapped key and crypto ops

2018-09-07 Thread James Morris
this code. In addition, we have test cases for > it in ELL (embedded linux library). > > Tested-by: Marcel Holtmann > Reviewed-by: Marcel Holtmann Thanks! -- James Morris

Re: [PATCH 00/22] KEYS: Support TPM-wrapped key and crypto ops

2018-09-07 Thread James Morris
this code. In addition, we have test cases for > it in ELL (embedded linux library). > > Tested-by: Marcel Holtmann > Reviewed-by: Marcel Holtmann Thanks! -- James Morris

Re: [PATCH] seccomp: remove unnecessary unlikely()

2018-09-06 Thread James Morris
l.org Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general and next-testing. Thanks! -- James Morris

Re: [PATCH] seccomp: remove unnecessary unlikely()

2018-09-06 Thread James Morris
l.org Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general and next-testing. Thanks! -- James Morris

Re: [PATCH] seccomp: remove unnecessary unlikely()

2018-09-05 Thread James Morris
ed out it doesn't really work to have a generic series against 20 > > trees :-/ > > I know that pain very well! > > > I'm submitting them individually to each subsystem. > > So this one is just for security. > > Sounds good. > > James, can you take this directly, or would you prefer a pull request from me? I'll take it with your ack. -- James Morris

Re: [PATCH] seccomp: remove unnecessary unlikely()

2018-09-05 Thread James Morris
ed out it doesn't really work to have a generic series against 20 > > trees :-/ > > I know that pain very well! > > > I'm submitting them individually to each subsystem. > > So this one is just for security. > > Sounds good. > > James, can you take this directly, or would you prefer a pull request from me? I'll take it with your ack. -- James Morris

Re: [PATCH 00/22] KEYS: Support TPM-wrapped key and crypto ops

2018-09-05 Thread James Morris
On Wed, 5 Sep 2018, David Howells wrote: > > Hi James, > > Here's a set of patches that does the following, if you could pull it please: Thanks, it would be good to see more review / acks, though. -- James Morris

Re: [PATCH 00/22] KEYS: Support TPM-wrapped key and crypto ops

2018-09-05 Thread James Morris
On Wed, 5 Sep 2018, David Howells wrote: > > Hi James, > > Here's a set of patches that does the following, if you could pull it please: Thanks, it would be good to see more review / acks, though. -- James Morris

Re: [PATCH] security: tomoyo: Fix obsolete function

2018-09-04 Thread James Morris
On Tue, 4 Sep 2018, Ding Xiang wrote: > simple_strtoul is obsolete, and use kstrtouint instead > > Signed-off-by: Ding Xiang Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general and next-testing. -- James Morris

Re: [PATCH] security: tomoyo: Fix obsolete function

2018-09-04 Thread James Morris
On Tue, 4 Sep 2018, Ding Xiang wrote: > simple_strtoul is obsolete, and use kstrtouint instead > > Signed-off-by: Ding Xiang Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general and next-testing. -- James Morris

Re: [PATCH v1] security/capabilities: remove check for -EINVAL

2018-08-29 Thread James Morris
On Wed, 22 Aug 2018, Christian Brauner wrote: > bprm_caps_from_vfs_caps() never returned -EINVAL so remove the > rc == -EINVAL check. > > Signed-off-by: Christian Brauner Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general -- James Morris

Re: [PATCH v1] security/capabilities: remove check for -EINVAL

2018-08-29 Thread James Morris
On Wed, 22 Aug 2018, Christian Brauner wrote: > bprm_caps_from_vfs_caps() never returned -EINVAL so remove the > rc == -EINVAL check. > > Signed-off-by: Christian Brauner Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general -- James Morris

Re: [PATCH v1] security/capabilities: remove check for -EINVAL

2018-08-23 Thread James Morris
d-by: Serge Hallyn Thanks, I'll queue this for post -rc1 next-general. -- James Morris

Re: [PATCH v1] security/capabilities: remove check for -EINVAL

2018-08-23 Thread James Morris
d-by: Serge Hallyn Thanks, I'll queue this for post -rc1 next-general. -- James Morris

[GIT PULL] security subsystem: Integrity updates for v4.19

2018-08-15 Thread James Morris
>From Mimi Zohar: "This pull request adds support for EVM signatures based on larger digests, contains a new audit record AUDIT_INTEGRITY_POLICY_RULE to differentiate the IMA policy rules from the IMA-audit messages, addresses two deadlocks due to either loading or searching for crypto

[GIT PULL] security subsystem: Integrity updates for v4.19

2018-08-15 Thread James Morris
>From Mimi Zohar: "This pull request adds support for EVM signatures based on larger digests, contains a new audit record AUDIT_INTEGRITY_POLICY_RULE to differentiate the IMA policy rules from the IMA-audit messages, addresses two deadlocks due to either loading or searching for crypto

[GIT PULL] security subsystem: TPM updates for v4.19

2018-08-15 Thread James Morris
0728' of git://git.infradead.org/users/jjs/linux-tpmdd into next-tpm (2018-07-30 09:30:19 -0700) ---- James Morris (1): Merge tag 'tpmdd-next-20180728' of git://git.infradead.org/users/jjs/linux-tpmdd into next-tpm Jarkko Sakkinen (5):

[GIT PULL] security subsystem: TPM updates for v4.19

2018-08-15 Thread James Morris
0728' of git://git.infradead.org/users/jjs/linux-tpmdd into next-tpm (2018-07-30 09:30:19 -0700) ---- James Morris (1): Merge tag 'tpmdd-next-20180728' of git://git.infradead.org/users/jjs/linux-tpmdd into next-tpm Jarkko Sakkinen (5):

[GIT PULL] security subsystem: Smack updates for v4.19

2018-08-15 Thread James Morris
) James Morris (1): Merge branch 'smack-for-4.19-a' of https://github.com/cschaufler/next-smack into next-smack Piotr Sawicki (3): Smack: Fix handling of IPv4 traffic received by PF_INET6 sockets Smack: Check UDP-Lite and DCCP

[GIT PULL] security subsystem: Smack updates for v4.19

2018-08-15 Thread James Morris
) James Morris (1): Merge branch 'smack-for-4.19-a' of https://github.com/cschaufler/next-smack into next-smack Piotr Sawicki (3): Smack: Fix handling of IPv4 traffic received by PF_INET6 sockets Smack: Check UDP-Lite and DCCP

Re: [PATCH 1/7] evmtest: Regression testing Integrity Subsystem

2018-08-14 Thread James Morris
On Tue, 14 Aug 2018, David Jacobson wrote: > This patchset introduces evmtest — a stand alone tool for regression > testing IMA. Nice! I usually run the SELinux testsuite as a general sanity check of LSM before pushing to Linus, and I'll also run this once it's merged. -- James Morris

Re: [PATCH 1/7] evmtest: Regression testing Integrity Subsystem

2018-08-14 Thread James Morris
On Tue, 14 Aug 2018, David Jacobson wrote: > This patchset introduces evmtest — a stand alone tool for regression > testing IMA. Nice! I usually run the SELinux testsuite as a general sanity check of LSM before pushing to Linus, and I'll also run this once it's merged. -- James Morris

Re: [GIT PULL] security subsystem: general update for v4.19

2018-08-13 Thread James Morris
On Tue, 14 Aug 2018, James Morris wrote: > I'll followup with updates for Smack and TPM once this is merged. As well as an update for the Integrity subsystem. -- James Morris

Re: [GIT PULL] security subsystem: general update for v4.19

2018-08-13 Thread James Morris
On Tue, 14 Aug 2018, James Morris wrote: > I'll followup with updates for Smack and TPM once this is merged. As well as an update for the Integrity subsystem. -- James Morris

[GIT PULL] security subsystem: general update for v4.19

2018-08-13 Thread James Morris
): security: export security_kernel_load_data function Eric Biggers (1): security: check for kstrdup() failure in lsm_append() James Morris (1): Merge tag 'v4.18-rc2' into next-general Mimi Zohar (8): security: define new LSM hook named security_kernel_load_data kexec

[GIT PULL] security subsystem: general update for v4.19

2018-08-13 Thread James Morris
): security: export security_kernel_load_data function Eric Biggers (1): security: check for kstrdup() failure in lsm_append() James Morris (1): Merge tag 'v4.18-rc2' into next-general Mimi Zohar (8): security: define new LSM hook named security_kernel_load_data kexec

Re: [PATCH 08/33] vfs: Add LSM hooks for the new mount API [ver #11]

2018-08-01 Thread James Morris
On Wed, 1 Aug 2018, David Howells wrote: > (2) A hook to snoop source specifications. What are source specifications? -- James Morris

Re: [PATCH 08/33] vfs: Add LSM hooks for the new mount API [ver #11]

2018-08-01 Thread James Morris
On Wed, 1 Aug 2018, David Howells wrote: > (2) A hook to snoop source specifications. What are source specifications? -- James Morris

Re: [PATCH v2] security: Add LSM fixup hooks to set*gid syscalls.

2018-08-01 Thread James Morris
a policy generally of the kernel that we only add features to support in-tree code. -- James Morris

Re: [PATCH v2] security: Add LSM fixup hooks to set*gid syscalls.

2018-08-01 Thread James Morris
a policy generally of the kernel that we only add features to support in-tree code. -- James Morris

Re: [PATCH v2] security: Add LSM fixup hooks to set*gid syscalls.

2018-07-31 Thread James Morris
On Tue, 31 Jul 2018, Micah Morton wrote: > +static inline int security_task_fix_setgid(struct cred *new, > +const struct cred *old, > +int flags) > +{ > + return 0; > +} > + This looks whitespace-damaged. Please send patches as plain text. -- James Morris

Re: [PATCH v2] security: Add LSM fixup hooks to set*gid syscalls.

2018-07-31 Thread James Morris
On Tue, 31 Jul 2018, Micah Morton wrote: > +static inline int security_task_fix_setgid(struct cred *new, > +const struct cred *old, > +int flags) > +{ > + return 0; > +} > + This looks whitespace-damaged. Please send patches as plain text. -- James Morris

Re: [GIT PULL] tpmdd 4.19 updates

2018-07-30 Thread James Morris
* Bug fixes and clean ups. Thanks, pulled to next-tpm and next-testing. -- James Morris

Re: [GIT PULL] tpmdd 4.19 updates

2018-07-30 Thread James Morris
* Bug fixes and clean ups. Thanks, pulled to next-tpm and next-testing. -- James Morris

Re: [PATCH] security: export security_kernel_load_data function

2018-07-17 Thread James Morris
rnd Bergmann Thanks! Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general -- James Morris

Re: [PATCH] security: export security_kernel_load_data function

2018-07-17 Thread James Morris
rnd Bergmann Thanks! Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general -- James Morris

Re: [PATCH 2/2] KEYS: trusted: Find tpm_chip and use it until module shutdown

2018-07-03 Thread James Morris
anyone does not vote against, I'll send a patch. > For Keys? That would would be useful to help reduce the workload on David. -- James Morris

Re: [PATCH 2/2] KEYS: trusted: Find tpm_chip and use it until module shutdown

2018-07-03 Thread James Morris
anyone does not vote against, I'll send a patch. > For Keys? That would would be useful to help reduce the workload on David. -- James Morris

Re: [PATCH 0/3] KEYS: Miscellaneous fixes

2018-06-26 Thread James Morris
t; Nick Desaulniers (1): > certs/blacklist: fix const confusion > > > certs/blacklist.h |2 +- > crypto/asymmetric_keys/x509_cert_parser.c |9 + > security/keys/dh.c|6 -- > 3 files changed, 14 insertions(+), 3 deletions(-) > -- James Morris

Re: [PATCH 0/3] KEYS: Miscellaneous fixes

2018-06-26 Thread James Morris
t; Nick Desaulniers (1): > certs/blacklist: fix const confusion > > > certs/blacklist.h |2 +- > crypto/asymmetric_keys/x509_cert_parser.c |9 + > security/keys/dh.c|6 -- > 3 files changed, 14 insertions(+), 3 deletions(-) > -- James Morris

Re: [PULL] Smack: Mark inode instant in smack_task_to_inode

2018-06-25 Thread James Morris
--- > Casey Schaufler (1): > Smack: Mark inode instant in smack_task_to_inode > > security/smack/smack_lsm.c | 1 + > 1 file changed, 1 insertion(+) > -- James Morris

Re: [PULL] Smack: Mark inode instant in smack_task_to_inode

2018-06-25 Thread James Morris
--- > Casey Schaufler (1): > Smack: Mark inode instant in smack_task_to_inode > > security/smack/smack_lsm.c | 1 + > 1 file changed, 1 insertion(+) > -- James Morris

[GIT PULL] Security subsystem: Smack updates for v4.18

2018-06-07 Thread James Morris
ck into next-smack (2018-06-05 13:18:58 -0700) Casey Schaufler (1): Smack: Fix memory leak in smack_inode_getsecctx James Morris (1): Merge branch 'smack-for-4.18' of https://github.com/cschaufler/next-smack in

[GIT PULL] Security subsystem: Smack updates for v4.18

2018-06-07 Thread James Morris
ck into next-smack (2018-06-05 13:18:58 -0700) Casey Schaufler (1): Smack: Fix memory leak in smack_inode_getsecctx James Morris (1): Merge branch 'smack-for-4.18' of https://github.com/cschaufler/next-smack in

[GIT PULL] Security subsystem: TPM updates for v4.18

2018-06-07 Thread James Morris
lling mistake in macro ST33ZP24_TISREGISTER_UKNOWN James Morris (2): Merge tag 'tpmdd-next-20180518' of git://git.infradead.org/users/jjs/linux-tpmdd into next-tpm Merge tag 'tpmdd-next-20180602' of git://git.infradead.org/users/jjs/linux-tpmdd into next-tpm Jerry Snitselaar (1):

[GIT PULL] Security subsystem: TPM updates for v4.18

2018-06-07 Thread James Morris
lling mistake in macro ST33ZP24_TISREGISTER_UKNOWN James Morris (2): Merge tag 'tpmdd-next-20180518' of git://git.infradead.org/users/jjs/linux-tpmdd into next-tpm Merge tag 'tpmdd-next-20180602' of git://git.infradead.org/users/jjs/linux-tpmdd into next-tpm Jerry Snitselaar (1):

[GIT PULL] Security subsystem: Integrity updates for v4.18

2018-06-07 Thread James Morris
Please pull these updates for the Integrity subsystem. >From Mimi: - adds run time support for specifying additional security xattrs included in the security.evm HMAC/signature - some code clean up and bug fixes. --- The following changes since commit

[GIT PULL] Security subsystem: Integrity updates for v4.18

2018-06-07 Thread James Morris
Please pull these updates for the Integrity subsystem. >From Mimi: - adds run time support for specifying additional security xattrs included in the security.evm HMAC/signature - some code clean up and bug fixes. --- The following changes since commit

[GIT PULL] Security subsystem: general updates for v4.18

2018-06-05 Thread James Morris
security: add hook for socketpair() net: hook socketpair() into LSM selinux: provide socketpair callback James Morris (2): Merge tag 'v4.17-rc2' into next-general Merge tag 'v4.17-rc3' into next-general Sargun Dhillon (1): security: remove security_settime Tom Gu

[GIT PULL] Security subsystem: general updates for v4.18

2018-06-05 Thread James Morris
security: add hook for socketpair() net: hook socketpair() into LSM selinux: provide socketpair callback James Morris (2): Merge tag 'v4.17-rc2' into next-general Merge tag 'v4.17-rc3' into next-general Sargun Dhillon (1): security: remove security_settime Tom Gu

Re: [GIT PULL] tpmdd 4.17 fixes

2018-06-04 Thread James Morris
mmon_write() > > drivers/char/tpm/tpm-dev-common.c | 40 > ++- > drivers/char/tpm/tpm-dev.h| 2 +- > 2 files changed, 19 insertions(+), 23 deletions(-) > -- James Morris

Re: [GIT PULL] tpmdd 4.17 fixes

2018-06-04 Thread James Morris
mmon_write() > > drivers/char/tpm/tpm-dev-common.c | 40 > ++- > drivers/char/tpm/tpm-dev.h| 2 +- > 2 files changed, 19 insertions(+), 23 deletions(-) > -- James Morris

Re: [PATCH v3 1/7] security: rename security_kernel_read_file() hook

2018-05-25 Thread James Morris
file_read and > security_kernel_file_post_read have been abused for. NAK. This abstraction is too semantically weak. LSM hooks need to map to stronger semantics so we can reason about what the hook and the policy is supposed to be mediating. -- James Morris <jmor...@namei.org>

Re: [PATCH v3 1/7] security: rename security_kernel_read_file() hook

2018-05-25 Thread James Morris
file_read and > security_kernel_file_post_read have been abused for. NAK. This abstraction is too semantically weak. LSM hooks need to map to stronger semantics so we can reason about what the hook and the policy is supposed to be mediating. -- James Morris

Re: [GIT PULL] tpmdd updates for v4.18

2018-05-18 Thread James Morris
jmorris/linux-security.git next-tpm and next-testing -- James Morris <jmor...@namei.org>

Re: [GIT PULL] tpmdd updates for v4.18

2018-05-18 Thread James Morris
jmorris/linux-security.git next-tpm and next-testing -- James Morris

Re: [PATCH v2 3/9] security: define security_kernel_read_blob() wrapper

2018-05-18 Thread James Morris
hen instead of two if (!file && read_id == X) checks, have: if (!file) { switch (read_id) { } } -- James Morris <jmor...@namei.org>

Re: [PATCH v2 3/9] security: define security_kernel_read_blob() wrapper

2018-05-18 Thread James Morris
if (!file && read_id == X) checks, have: if (!file) { switch (read_id) { } } -- James Morris

Re: [PATCH v3 1/3] big key: get rid of stack array allocation

2018-05-11 Thread James Morris
l applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general and next-testing Thanks! -- James Morris <jmor...@namei.org>

Re: [PATCH v3 1/3] big key: get rid of stack array allocation

2018-05-11 Thread James Morris
l applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general and next-testing Thanks! -- James Morris

Re: [PATCH 17/18] security: encrypted-keys: Remove pr_fmt duplicate logging prefixes

2018-05-10 Thread James Morris
ncrypted-keys/encrypted.c | 63 > ++-- > 1 file changed, 28 insertions(+), 35 deletions(-) Reviewed-by: James Morris <james.mor...@microsoft.com> -- James Morris <jmor...@namei.org>

Re: [PATCH 17/18] security: encrypted-keys: Remove pr_fmt duplicate logging prefixes

2018-05-10 Thread James Morris
ed-keys/encrypted.c | 63 > ++-- > 1 file changed, 28 insertions(+), 35 deletions(-) Reviewed-by: James Morris -- James Morris

Re: [PATCH v2 11/11] docs: fix broken references with multiple hints

2018-05-09 Thread James Morris
Chehab <mchehab+sams...@kernel.org> Acked-by: James Morris <james.mor...@microsoft.com> -- James Morris <jmor...@namei.org>

Re: [PATCH v2 11/11] docs: fix broken references with multiple hints

2018-05-09 Thread James Morris
ab Acked-by: James Morris -- James Morris

Re: [PATCH v3 1/3] big key: get rid of stack array allocation

2018-05-09 Thread James Morris
On Thu, 10 May 2018, James Morris wrote: > > > > Reviewed-by: Kees Cook <keesc...@chromium.org> > > > > James, are these something you can take into your tree? > > Sure! Although, normally, these would likely come in to mine via David's tree. Please d

Re: [PATCH v3 1/3] big key: get rid of stack array allocation

2018-05-09 Thread James Morris
On Thu, 10 May 2018, James Morris wrote: > > > > Reviewed-by: Kees Cook > > > > James, are these something you can take into your tree? > > Sure! Although, normally, these would likely come in to mine via David's tree. Please do that unless there's a special case here. -- James Morris

Re: [PATCH v3 1/3] big key: get rid of stack array allocation

2018-05-09 Thread James Morris
fails > > * define a constant for big key IV size so it can be changed along side > > the algorithm in the code > > > > Signed-off-by: Tycho Andersen <ty...@tycho.ws> > > CC: David Howells <dhowe...@redhat.com> > > CC: James Morris <jmor...

Re: [PATCH v3 1/3] big key: get rid of stack array allocation

2018-05-09 Thread James Morris
a constant for big key IV size so it can be changed along side > > the algorithm in the code > > > > Signed-off-by: Tycho Andersen > > CC: David Howells > > CC: James Morris > > CC: "Serge E. Hallyn" > > CC: Jason A. Done

Re: [PATCH 2/2] tpm: replace kmalloc() + memcpy() with kmemdup()

2018-05-09 Thread James Morris
On Wed, 9 May 2018, Ji-Hun Kim wrote: > Use kmemdup rather than duplicating its implementation. > > Signed-off-by: Ji-Hun Kim <ji_hun@samsung.com> Reviewed-by: James Morris <james.mor...@microsoft.com> -- James Morris <jmor...@namei.org>

Re: [PATCH 2/2] tpm: replace kmalloc() + memcpy() with kmemdup()

2018-05-09 Thread James Morris
On Wed, 9 May 2018, Ji-Hun Kim wrote: > Use kmemdup rather than duplicating its implementation. > > Signed-off-by: Ji-Hun Kim Reviewed-by: James Morris -- James Morris

Re: [PATCH 1/2] tpm: replace kmalloc() + memcpy() with kmemdup()

2018-05-09 Thread James Morris
On Wed, 9 May 2018, Ji-Hun Kim wrote: > Use kmemdup rather than duplicating its implementation. > > Signed-off-by: Ji-Hun Kim <ji_hun@samsung.com> Reviewed-by: James Morris <james.mor...@microsoft.com> -- James Morris <jmor...@namei.org>

Re: [PATCH 1/2] tpm: replace kmalloc() + memcpy() with kmemdup()

2018-05-09 Thread James Morris
On Wed, 9 May 2018, Ji-Hun Kim wrote: > Use kmemdup rather than duplicating its implementation. > > Signed-off-by: Ji-Hun Kim Reviewed-by: James Morris -- James Morris

Re: [PATCH v2 0/4] Introduce LSM-hook for socketpair(2)

2018-05-04 Thread James Morris
On Fri, 4 May 2018, David Herrmann wrote: > Hi > > This is v2 of the socketpair(2) LSM hook introduction. Thanks, all applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general -- James Morris <jmor...@namei.org>

Re: [PATCH v2 0/4] Introduce LSM-hook for socketpair(2)

2018-05-04 Thread James Morris
On Fri, 4 May 2018, David Herrmann wrote: > Hi > > This is v2 of the socketpair(2) LSM hook introduction. Thanks, all applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general -- James Morris

Re: [PATCH v6 0/4] Certificate insertion support for x86 bzImages

2018-05-03 Thread James Morris
distributed, the insert-sys-cert script can be used to insert the > certificate for x86. Can you provide more explanation of how this is useful and who would use it? -- James Morris <jmor...@namei.org>

Re: [PATCH v6 0/4] Certificate insertion support for x86 bzImages

2018-05-03 Thread James Morris
distributed, the insert-sys-cert script can be used to insert the > certificate for x86. Can you provide more explanation of how this is useful and who would use it? -- James Morris

Re: [PATCH v2 3/4] seccomp: Audit attempts to modify the actions_logged sysctl

2018-05-02 Thread James Morris
actions_logged sysctl. > > Suggested-by: Steve Grubb <sgr...@redhat.com> > Signed-off-by: Tyler Hicks <tyhi...@canonical.com> Reviewed-by: James Morris <james.mor...@microsoft.com> -- James Morris <jmor...@namei.org>

Re: [PATCH v2 3/4] seccomp: Audit attempts to modify the actions_logged sysctl

2018-05-02 Thread James Morris
actions_logged sysctl. > > Suggested-by: Steve Grubb > Signed-off-by: Tyler Hicks Reviewed-by: James Morris -- James Morris

Re: [PATCH v2 2/4] seccomp: Configurable separator for the actions_logged string

2018-05-02 Thread James Morris
eparator. This patch allows the separator character to be > configurable to meet both needs. > > Signed-off-by: Tyler Hicks <tyhi...@canonical.com> Reviewed-by: James Morris <james.mor...@microsoft.com> -- James Morris <jmor...@namei.org>

Re: [PATCH v2 2/4] seccomp: Configurable separator for the actions_logged string

2018-05-02 Thread James Morris
eparator. This patch allows the separator character to be > configurable to meet both needs. > > Signed-off-by: Tyler Hicks Reviewed-by: James Morris -- James Morris

Re: [PATCH v2 1/4] seccomp: Separate read and write code for actions_logged sysctl

2018-05-02 Thread James Morris
itional code paths on whether or not the > 'write' parameter evaluates to true. > > Signed-off-by: Tyler Hicks <tyhi...@canonical.com> Reviewed-by: James Morris <james.mor...@microsoft.com> -- James Morris <jmor...@namei.org>

Re: [PATCH v2 1/4] seccomp: Separate read and write code for actions_logged sysctl

2018-05-02 Thread James Morris
itional code paths on whether or not the > 'write' parameter evaluates to true. > > Signed-off-by: Tyler Hicks Reviewed-by: James Morris -- James Morris

Re: [PATCH 0/3] Introduce LSM-hook for socketpair(2)

2018-04-25 Thread James Morris
On Wed, 25 Apr 2018, Paul Moore wrote: > On Wed, Apr 25, 2018 at 2:44 PM, James Morris <jmor...@namei.org> wrote: > > On Mon, 23 Apr 2018, David Herrmann wrote: > >> This patch series tries to close this gap and makes both behave the > >> same. A new LSM-hook is

Re: [PATCH 0/3] Introduce LSM-hook for socketpair(2)

2018-04-25 Thread James Morris
On Wed, 25 Apr 2018, Paul Moore wrote: > On Wed, Apr 25, 2018 at 2:44 PM, James Morris wrote: > > On Mon, 23 Apr 2018, David Herrmann wrote: > >> This patch series tries to close this gap and makes both behave the > >> same. A new LSM-hook is added which allow

Re: [PATCH 0/3] Introduce LSM-hook for socketpair(2)

2018-04-25 Thread James Morris
k backend and maybe the hook name change, I'll merge this unless DaveM wants it to go in via his networking tree. -- James Morris <jmor...@namei.org>

Re: [PATCH 0/3] Introduce LSM-hook for socketpair(2)

2018-04-25 Thread James Morris
k backend and maybe the hook name change, I'll merge this unless DaveM wants it to go in via his networking tree. -- James Morris

Re: [PATCH] security: remove security_settime

2018-04-16 Thread James Morris
f-by: Sargun Dhillon <sar...@sargun.me> Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general and next-testing -- James Morris <jmor...@namei.org>

Re: [PATCH] security: remove security_settime

2018-04-16 Thread James Morris
Sargun Dhillon Applied to git://git.kernel.org/pub/scm/linux/kernel/git/jmorris/linux-security.git next-general and next-testing -- James Morris

Re: KASAN: null-ptr-deref Read in xattr_getsecurity

2018-04-10 Thread James Morris
ave preferred path for getting > this change merged? > It can go via my tree if needed, but otherwise: Acked-by: James Morris <james.mor...@microsoft.com> -- James Morris <jmor...@namei.org>

<    1   2   3   4   5   6   7   8   9   10   >