From 84c6ae205883b8f397fdc7a06f6ed233a9c14e1d Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?V=C3=A1clav=20Pavl=C3=ADn?= <vpavlin@redhat.com> Date: Fri, 21 Sep 2012 13:01:39 +0200 Subject: [PATCH] logind: check return value, log warning on error (cherry picked from commit 5a165aa6b9aa921d8b069059026d84e879ac38cc) --- src/login/logind-session.c | 4 +++- src/login/logind-user.c | 4 +++- 2 files changed, 6 insertions(+), 2 deletions(-) diff --git a/src/login/logind-session.c b/src/login/logind-session.c index e4a7473..80f41df 100644 --- a/src/login/logind-session.c +++ b/src/login/logind-session.c @@ -529,7 +529,9 @@ static int session_create_cgroup(Session *s) { } } - hashmap_put(s->manager->session_cgroups, s->cgroup_path, s); + r = hashmap_put(s->manager->session_cgroups, s->cgroup_path, s); + if (r < 0) + log_warning("Failed to create mapping between cgroup and session"); return 0; } diff --git a/src/login/logind-user.c b/src/login/logind-user.c index fa12e1d..31c76e3 100644 --- a/src/login/logind-user.c +++ b/src/login/logind-user.c @@ -315,7 +315,9 @@ static int user_create_cgroup(User *u) { log_warning("Failed to create cgroup %s:%s: %s", *k, p, strerror(-r)); } - hashmap_put(u->manager->user_cgroups, u->cgroup_path, u); + r = hashmap_put(u->manager->user_cgroups, u->cgroup_path, u); + if (r < 0) + log_warning("Failed to create mapping between cgroup and user"); return 0; }