From: kernel test robot <lkp@intel.com>
To: Casey Schaufler <casey@schaufler-ca.com>,
casey.schaufler@intel.com, jmorris@namei.org,
linux-security-module@vger.kernel.org, selinux@vger.kernel.org
Cc: kbuild-all@lists.01.org, casey@schaufler-ca.com,
linux-audit@redhat.com, keescook@chromium.org,
john.johansen@canonical.com, penguin-kernel@i-love.sakura.ne.jp,
paul@paul-moore.com
Subject: Re: [PATCH v22 20/23] Audit: Add new record for multiple process LSM attributes
Date: Wed, 11 Nov 2020 14:37:07 +0800 [thread overview]
Message-ID: <202011111418.xfg4Zq5q-lkp@intel.com> (raw)
In-Reply-To: <20201105004924.11651-21-casey@schaufler-ca.com>
[-- Attachment #1: Type: text/plain, Size: 3932 bytes --]
Hi Casey,
I love your patch! Perhaps something to improve:
[auto build test WARNING on nf-next/master]
[also build test WARNING on nf/master linus/master v5.10-rc3 next-20201110]
[cannot apply to security/next-testing]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Casey-Schaufler/LSM-Module-stacking-for-AppArmor/20201105-091817
base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next.git master
config: i386-randconfig-a004-20201104 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
# https://github.com/0day-ci/linux/commit/3fc93335d8a4090601fe221c305017ca73c23c61
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Casey-Schaufler/LSM-Module-stacking-for-AppArmor/20201105-091817
git checkout 3fc93335d8a4090601fe221c305017ca73c23c61
# save the attached .config to linux build tree
make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from security/integrity/ima/ima.h:22,
from security/integrity/ima/ima_api.c:18:
include/linux/audit.h:585:1: error: expected identifier or '(' before '+' token
585 | +static inline struct audit_context *audit_alloc_local(gfp_t gfpflags)
| ^
security/integrity/ima/ima_api.c: In function 'ima_audit_measurement':
security/integrity/ima/ima_api.c:360:12: error: implicit declaration of function 'audit_alloc_for_lsm'; did you mean 'audit_log_lsm'? [-Werror=implicit-function-declaration]
360 | context = audit_alloc_for_lsm(GFP_KERNEL);
| ^~~~~~~~~~~~~~~~~~~
| audit_log_lsm
>> security/integrity/ima/ima_api.c:360:10: warning: assignment to 'struct audit_context *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
360 | context = audit_alloc_for_lsm(GFP_KERNEL);
| ^
In file included from security/integrity/ima/ima.h:22,
from security/integrity/ima/ima_api.c:18:
At top level:
include/linux/audit.h:261:13: warning: 'audit_log_lsm' defined but not used [-Wunused-function]
261 | static void audit_log_lsm(struct audit_context *context)
| ^~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +360 security/integrity/ima/ima_api.c
339
340 void ima_audit_measurement(struct integrity_iint_cache *iint,
341 const unsigned char *filename)
342 {
343 struct audit_context *context;
344 struct audit_buffer *ab;
345 char *hash;
346 const char *algo_name = hash_algo_name[iint->ima_hash->algo];
347 int i;
348
349 if (iint->flags & IMA_AUDITED)
350 return;
351
352 hash = kzalloc((iint->ima_hash->length * 2) + 1, GFP_KERNEL);
353 if (!hash)
354 return;
355
356 for (i = 0; i < iint->ima_hash->length; i++)
357 hex_byte_pack(hash + (i * 2), iint->ima_hash->digest[i]);
358 hash[i * 2] = '\0';
359
> 360 context = audit_alloc_for_lsm(GFP_KERNEL);
361 ab = audit_log_start(context, GFP_KERNEL, AUDIT_INTEGRITY_RULE);
362 if (!ab)
363 goto out;
364
365 audit_log_format(ab, "file=");
366 audit_log_untrustedstring(ab, filename);
367 audit_log_format(ab, " hash=\"%s:%s\"", algo_name, hash);
368
369 audit_log_task_info(ab);
370 audit_log_end_local(ab, context);
371
372 iint->flags |= IMA_AUDITED;
373 out:
374 kfree(hash);
375 return;
376 }
377
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 29782 bytes --]
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: Casey Schaufler <casey@schaufler-ca.com>,
casey.schaufler@intel.com, jmorris@namei.org,
linux-security-module@vger.kernel.org, selinux@vger.kernel.org
Cc: john.johansen@canonical.com, kbuild-all@lists.01.org,
linux-audit@redhat.com
Subject: Re: [PATCH v22 20/23] Audit: Add new record for multiple process LSM attributes
Date: Wed, 11 Nov 2020 14:37:07 +0800 [thread overview]
Message-ID: <202011111418.xfg4Zq5q-lkp@intel.com> (raw)
In-Reply-To: <20201105004924.11651-21-casey@schaufler-ca.com>
[-- Attachment #1: Type: text/plain, Size: 3932 bytes --]
Hi Casey,
I love your patch! Perhaps something to improve:
[auto build test WARNING on nf-next/master]
[also build test WARNING on nf/master linus/master v5.10-rc3 next-20201110]
[cannot apply to security/next-testing]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Casey-Schaufler/LSM-Module-stacking-for-AppArmor/20201105-091817
base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next.git master
config: i386-randconfig-a004-20201104 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
# https://github.com/0day-ci/linux/commit/3fc93335d8a4090601fe221c305017ca73c23c61
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Casey-Schaufler/LSM-Module-stacking-for-AppArmor/20201105-091817
git checkout 3fc93335d8a4090601fe221c305017ca73c23c61
# save the attached .config to linux build tree
make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from security/integrity/ima/ima.h:22,
from security/integrity/ima/ima_api.c:18:
include/linux/audit.h:585:1: error: expected identifier or '(' before '+' token
585 | +static inline struct audit_context *audit_alloc_local(gfp_t gfpflags)
| ^
security/integrity/ima/ima_api.c: In function 'ima_audit_measurement':
security/integrity/ima/ima_api.c:360:12: error: implicit declaration of function 'audit_alloc_for_lsm'; did you mean 'audit_log_lsm'? [-Werror=implicit-function-declaration]
360 | context = audit_alloc_for_lsm(GFP_KERNEL);
| ^~~~~~~~~~~~~~~~~~~
| audit_log_lsm
>> security/integrity/ima/ima_api.c:360:10: warning: assignment to 'struct audit_context *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
360 | context = audit_alloc_for_lsm(GFP_KERNEL);
| ^
In file included from security/integrity/ima/ima.h:22,
from security/integrity/ima/ima_api.c:18:
At top level:
include/linux/audit.h:261:13: warning: 'audit_log_lsm' defined but not used [-Wunused-function]
261 | static void audit_log_lsm(struct audit_context *context)
| ^~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +360 security/integrity/ima/ima_api.c
339
340 void ima_audit_measurement(struct integrity_iint_cache *iint,
341 const unsigned char *filename)
342 {
343 struct audit_context *context;
344 struct audit_buffer *ab;
345 char *hash;
346 const char *algo_name = hash_algo_name[iint->ima_hash->algo];
347 int i;
348
349 if (iint->flags & IMA_AUDITED)
350 return;
351
352 hash = kzalloc((iint->ima_hash->length * 2) + 1, GFP_KERNEL);
353 if (!hash)
354 return;
355
356 for (i = 0; i < iint->ima_hash->length; i++)
357 hex_byte_pack(hash + (i * 2), iint->ima_hash->digest[i]);
358 hash[i * 2] = '\0';
359
> 360 context = audit_alloc_for_lsm(GFP_KERNEL);
361 ab = audit_log_start(context, GFP_KERNEL, AUDIT_INTEGRITY_RULE);
362 if (!ab)
363 goto out;
364
365 audit_log_format(ab, "file=");
366 audit_log_untrustedstring(ab, filename);
367 audit_log_format(ab, " hash=\"%s:%s\"", algo_name, hash);
368
369 audit_log_task_info(ab);
370 audit_log_end_local(ab, context);
371
372 iint->flags |= IMA_AUDITED;
373 out:
374 kfree(hash);
375 return;
376 }
377
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org
[-- Attachment #2: .config.gz --]
[-- Type: application/gzip, Size: 29782 bytes --]
[-- Attachment #3: Type: text/plain, Size: 102 bytes --]
--
Linux-audit mailing list
Linux-audit@redhat.com
https://www.redhat.com/mailman/listinfo/linux-audit
WARNING: multiple messages have this Message-ID (diff)
From: kernel test robot <lkp@intel.com>
To: kbuild-all@lists.01.org
Subject: Re: [PATCH v22 20/23] Audit: Add new record for multiple process LSM attributes
Date: Wed, 11 Nov 2020 14:37:07 +0800 [thread overview]
Message-ID: <202011111418.xfg4Zq5q-lkp@intel.com> (raw)
In-Reply-To: <20201105004924.11651-21-casey@schaufler-ca.com>
[-- Attachment #1: Type: text/plain, Size: 4028 bytes --]
Hi Casey,
I love your patch! Perhaps something to improve:
[auto build test WARNING on nf-next/master]
[also build test WARNING on nf/master linus/master v5.10-rc3 next-20201110]
[cannot apply to security/next-testing]
[If your patch is applied to the wrong git tree, kindly drop us a note.
And when submitting patch, we suggest to use '--base' as documented in
https://git-scm.com/docs/git-format-patch]
url: https://github.com/0day-ci/linux/commits/Casey-Schaufler/LSM-Module-stacking-for-AppArmor/20201105-091817
base: https://git.kernel.org/pub/scm/linux/kernel/git/pablo/nf-next.git master
config: i386-randconfig-a004-20201104 (attached as .config)
compiler: gcc-9 (Debian 9.3.0-15) 9.3.0
reproduce (this is a W=1 build):
# https://github.com/0day-ci/linux/commit/3fc93335d8a4090601fe221c305017ca73c23c61
git remote add linux-review https://github.com/0day-ci/linux
git fetch --no-tags linux-review Casey-Schaufler/LSM-Module-stacking-for-AppArmor/20201105-091817
git checkout 3fc93335d8a4090601fe221c305017ca73c23c61
# save the attached .config to linux build tree
make W=1 ARCH=i386
If you fix the issue, kindly add following tag as appropriate
Reported-by: kernel test robot <lkp@intel.com>
All warnings (new ones prefixed by >>):
In file included from security/integrity/ima/ima.h:22,
from security/integrity/ima/ima_api.c:18:
include/linux/audit.h:585:1: error: expected identifier or '(' before '+' token
585 | +static inline struct audit_context *audit_alloc_local(gfp_t gfpflags)
| ^
security/integrity/ima/ima_api.c: In function 'ima_audit_measurement':
security/integrity/ima/ima_api.c:360:12: error: implicit declaration of function 'audit_alloc_for_lsm'; did you mean 'audit_log_lsm'? [-Werror=implicit-function-declaration]
360 | context = audit_alloc_for_lsm(GFP_KERNEL);
| ^~~~~~~~~~~~~~~~~~~
| audit_log_lsm
>> security/integrity/ima/ima_api.c:360:10: warning: assignment to 'struct audit_context *' from 'int' makes pointer from integer without a cast [-Wint-conversion]
360 | context = audit_alloc_for_lsm(GFP_KERNEL);
| ^
In file included from security/integrity/ima/ima.h:22,
from security/integrity/ima/ima_api.c:18:
At top level:
include/linux/audit.h:261:13: warning: 'audit_log_lsm' defined but not used [-Wunused-function]
261 | static void audit_log_lsm(struct audit_context *context)
| ^~~~~~~~~~~~~
cc1: some warnings being treated as errors
vim +360 security/integrity/ima/ima_api.c
339
340 void ima_audit_measurement(struct integrity_iint_cache *iint,
341 const unsigned char *filename)
342 {
343 struct audit_context *context;
344 struct audit_buffer *ab;
345 char *hash;
346 const char *algo_name = hash_algo_name[iint->ima_hash->algo];
347 int i;
348
349 if (iint->flags & IMA_AUDITED)
350 return;
351
352 hash = kzalloc((iint->ima_hash->length * 2) + 1, GFP_KERNEL);
353 if (!hash)
354 return;
355
356 for (i = 0; i < iint->ima_hash->length; i++)
357 hex_byte_pack(hash + (i * 2), iint->ima_hash->digest[i]);
358 hash[i * 2] = '\0';
359
> 360 context = audit_alloc_for_lsm(GFP_KERNEL);
361 ab = audit_log_start(context, GFP_KERNEL, AUDIT_INTEGRITY_RULE);
362 if (!ab)
363 goto out;
364
365 audit_log_format(ab, "file=");
366 audit_log_untrustedstring(ab, filename);
367 audit_log_format(ab, " hash=\"%s:%s\"", algo_name, hash);
368
369 audit_log_task_info(ab);
370 audit_log_end_local(ab, context);
371
372 iint->flags |= IMA_AUDITED;
373 out:
374 kfree(hash);
375 return;
376 }
377
---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all(a)lists.01.org
[-- Attachment #2: config.gz --]
[-- Type: application/gzip, Size: 29782 bytes --]
next prev parent reply other threads:[~2020-11-11 6:37 UTC|newest]
Thread overview: 68+ messages / expand[flat|nested] mbox.gz Atom feed top
[not found] <20201105004924.11651-1-casey.ref@schaufler-ca.com>
2020-11-05 0:49 ` [PATCH v22 00/23] LSM: Module stacking for AppArmor Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 01/23] LSM: Infrastructure management of the sock security Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 02/23] LSM: Create and manage the lsmblob data structure Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 03/23] LSM: Use lsmblob in security_audit_rule_match Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 04/23] LSM: Use lsmblob in security_kernel_act_as Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 05/23] LSM: Use lsmblob in security_secctx_to_secid Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-10 6:53 ` James Morris
2020-11-10 6:53 ` James Morris
2020-11-05 0:49 ` [PATCH v22 06/23] LSM: Use lsmblob in security_secid_to_secctx Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-10 6:56 ` James Morris
2020-11-10 6:56 ` James Morris
2020-11-05 0:49 ` [PATCH v22 07/23] LSM: Use lsmblob in security_ipc_getsecid Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 08/23] LSM: Use lsmblob in security_task_getsecid Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 09/23] LSM: Use lsmblob in security_inode_getsecid Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 10/23] LSM: Use lsmblob in security_cred_getsecid Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 11/23] IMA: Change internal interfaces to use lsmblobs Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 12/23] LSM: Specify which LSM to display Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 13/23] LSM: Ensure the correct LSM context releaser Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 14/23] LSM: Use lsmcontext in security_secid_to_secctx Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 15/23] LSM: Use lsmcontext in security_inode_getsecctx Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 16/23] LSM: security_secid_to_secctx in netlink netfilter Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-10 13:37 ` Pablo Neira Ayuso
2020-11-10 13:37 ` Pablo Neira Ayuso
2020-11-10 19:26 ` James Morris
2020-11-10 19:26 ` James Morris
2020-11-05 0:49 ` [PATCH v22 17/23] NET: Store LSM netlabel data in a lsmblob Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 18/23] LSM: Verify LSM display sanity in binder Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 19/23] audit: add support for non-syscall auxiliary records Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-16 0:13 ` kernel test robot
2020-11-16 0:13 ` kernel test robot
2020-11-16 0:13 ` kernel test robot
2020-11-05 0:49 ` [PATCH v22 20/23] Audit: Add new record for multiple process LSM attributes Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 9:31 ` kernel test robot
2020-11-05 9:31 ` kernel test robot
2020-11-05 9:31 ` kernel test robot
2020-11-11 4:45 ` kernel test robot
2020-11-11 4:45 ` kernel test robot
2020-11-11 4:45 ` kernel test robot
2020-11-11 6:37 ` kernel test robot [this message]
2020-11-11 6:37 ` kernel test robot
2020-11-11 6:37 ` kernel test robot
2020-11-05 0:49 ` [PATCH v22 21/23] Audit: Add a new record for multiple object " Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 22/23] LSM: Add /proc attr entry for full LSM context Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
2020-11-05 0:49 ` [PATCH v22 23/23] AppArmor: Remove the exclusive flag Casey Schaufler
2020-11-05 0:49 ` Casey Schaufler
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=202011111418.xfg4Zq5q-lkp@intel.com \
--to=lkp@intel.com \
--cc=casey.schaufler@intel.com \
--cc=casey@schaufler-ca.com \
--cc=jmorris@namei.org \
--cc=john.johansen@canonical.com \
--cc=kbuild-all@lists.01.org \
--cc=keescook@chromium.org \
--cc=linux-audit@redhat.com \
--cc=linux-security-module@vger.kernel.org \
--cc=paul@paul-moore.com \
--cc=penguin-kernel@i-love.sakura.ne.jp \
--cc=selinux@vger.kernel.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.