From 1acacd737b2a831e7cce4049d42df254718a4543 Mon Sep 17 00:00:00 2001
From: Lennart Poettering <lennart@poettering.net>
Date: Mon, 19 Nov 2018 13:05:43 +0100
Subject: [PATCH] test-execute: fix killing of unit processes

Let's use the correct wrapper for calling unit vtable functions. Let's
make sure we always use the right wrappers, and never bypass them
needlessly.

Moreover use SIGKILL rather than "9" as signal name. Let's not be
needlessly cryptic.

Follow-up for: f7f8e8cbb953686c520969a66a9718675a972af5
---
 src/test/test-execute.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/test/test-execute.c b/src/test/test-execute.c
index 0e378f68c0..18a021dcea 100644
--- a/src/test/test-execute.c
+++ b/src/test/test-execute.c
@@ -54,7 +54,7 @@ static void check(Manager *m, Unit *unit, int status_expected, int code_expected
                 n = now(CLOCK_MONOTONIC);
                 if (ts + timeout < n) {
                         log_error("Test timeout when testing %s", unit->id);
-                        r = UNIT_VTABLE(unit)->kill(unit, KILL_ALL, 9, NULL);
+                        r = unit_kill(unit, KILL_ALL, SIGKILL, NULL);
                         if (r < 0)
                                 log_error_errno(r, "Failed to kill %s: %m", unit->id);
                         exit(EXIT_FAILURE);