[SSSD] [PATCHES] UTIL: Fix memory leak in switch_creds
Petr Cech
pcech at redhat.com
Wed Nov 11 06:58:46 UTC 2015
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/cwrap/become_user-tests.log
>
> 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 at 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 at 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 at 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 at 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 at 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
More information about the sssd-devel
mailing list