lkml.org 
[lkml]   [2010]   [Apr]   [21]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
    Patch in this message
    /
    SubjectRe: [PATCH 00/14] EVM
    From
    Date
    On Wed, 2010-04-21 at 14:58 -0700, Randy Dunlap wrote:
    > On Wed, 21 Apr 2010 17:49:40 -0400 Mimi Zohar wrote:
    >
    > > Extended Verification Module(EVM) detects offline tampering of the
    > > security extended attributes (e.g. security.selinux, security.SMACK64,
    > > security.ima), which is the basis for LSM permission decisions and,
    > > with this set of patches, integrity appraisal decisions. To detect
    > > offline tampering of the extended attributes, EVM maintains an
    > > HMAC-sha1 across a set of security extended attributes, storing the
    > > HMAC as the extended attribute 'security.evm'. To verify the integrity
    > > of an extended attribute, EVM exports evm_verifyxattr(), which
    > > re-calculates the HMAC and compares it with the version stored in
    > > 'security.evm'.
    > >
    > ...
    > >
    > > Much appreciation to Dave Hansen, Serge Hallyn, and Matt Helsley for
    > > reviewing the patches.
    > >
    > > Mimi
    > >
    > > Mimi Zohar (14):
    > > integrity: move ima inode integrity data management
    > > security: move LSM xattrnames to xattr.h
    > > xattr: define vfs_getxattr_alloc and vfs_xattr_cmp
    > > evm: re-release
    > > ima: move ima_file_free before releasing the file
    > > security: imbed evm calls in security hooks
    > > evm: inode post removexattr
    > > evm: imbed evm_inode_post_setattr
    > > evm: inode_post_init
    > > fs: add evm_inode_post_init calls
    > > ima: integrity appraisal extension
    > > ima: appraise default rules
    > > ima: inode post_setattr
    > > ima: add ima_inode_setxattr and ima_inode_removexattr
    > > --
    >
    > A summary diffstat would be good to see in patch 00/14.
    >
    > Lacking that, at least each individual patch should have a diffstat summary
    > in it. Please read Documentation/SubmittingPatches.
    >
    > ---
    > ~Randy

    Only two minor changes from the RFC posting:

    0011-ima-integrity-appraisal-extension.patch adds a missing
    ima_fix_xattr() call.

    Index: security-testing-2.6/security/integrity/ima/ima_appraise.c
    ===================================================================
    --- security-testing-2.6.orig/security/integrity/ima/ima_appraise.c
    +++ security-testing-2.6/security/integrity/ima/ima_appraise.c
    @@ -92,8 +92,13 @@ int ima_appraise_measurement(struct inte
    ima_fix_xattr(dentry, iint);
    integrity_audit_msg(AUDIT_INTEGRITY_DATA, inode,
    filename, op, cause, 1, 0);
    - } else
    + } else {
    status = INTEGRITY_PASS;
    +
    + if ((status == INTEGRITY_UNKNOWN)
    + && (ima_appraise & IMA_APPRAISE_FIX))
    + ima_fix_xattr(dentry, iint);
    + }
    iint->flags |= IMA_APPRAISED;
    } else {
    if (status == INTEGRITY_NOLABEL)
    0012-ima-appraise-default-rules.patch replaces audit_log_format() calls
    with ima_log_string(), introduced in Eric's patches.

    Mimi




    \
     
     \ /
      Last update: 2010-04-22 00:21    [W:5.525 / U:0.076 seconds]
    ©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site