On (03/12/14 22:00), Jakub Hrozek wrote:
On Wed, Dec 03, 2014 at 09:15:45PM +0100, Lukas Slebodnik wrote:
> On (03/12/14 21:44), Nikolai Kondrashov wrote:
> >On 12/03/2014 07:23 PM, Lukas Slebodnik wrote:
> >>On (03/12/14 18:17), Jakub Hrozek wrote:
> >>>On Wed, Dec 03, 2014 at 10:18:33AM -0500, Lukas Slebodnik wrote:
> >>>>----- Original Message -----
> >>>>>From: "Nikolai Kondrashov"
<Nikolai.Kondrashov(a)redhat.com>
> >>>>>To: "Development of the System Security Services
Daemon" <sssd-devel(a)lists.fedorahosted.org>
> >>>>>Sent: Wednesday, December 3, 2014 4:00:14 PM
> >>>>>Subject: Re: [SSSD] [PATCH] CI: Update valgrind suppresion
database for libselinux
> >>>>>
> >>>>>On 12/03/2014 04:23 PM, Lukas Slebodnik wrote:
> >>>>>> From 7f041a011f44a8fb1c128095ad55c0669287209d Mon Sep 17
00:00:00 2001
> >>>>>>From: Lukas Slebodnik<lslebodn(a)redhat.com>
> >>>>>>Date: Wed, 3 Dec 2014 14:34:43 +0100
> >>>>>>Subject: [PATCH 1/2] CI: Update valgrind suppresion database
for libselinux
> >>>>>>
> >>>>>>problem is already fixed in fedora >= 21
> >>>>>
> >>>>>Thanks, Lukas!
> >>>>>Do you have any information on what the problem was?
> >>>>>
> >>>>I'm sorry it isn't important information for me.
> >>>>Problem is fixed on fedora 21. And that's al what I need to
know.
> >>>>
> >>>>LS
> >>>
> >>>Can you change the name as Nikolai requested so that we can push the
> >>>patch and unbreak CI?
> >>I do not see a reason why there should be prefix sssd-
> >>
> >>The problem is in libselinux and not in sssd.
> >
> >Yes, but this way we'll also be able to see that it is *our* suppression and
> >not some other package suppression from /usr/lib64/valgrind.
> >
> valgrind will not use prefix "libselinux-"
I really don't want to spend more time on naming of a valgrind
suppresion. I think I, and I hope other developers too, have more
important things to do.
If the patch solves the problem, then fine.
I added problematic function from
libseliux to description.
LS