[apparmor] [patch] parser - add valgrind test script (was fix dbus peer_conds memory leak)
Tyler Hicks
tyhicks at canonical.com
Wed Sep 11 08:57:13 UTC 2013
On 2013-09-09 11:29:28, Steve Beattie wrote:
> On Fri, Sep 06, 2013 at 11:57:51AM -0700, Tyler Hicks wrote:
> > Nice catch! (and shame on me for not using valgrind)
>
> Well, we don't really make it easy to run valgrind regularly. In the
> interest of fixing that, attached is a simple script for doing so,
> along with a separate patch for fixing a few leaks that it found;
> most occur in error paths, though there's a couple of legitimate ones
> it found in parser_yacc.y. There are also more reported leaks that
> I have not fully investigated yet.
Great idea!
>
> It's also finding what may be some false positives around
> invalid reads around profile serialization (e.g. on
> simple_tests/conditional/else_if_5.sd) and conditional
> jump or move depends on uninitialized value(s) (e.g.
> simple_tests/network/network_ok_1.sd) that need more investigation,
> to determine whether it's a legitimate issue or a valgrind check that
> hopefully can be disabled.
The memory leak fixes get my
Acked-by: Tyler Hicks <tyhicks at canonical.com>
I rebased the C++ patches on that patch, so please go ahead and commit
it when you get a chance.
I need some more time to look at the valgrind test patch. It doesn't
like the C++ conversion and ever test was failing before I killed it.
I'm not immediately sure what is happening. Running the parser through
valgrind on the same testcases shows no issue. I'll have to look into
more.
Tyler
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 836 bytes
Desc: Digital signature
URL: <https://lists.ubuntu.com/archives/apparmor/attachments/20130911/f64b6b2b/attachment.pgp>
More information about the AppArmor
mailing list