Re: [systemd-devel] Patch: fix memleak in logind

2012-05-07 Thread Lennart Poettering 
On Fri, 04.05.12 16:19, Frederic Crozat (fcro...@suse.com) wrote:

 Hi,
 
 while researching logind bug with su / sudo, I ran logind under valgrind
 which spotted at least one memleak.
 
 Attached patch fixes the issue.

Thanks!

Applied!

-- 
Lennart Poettering - Red Hat, Inc.
___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel

[systemd-devel] Patch: fix memleak in logind

2012-05-04 Thread Frederic Crozat 
Hi,

while researching logind bug with su / sudo, I ran logind under valgrind
which spotted at least one memleak.

Attached patch fixes the issue.
-- 
Frederic Crozat fcro...@suse.com
SUSE
From f023b8ccdaeff9696a9ef2523bce256f96586cbf Mon Sep 17 00:00:00 2001
From: Frederic Crozat fcro...@suse.com
Date: Fri, 4 May 2012 16:14:19 +0200
Subject: [PATCH] logind: fix memory leak

---
 src/login/logind-session.c |1 +
 1 files changed, 1 insertions(+), 0 deletions(-)

diff --git a/src/login/logind-session.c b/src/login/logind-session.c
index 5490366..e297d3f 100644
--- a/src/login/logind-session.c
+++ b/src/login/logind-session.c
@@ -327,6 +327,7 @@ finish:
 free(vtnr);
 free(leader);
 free(audit_id);
+free(class);
 
 return r;
 }
-- 
1.7.7

___
systemd-devel mailing list
systemd-devel@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/systemd-devel