[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