From b7a6e42e9148427e249ba5f59273c4064bfe55f0 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Zbigniew=20J=C4=99drzejewski-Szmek?= Date: Fri, 11 Oct 2013 19:33:36 -0400 Subject: [PATCH] drop-ins: check return value If the function failed, nothing serious would happen because unlink would probably return EFAULT, but this would obscure the real error and is a bit sloppy. Conflicts: src/core/unit.c --- src/core/unit.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/core/unit.c b/src/core/unit.c index 502fbf1d4..52484022c 100644 --- a/src/core/unit.c +++ b/src/core/unit.c @@ -2940,6 +2940,9 @@ int unit_remove_drop_in(Unit *u, bool runtime, const char *name) { assert(u); r = drop_in_file(u, runtime, name, &p, &q); + if (r < 0) + return r; + if (unlink(q) < 0) r = -errno; else