activation test: don't crash if AppArmor is built but unavailable

Also don't try to clean up a process we didn't start.

Bug: https://bugs.freedesktop.org/show_bug.cgi?id=98666
Signed-off-by: Simon McVittie <smcv@debian.org>
Reviewed-by: Philip Withnall <philip.withnall@collabora.co.uk>

diff --git a/test/sd-activation.c b/test/sd-activation.c
index 058a247..da63f7d 100644 (file)
--- a/test/sd-activation.c
+++ b/test/sd-activation.c
@@ -801,9 +801,15 @@ teardown (Fixture *f,
       f->activated = NULL;
     }
 
-  test_kill_pid (f->daemon_pid);
-  g_spawn_close_pid (f->daemon_pid);
-  test_main_context_unref (f->ctx);
+  if (f->daemon_pid != 0)
+    {
+      test_kill_pid (f->daemon_pid);
+      g_spawn_close_pid (f->daemon_pid);
+    }
+
+  if (f->ctx != NULL)
+    test_main_context_unref (f->ctx);
+
   g_free (f->address);
 }