chiark
/
gitweb
/
~ianmdlvl
/
elogind.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
|
inline
| side by side (parent:
aea275c
)
login/sd-login.c: make use of _cleanup_free_ and friends
author
Harald Hoyer
<harald@redhat.com>
Wed, 17 Apr 2013 12:05:24 +0000
(14:05 +0200)
committer
Harald Hoyer
<harald@redhat.com>
Thu, 18 Apr 2013 06:39:07 +0000
(08:39 +0200)
src/login/sd-login.c
patch
|
blob
|
history
diff --git
a/src/login/sd-login.c
b/src/login/sd-login.c
index c97d75c000f5883b425b0c321a06da2aa7996c7f..45523921b5dcf03fb8904f0d77abd0c63a58e93c 100644
(file)
--- a/
src/login/sd-login.c
+++ b/
src/login/sd-login.c
@@
-74,7
+74,7
@@
_public_ int sd_pid_get_machine_name(pid_t pid, char **name) {
_public_ int sd_pid_get_owner_uid(pid_t pid, uid_t *uid) {
int r;
_public_ int sd_pid_get_owner_uid(pid_t pid, uid_t *uid) {
int r;
-
char *root, *cgroup, *p, *cc
;
+
_cleanup_free_ char *root = NULL, *cgroup = NULL, *p = NULL, *cc = NULL
;
struct stat st;
if (pid < 0)
struct stat st;
if (pid < 0)
@@
-87,31
+87,23
@@
_public_ int sd_pid_get_owner_uid(pid_t pid, uid_t *uid) {
if (r < 0)
return r;
if (r < 0)
return r;
- if (!startswith(cgroup, "/user/")) {
- free(cgroup);
- free(root);
+ if (!startswith(cgroup, "/user/"))
return -ENOENT;
return -ENOENT;
- }
p = strchr(cgroup + 6, '/');
p = strchr(cgroup + 6, '/');
- if (!p) {
- free(cgroup);
+ if (!p)
return -ENOENT;
return -ENOENT;
- }
p++;
p += strcspn(p, "/");
*p = 0;
r = cg_get_path(SYSTEMD_CGROUP_CONTROLLER, root, cgroup, &cc);
p++;
p += strcspn(p, "/");
*p = 0;
r = cg_get_path(SYSTEMD_CGROUP_CONTROLLER, root, cgroup, &cc);
- free(root);
- free(cgroup);
if (r < 0)
return -ENOMEM;
r = lstat(cc, &st);
if (r < 0)
return -ENOMEM;
r = lstat(cc, &st);
- free(cc);
if (r < 0)
return -errno;
if (r < 0)
return -errno;
@@
-155,7
+147,8
@@
_public_ int sd_uid_get_state(uid_t uid, char**state) {
}
_public_ int sd_uid_is_on_seat(uid_t uid, int require_active, const char *seat) {
}
_public_ int sd_uid_is_on_seat(uid_t uid, int require_active, const char *seat) {
- char *p, *w, *t, *state, *s = NULL;
+ char *w, *state;
+ _cleanup_free_ char *t = NULL, *s = NULL, *p = NULL;
size_t l;
int r;
const char *variable;
size_t l;
int r;
const char *variable;
@@
-170,38
+163,26
@@
_public_ int sd_uid_is_on_seat(uid_t uid, int require_active, const char *seat)
return -ENOMEM;
r = parse_env_file(p, NEWLINE, variable, &s, NULL);
return -ENOMEM;
r = parse_env_file(p, NEWLINE, variable, &s, NULL);
- free(p);
- if (r < 0) {
- free(s);
+ if (r < 0)
return r;
return r;
- }
if (!s)
return -EIO;
if (!s)
return -EIO;
- if (asprintf(&t, "%lu", (unsigned long) uid) < 0) {
- free(s);
+ if (asprintf(&t, "%lu", (unsigned long) uid) < 0)
return -ENOMEM;
return -ENOMEM;
- }
FOREACH_WORD(w, l, s, state) {
FOREACH_WORD(w, l, s, state) {
- if (strneq(t, w, l)) {
- free(s);
- free(t);
-
+ if (strneq(t, w, l))
return 1;
return 1;
- }
}
}
- free(s);
- free(t);
-
return 0;
}
static int uid_get_array(uid_t uid, const char *variable, char ***array) {
return 0;
}
static int uid_get_array(uid_t uid, const char *variable, char ***array) {
-
char *p
, *s = NULL;
+
_cleanup_free_ char *p = NULL
, *s = NULL;
char **a;
int r;
char **a;
int r;
@@
-211,11
+192,7
@@
static int uid_get_array(uid_t uid, const char *variable, char ***array) {
r = parse_env_file(p, NEWLINE,
variable, &s,
NULL);
r = parse_env_file(p, NEWLINE,
variable, &s,
NULL);
- free(p);
-
if (r < 0) {
if (r < 0) {
- free(s);
-
if (r == -ENOENT) {
if (array)
*array = NULL;
if (r == -ENOENT) {
if (array)
*array = NULL;
@@
-232,7
+209,6
@@
static int uid_get_array(uid_t uid, const char *variable, char ***array) {
}
a = strv_split(s, " ");
}
a = strv_split(s, " ");
- free(s);
if (!a)
return -ENOMEM;
if (!a)
return -ENOMEM;
@@
-294,31
+270,27
@@
static int file_of_session(const char *session, char **_p) {
_public_ int sd_session_is_active(const char *session) {
int r;
_public_ int sd_session_is_active(const char *session) {
int r;
-
char *p
, *s = NULL;
+
_cleanup_free_ char *p = NULL
, *s = NULL;
r = file_of_session(session, &p);
if (r < 0)
return r;
r = parse_env_file(p, NEWLINE, "ACTIVE", &s, NULL);
r = file_of_session(session, &p);
if (r < 0)
return r;
r = parse_env_file(p, NEWLINE, "ACTIVE", &s, NULL);
- free(p);
- if (r < 0) {
- free(s);
+ if (r < 0)
return r;
return r;
- }
if (!s)
return -EIO;
r = parse_boolean(s);
if (!s)
return -EIO;
r = parse_boolean(s);
- free(s);
return r;
}
_public_ int sd_session_get_state(const char *session, char **state) {
return r;
}
_public_ int sd_session_get_state(const char *session, char **state) {
-
char *p
, *s = NULL;
+
_cleanup_free_ char *p = NULL
, *s = NULL;
int r;
if (!state)
int r;
if (!state)
@@
-329,21
+301,21
@@
_public_ int sd_session_get_state(const char *session, char **state) {
return r;
r = parse_env_file(p, NEWLINE, "STATE", &s, NULL);
return r;
r = parse_env_file(p, NEWLINE, "STATE", &s, NULL);
- free(p);
- if (r < 0) {
- free(s);
+ if (r < 0)
return r;
return r;
-
}
else if (!s)
+ else if (!s)
return -EIO;
*state = s;
return -EIO;
*state = s;
+ s = NULL;
+
return 0;
}
_public_ int sd_session_get_uid(const char *session, uid_t *uid) {
int r;
return 0;
}
_public_ int sd_session_get_uid(const char *session, uid_t *uid) {
int r;
-
char *p
, *s = NULL;
+
_cleanup_free_ char *p = NULL
, *s = NULL;
if (!uid)
return -EINVAL;
if (!uid)
return -EINVAL;
@@
-353,24
+325,20
@@
_public_ int sd_session_get_uid(const char *session, uid_t *uid) {
return r;
r = parse_env_file(p, NEWLINE, "UID", &s, NULL);
return r;
r = parse_env_file(p, NEWLINE, "UID", &s, NULL);
- free(p);
- if (r < 0) {
- free(s);
+ if (r < 0)
return r;
return r;
- }
if (!s)
return -EIO;
r = parse_uid(s, uid);
if (!s)
return -EIO;
r = parse_uid(s, uid);
- free(s);
return r;
}
static int session_get_string(const char *session, const char *field, char **value) {
return r;
}
static int session_get_string(const char *session, const char *field, char **value) {
-
char *p
, *s = NULL;
+
_cleanup_free_ char *p = NULL
, *s = NULL;
int r;
if (!value)
int r;
if (!value)
@@
-381,17
+349,15
@@
static int session_get_string(const char *session, const char *field, char **val
return r;
r = parse_env_file(p, NEWLINE, field, &s, NULL);
return r;
r = parse_env_file(p, NEWLINE, field, &s, NULL);
- free(p);
- if (r < 0) {
- free(s);
+ if (r < 0)
return r;
return r;
- }
if (isempty(s))
return -ENOENT;
*value = s;
if (isempty(s))
return -ENOENT;
*value = s;
+ s = NULL;
return 0;
}
return 0;
}
@@
-428,25
+394,25
@@
static int file_of_seat(const char *seat, char **_p) {
if (seat)
p = strappend("/run/systemd/seats/", seat);
else {
if (seat)
p = strappend("/run/systemd/seats/", seat);
else {
-
char *buf
;
+
_cleanup_free_ char *buf = NULL
;
r = sd_session_get_seat(NULL, &buf);
if (r < 0)
return r;
p = strappend("/run/systemd/seats/", buf);
r = sd_session_get_seat(NULL, &buf);
if (r < 0)
return r;
p = strappend("/run/systemd/seats/", buf);
- free(buf);
}
if (!p)
return -ENOMEM;
*_p = p;
}
if (!p)
return -ENOMEM;
*_p = p;
+ p = NULL;
return 0;
}
_public_ int sd_seat_get_active(const char *seat, char **session, uid_t *uid) {
return 0;
}
_public_ int sd_seat_get_active(const char *seat, char **session, uid_t *uid) {
-
char *p
, *s = NULL, *t = NULL;
+
_cleanup_free_ char *p = NULL
, *s = NULL, *t = NULL;
int r;
if (!session && !uid)
int r;
if (!session && !uid)
@@
-460,46
+426,33
@@
_public_ int sd_seat_get_active(const char *seat, char **session, uid_t *uid) {
"ACTIVE", &s,
"ACTIVE_UID", &t,
NULL);
"ACTIVE", &s,
"ACTIVE_UID", &t,
NULL);
- free(p);
-
- if (r < 0) {
- free(s);
- free(t);
+ if (r < 0)
return r;
return r;
- }
- if (session && !s) {
- free(t);
+ if (session && !s)
return -ENOENT;
return -ENOENT;
- }
- if (uid && !t) {
- free(s);
+ if (uid && !t)
return -ENOENT;
return -ENOENT;
- }
if (uid && t) {
r = parse_uid(t, uid);
if (uid && t) {
r = parse_uid(t, uid);
- if (r < 0) {
- free(t);
- free(s);
+ if (r < 0)
return r;
return r;
- }
}
}
- free(t);
-
- if (session && s)
+ if (session && s) {
*session = s;
*session = s;
- else
- free(s);
+ s = NULL;
+ }
return 0;
}
_public_ int sd_seat_get_sessions(const char *seat, char ***sessions, uid_t **uids, unsigned *n_uids) {
return 0;
}
_public_ int sd_seat_get_sessions(const char *seat, char ***sessions, uid_t **uids, unsigned *n_uids) {
- char *p, *s = NULL, *t = NULL, **a = NULL;
- uid_t *b = NULL;
+ _cleanup_free_ char *p = NULL, *s = NULL, *t = NULL;
+ _cleanup_strv_free_ char **a = NULL;
+ _cleanup_free_ uid_t *b = NULL;
unsigned n = 0;
int r;
unsigned n = 0;
int r;
@@
-511,25
+464,16
@@
_public_ int sd_seat_get_sessions(const char *seat, char ***sessions, uid_t **ui
"SESSIONS", &s,
"ACTIVE_SESSIONS", &t,
NULL);
"SESSIONS", &s,
"ACTIVE_SESSIONS", &t,
NULL);
- free(p);
- if (r < 0) {
- free(s);
- free(t);
+ if (r < 0)
return r;
return r;
- }
if (s) {
a = strv_split(s, " ");
if (s) {
a = strv_split(s, " ");
- if (!a) {
- free(s);
- free(t);
+ if (!a)
return -ENOMEM;
return -ENOMEM;
- }
}
}
- free(s);
-
if (uids && t) {
char *w, *state;
size_t l;
if (uids && t) {
char *w, *state;
size_t l;
@@
-537,30
+481,22
@@
_public_ int sd_seat_get_sessions(const char *seat, char ***sessions, uid_t **ui
FOREACH_WORD(w, l, t, state)
n++;
FOREACH_WORD(w, l, t, state)
n++;
- if (n == 0)
- b = NULL;
- else {
+ if (n > 0) {
unsigned i = 0;
b = new(uid_t, n);
unsigned i = 0;
b = new(uid_t, n);
- if (!b) {
- strv_free(a);
+ if (!b)
return -ENOMEM;
return -ENOMEM;
- }
FOREACH_WORD(w, l, t, state) {
FOREACH_WORD(w, l, t, state) {
-
char *k
;
+
_cleanup_free_ char *k = NULL
;
k = strndup(w, l);
k = strndup(w, l);
- if (!k) {
- free(t);
- free(b);
- strv_free(a);
+ if (!k)
return -ENOMEM;
return -ENOMEM;
- }
r = parse_uid(k, b + i);
r = parse_uid(k, b + i);
- free(k);
+
if (r < 0)
continue;
if (r < 0)
continue;
@@
-569,17
+505,17
@@
_public_ int sd_seat_get_sessions(const char *seat, char ***sessions, uid_t **ui
}
}
}
}
- free(t);
-
r = strv_length(a);
r = strv_length(a);
- if (sessions)
+ if (sessions)
{
*sessions = a;
*sessions = a;
- else
- strv_free(a);
+ a = NULL;
+ }
- if (uids)
+ if (uids)
{
*uids = b;
*uids = b;
+ b = NULL;
+ }
if (n_uids)
*n_uids = n;
if (n_uids)
*n_uids = n;
@@
-588,7
+524,7
@@
_public_ int sd_seat_get_sessions(const char *seat, char ***sessions, uid_t **ui
}
static int seat_get_can(const char *seat, const char *variable) {
}
static int seat_get_can(const char *seat, const char *variable) {
-
char *p
, *s = NULL;
+
_cleanup_free_ char *p = NULL
, *s = NULL;
int r;
r = file_of_seat(seat, &p);
int r;
r = file_of_seat(seat, &p);
@@
-598,17
+534,12
@@
static int seat_get_can(const char *seat, const char *variable) {
r = parse_env_file(p, NEWLINE,
variable, &s,
NULL);
r = parse_env_file(p, NEWLINE,
variable, &s,
NULL);
- free(p);
-
- if (r < 0) {
- free(s);
+ if (r < 0)
return r;
return r;
- }
- if (s)
{
+ if (s)
r = parse_boolean(s);
r = parse_boolean(s);
- free(s);
- } else
+ else
r = 0;
return r;
r = 0;
return r;
@@
-635,10
+566,10
@@
_public_ int sd_get_sessions(char ***sessions) {
}
_public_ int sd_get_uids(uid_t **users) {
}
_public_ int sd_get_uids(uid_t **users) {
- DIR *d;
+
_cleanup_closedir_
DIR *d;
int r = 0;
unsigned n = 0;
int r = 0;
unsigned n = 0;
- uid_t *l = NULL;
+
_cleanup_free_
uid_t *l = NULL;
d = opendir("/run/systemd/users/");
if (!d)
d = opendir("/run/systemd/users/");
if (!d)
@@
-651,10
+582,8
@@
_public_ int sd_get_uids(uid_t **users) {
uid_t uid;
k = readdir_r(d, &buf.de, &de);
uid_t uid;
k = readdir_r(d, &buf.de, &de);
- if (k != 0) {
- r = -k;
- goto finish;
- }
+ if (k != 0)
+ return -k;
if (!de)
break;
if (!de)
break;
@@
-674,10
+603,8
@@
_public_ int sd_get_uids(uid_t **users) {
n = MAX(16, 2*r);
t = realloc(l, sizeof(uid_t) * n);
n = MAX(16, 2*r);
t = realloc(l, sizeof(uid_t) * n);
- if (!t) {
- r = -ENOMEM;
- goto finish;
- }
+ if (!t)
+ return -ENOMEM;
l = t;
}
l = t;
}
@@
-688,15
+615,10
@@
_public_ int sd_get_uids(uid_t **users) {
r++;
}
r++;
}
-finish:
- if (d)
- closedir(d);
-
- if (r >= 0) {
- if (users)
- *users = l;
- } else
- free(l);
+ if (users) {
+ *users = l;
+ l = NULL;
+ }
return r;
}
return r;
}