Re: [RFC 02/20] ima: Define ns_status for storing namespaced iint data

Re: [RFC 02/20] ima: Define ns_status for storing namespaced iint data

[kernel test robot]

Hi Stefan,

[FYI, it's a private test report for your RFC patch.]
[auto build test WARNING on zohar-integrity/next-integrity]
[also build test WARNING on pcmoore-selinux/next linus/master v5.16-rc3 next-20211130]
[cannot apply to jmorris-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/Stefan-Berger/ima-Namespace-IMA-with-audit-support-in-IMA-ns/20211201-001242
base:   https://git.kernel.org/pub/scm/linux/kernel/git/zohar/linux-integrity.git next-integrity
config: arm-randconfig-r014-20211128 (https://download.01.org/0day-ci/archive/20211201/202112010752.p7PfhOZ8-lkp@intel.com/config)
compiler: clang version 14.0.0 (https://github.com/llvm/llvm-project 25eb7fa01d7ebbe67648ea03841cda55b4239ab2)
reproduce (this is a W=1 build):
        wget https://raw.githubusercontent.com/intel/lkp-tests/master/sbin/make.cross -O ~/bin/make.cross
        chmod +x ~/bin/make.cross
        # install arm cross compiling tool for clang build
        # apt-get install binutils-arm-linux-gnueabi
        # https://github.com/0day-ci/linux/commit/21749848be3e256e789659aa2b343eee70dd3f65
        git remote add linux-review https://github.com/0day-ci/linux
        git fetch --no-tags linux-review Stefan-Berger/ima-Namespace-IMA-with-audit-support-in-IMA-ns/20211201-001242
        git checkout 21749848be3e256e789659aa2b343eee70dd3f65
        # save the config file to linux build tree
        mkdir build_dir
        COMPILER_INSTALL_PATH=$HOME/0day COMPILER=clang make.cross W=1 O=build_dir ARCH=arm SHELL=/bin/bash security/integrity/ima/

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 >>):

>> security/integrity/ima/ima_ns_status.c:72:6: warning: no previous prototype for function 'insert_ns_status' [-Wmissing-prototypes]
   void insert_ns_status(struct ima_namespace *ns, struct inode *inode,
        ^
   security/integrity/ima/ima_ns_status.c:72:1: note: declare 'static' if the function is not intended to be used outside of this translation unit
   void insert_ns_status(struct ima_namespace *ns, struct inode *inode,
   ^
   static 
   1 warning generated.


vim +/insert_ns_status +72 security/integrity/ima/ima_ns_status.c

    71	
  > 72	void insert_ns_status(struct ima_namespace *ns, struct inode *inode,
    73			      struct ns_status *status)
    74	{
    75		struct rb_node **p;
    76		struct rb_node *node, *parent = NULL;
    77		struct ns_status *test_status;
    78	
    79		p = &ns->ns_status_tree.rb_node;
    80		while (*p) {
    81			parent = *p;
    82			test_status = rb_entry(parent, struct ns_status, rb_node);
    83			if (inode < test_status->inode)
    84				p = &(*p)->rb_left;
    85			else
    86				p = &(*p)->rb_right;
    87		}
    88		node = &status->rb_node;
    89		rb_link_node(node, parent, p);
    90		rb_insert_color(node, &ns->ns_status_tree);
    91	}
    92	

---
0-DAY CI Kernel Test Service, Intel Corporation
https://lists.01.org/hyperkitty/list/kbuild-all@lists.01.org