Hello,

Am Donnerstag, 14. Januar 2016 schrieb John Johansen:
> On 01/14/2016 04:57 PM, Steve Beattie wrote:
> > 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 <john.johan...@canonical.com>
> > 
> > Acked-by: Steve Beattie <st...@nxnw.org> for both trunk and 2.10.
> > 
> > Is there a bug report open for this?
> 
> It does now.
> http://bugs.launchpad.net/bugs/1534405
> 
> I ran into it today when trying to give Jamie some newer timings for
> the parallel jobs patch.
> 
> It wasn't caught previously because most of my testing used -Q to skip
> the kernel load, and what kernel loads I did do where on a version
> pre the kernel_interface patch

I tend to disagree ;-)

About a month ago (2015-12-07), I told you about the 2.10 parser 
segfaulting when loading a directory on IRC (and sent you the core 
dump).

I just tested by compiling the parser before and after running bzr up 
(using trunk). Before, it segfaulted - and after bzr up, it doesn't 
segfault anymore. So it seems this patch fixes "my" segfault.

Therefore I officially claim the credits for reporting this bug first ;-))


BTW: Even if it looks like this patch fixes my issue, it might be a good 
idea to give my core dump a quick check if it really hit the same bug.


Regards,

Christian Boltz
-- 
Was glaubst Du, wie oft ich fluche, daß diese Windowskisten erst ein
explizites 'Nun speichere auch endlich in die Zwischenablage'
wünschen und ich immer erst ins Leere klicke, wenn ich's eilig habe,
nur weil ich ein strg-c vergessen habe? Menno, können die sich nicht
an den üblichen *nix-Standard halten?  [Helga Fischer in suse-linux]

Attachment: signature.asc
Description: This is a digitally signed message part.

-- 
AppArmor mailing list
AppArmor@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to