On 11/09/2015 08:06 AM, Lukas Slebodnik wrote:
ehlo,
You can see a leak in talloc report.
But it was ignored. So we didn't notice it for long time.
http://sssd-ci.duckdns.org/logs/job/29/90/rhel7/ci-build-debug/src/tests/...
The first patch fixes the leak and the last one is prevention
for such mistakes in future.
LS
Hi Lukáš,
CI tests passed.
0001-UTIL-Fix-memory-leak-in-switch_creds.patch
From 219d1bdd378f0a8c17a508e1f3e29a2d5435f4d0 Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik<lslebodn(a)redhat.com>
Date: Sat, 24 Oct 2015 14:19:11 +0200
Subject: [PATCH 1/5] UTIL: Fix memory leak in switch_creds
If we are already requested used then we needn't to call
setreeuid(), setresgid(). But we forgot to relase local
struct sss_creds *ssc, which is used for returnig saved credentials.
---
ACK
0002-TESTS-Initialize-leak-check.patch
From 318b862f473daf9606bf7752283a63b36934908b Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik<lslebodn(a)redhat.com>
Date: Sat, 24 Oct 2015 15:39:21 +0200
Subject: [PATCH 2/5] TESTS: Initialize leak check
If leak_check_setup is not called then global_talloc_context
was not initialized and check_leaks_pop(global_talloc_context) will fail.
---
ACK
0003-TESTS-Check-return-value-of-check_leaks_pop.patch
From 21bf7449bb53209ad24c0ec4079a5810bb6f707b Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik<lslebodn(a)redhat.com>
Date: Sat, 24 Oct 2015 15:15:39 +0200
Subject: [PATCH 3/5] TESTS: Check return value of check_leaks_pop
---
ACK
0004-TESTS-Make-check_leaks-static-function.patch
From 7e95820146c58e68d9cdf356198d18a3f748ff81 Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik<lslebodn(a)redhat.com>
Date: Fri, 6 Nov 2015 15:13:29 +0100
Subject: [PATCH 4/5] TESTS: Make check_leaks static function
---
ACK
0005-TESTS-Add-warning-for-unused-result-of-leak-check-fu.patch
From d9e428b18c3282fc877683b1a8228665c5f9d48a Mon Sep 17 00:00:00 2001
From: Lukas Slebodnik<lslebodn(a)redhat.com>
Date: Sat, 24 Oct 2015 15:48:26 +0200
Subject: [PATCH 5/5] TESTS: Add warning for unused result of leak check
functions
---
ACK
Regards
Petr