diff options
| author | Lennart Poettering <lennart@poettering.net> | 2012-10-16 19:50:26 +0200 |
|---|---|---|
| committer | Lennart Poettering <lennart@poettering.net> | 2012-10-16 19:50:26 +0200 |
| commit | b87705cdd2f791f8520edb78791d3e6f78afd481 (patch) | |
| tree | d848be1753e3ac2537d9183a751396d4e6837200 /src/core/socket.c | |
| parent | 770858811930c0658b189d980159ea1ac5663467 (diff) | |
mount: don't try to initialize extra deps for mount units before initializing their basic fields
Under some circumstances this could lead to a segfault since we we
half-initialized a mount unit, then tried to hook it into the network of
things and while doing that recursively ended up looking at our
half-initialized mount unit again assuming it was fully initialized.
Diffstat (limited to 'src/core/socket.c')
| -rw-r--r-- | src/core/socket.c | 9 |
1 files changed, 6 insertions, 3 deletions
diff --git a/src/core/socket.c b/src/core/socket.c index 26e7fd24de..71cdf2dfc8 100644 --- a/src/core/socket.c +++ b/src/core/socket.c @@ -265,7 +265,8 @@ int socket_add_one_mount_link(Socket *s, Mount *m) { if (!socket_needs_mount(s, m->where)) return 0; - if ((r = unit_add_two_dependencies(UNIT(s), UNIT_AFTER, UNIT_REQUIRES, UNIT(m), true)) < 0) + r = unit_add_two_dependencies(UNIT(s), UNIT_AFTER, UNIT_REQUIRES, UNIT(m), true); + if (r < 0) return r; return 0; @@ -277,9 +278,11 @@ static int socket_add_mount_links(Socket *s) { assert(s); - LIST_FOREACH(units_by_type, other, UNIT(s)->manager->units_by_type[UNIT_MOUNT]) - if ((r = socket_add_one_mount_link(s, MOUNT(other))) < 0) + LIST_FOREACH(units_by_type, other, UNIT(s)->manager->units_by_type[UNIT_MOUNT]) { + r = socket_add_one_mount_link(s, MOUNT(other)); + if (r < 0) return r; + } return 0; } |