Quoting Dwight Engen (dwight.en...@oracle.com): > Signed-off-by: Dwight Engen <dwight.en...@oracle.com> > --- > src/lxc/lsm/lsm.c | 9 ++++++++- > src/lxc/lsm/lsm.h | 26 ++++++++++++++------------ > 2 files changed, 22 insertions(+), 13 deletions(-) > > diff --git a/src/lxc/lsm/lsm.c b/src/lxc/lsm/lsm.c > index f022de9..508d640 100644 > --- a/src/lxc/lsm/lsm.c > +++ b/src/lxc/lsm/lsm.c > @@ -62,13 +62,20 @@ void lsm_init(void) > INFO("Initialized LSM security driver %s", drv->name); > } > > -int lsm_enabled() > +int lsm_enabled(void) > { > if (drv) > return drv->enabled(); > return 0; > } > > +const char *lsm_name(void) > +{ > + if (drv) > + return drv->name; > + return NULL;
I think it should return "none" here. (I'm not sure your use of the NULL return in the attach testcase is safe.) ------------------------------------------------------------------------------ October Webinars: Code for Performance Free Intel webinars can help you accelerate application performance. Explore tips for MPI, OpenMP, advanced profiling, and more. Get the most from the latest Intel processors and coprocessors. See abstracts and register > http://pubads.g.doubleclick.net/gampad/clk?id=60135031&iu=/4140/ostg.clktrk _______________________________________________ Lxc-devel mailing list Lxc-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/lxc-devel