[SSSD] [PATCH] CI: Suppress memory errors from poptGetNextOpt
Jakub Hrozek
jhrozek at redhat.com
Fri Dec 5 09:03:01 UTC 2014
On Fri, Dec 05, 2014 at 12:40:00AM +0100, Lukas Slebodnik wrote:
> On (04/12/14 22:53), Jakub Hrozek wrote:
> >Hi,
> >
> >Using this patch and the one that Lukas has pending on the list, the only
> >error in CI is the RHEL-6 build failure:
> > http://sssd-ci.duckdns.org/logs/job/4/70/summary.html
> >which is tracked by:
> > https://fedorahosted.org/sssd/ticket/2518
> >
> >I'm not thrilled by the use of ellipsis, but it was the only notation I
> >could come up with that would work on both Debian and all RHELs.
>
> >From 96913027b7e2babab659ae649d12a18f48a6ed14 Mon Sep 17 00:00:00 2001
> >From: Jakub Hrozek <jhrozek at redhat.com>
> >Date: Thu, 4 Dec 2014 21:12:45 +0100
> >Subject: [PATCH] CI: Suppress memory errors from poptGetNextOpt
> >
> >---
> > contrib/ci/sssd.supp | 31 +++++++++++++++++++++++++++++++
> > 1 file changed, 31 insertions(+)
> >
> >diff --git a/contrib/ci/sssd.supp b/contrib/ci/sssd.supp
> >index 5bd8c8d699d8807c260966f7e08dee312a0938c6..7ecc09d0def07de8595ce166db8ad84eb8d4049b 100644
> >--- a/contrib/ci/sssd.supp
> >+++ b/contrib/ci/sssd.supp
> >@@ -122,3 +122,34 @@
> > fun:exit
> > fun:(below main)
> > }
> >+
> >+# long-standing memory leak in popt
> >+{
> >+ popt-memleak-from-poptGetNextOpt-malloc
> >+ Memcheck:Leak
> >+ match-leak-kinds: definite
> >+ fun:malloc
> >+ ...
> >+ fun:poptGetNextOpt
> >+ fun:main
> >+}
> >+
> >+# long-standing memory leak in popt
> >+{
> >+ popt-memleak-from-poptGetNextOpt-realloc
> >+ Memcheck:Leak
> >+ match-leak-kinds: definite
> >+ fun:realloc
> >+ ...
> >+ fun:poptGetNextOpt
> >+ fun:main
> >+}
> I do not acare about memory leaks. So I'm fine with it.
>
> But the next explanation is pretty vague at least for error
> "Invalid read of size 4" -> Memcheck:Addr4
> I cannot see such warning with libpotp 1.16
> Please add comment which distro is problematic (the oldest one)
> and that it is fixed in new version of libpopt
OK, see the attached patch.
-------------- next part --------------
>From a7286f5051aec1e6f83630ab36c0f8179d0e07df Mon Sep 17 00:00:00 2001
From: Jakub Hrozek <jhrozek at redhat.com>
Date: Thu, 4 Dec 2014 21:12:45 +0100
Subject: [PATCH] CI: Suppress memory errors from poptGetNextOpt
---
contrib/ci/sssd.supp | 31 +++++++++++++++++++++++++++++++
1 file changed, 31 insertions(+)
diff --git a/contrib/ci/sssd.supp b/contrib/ci/sssd.supp
index 5bd8c8d699d8807c260966f7e08dee312a0938c6..c29fedb31a45317db0da2da193fca75124142449 100644
--- a/contrib/ci/sssd.supp
+++ b/contrib/ci/sssd.supp
@@ -122,3 +122,34 @@
fun:exit
fun:(below main)
}
+
+# long-standing memory leak in popt
+{
+ popt-memleak-from-poptGetNextOpt-malloc
+ Memcheck:Leak
+ match-leak-kinds: definite
+ fun:malloc
+ ...
+ fun:poptGetNextOpt
+ fun:main
+}
+
+# long-standing memory leak in popt
+{
+ popt-memleak-from-poptGetNextOpt-realloc
+ Memcheck:Leak
+ match-leak-kinds: definite
+ fun:realloc
+ ...
+ fun:poptGetNextOpt
+ fun:main
+}
+
+# popt was not good with read access either. Applies for popt <= 1.13
+{
+ popt-suppress-invalid-read
+ Memcheck:Addr4
+ ...
+ fun:poptGetNextOpt
+ fun:main
+}
--
1.9.3
More information about the sssd-devel
mailing list