On Mon, Aug 24, 2015 at 08:43:46AM -0700, Steve Beattie wrote:
> > SUMMARY: AddressSanitizer: alloc-dealloc-mismatch ??:0 operator 
> > delete(void*)
> > ==16428==HINT: if you don't care about these warnings you may set 
> > ASAN_OPTIONS=alloc_dealloc_mismatch=0
> > ==16428==ABORTING
> > Makefile:33: recipe for target 'error_output' failed
> > make[3]: *** [error_output] Error 1
> > make[3]: Leaving directory '/tmp/buildd/apparmor-2.10/parser/tst'
> > Makefile:298: recipe for target 'tests' failed
> 
> Looks like a legitimate type mismatch error when the objects are
> deleted.

Try the following patch:
---
 parser/libapparmor_re/expr-tree.h |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Index: b/parser/libapparmor_re/expr-tree.h
===================================================================
--- a/parser/libapparmor_re/expr-tree.h
+++ b/parser/libapparmor_re/expr-tree.h
@@ -672,7 +672,7 @@ public:
 
        ~hashedNodeVec()
        {
-               delete nodes;
+               delete[] nodes;
        }
 
        unsigned long size()const { return len; }

With that applied, local testing with AddressSanitzer shows the
caching tests failing due to leaking cached and feature objects
from libapparmor. I'll dig into it tomorrow.

(Looks like the packaging might be missing the caching related manpage.)

-- 
Steve Beattie
<[email protected]>
http://NxNW.org/~steve/

Attachment: signature.asc
Description: Digital signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to