On Thu, Jan 14, 2016 at 04:46:36PM -0800, John Johansen wrote: > Patch -r 2952 switched over to using the library kernel interface, and > added a kernel_interface parameter to the dir_cb struct, that is > used to process directories. > > Unfortunately kernel_interface parameter of the dir_cb struct is not being > properly initialized resulting in odd failures and sefaults when the parser > is processing directories. > > Signed-off-by: John Johansen <[email protected]>
Acked-by: Seth Arnold <[email protected]> It might also be nice to add; memset(&cb_data, 0, sizeof(cb_data)); If you agree, please consider it pre-acked ;) Thanks > diff --git a/parser/parser_main.c b/parser/parser_main.c > index f671876..93ead14 100644 > --- a/parser/parser_main.c > +++ b/parser/parser_main.c > @@ -1143,6 +1143,7 @@ int main(int argc, char *argv[]) > > cb_data.dirname = profilename; > cb_data.cachedir = cacheloc; > + cb_data.kernel_interface = kernel_interface; > cb = binary_input ? binary_dir_cb : profile_dir_cb; > if ((retval = dirat_for_each(AT_FDCWD, profilename, > &cb_data, cb))) {
signature.asc
Description: Digital signature
-- AppArmor mailing list [email protected] Modify settings or unsubscribe at: https://lists.ubuntu.com/mailman/listinfo/apparmor
