[PATCH i-g-t v3 2/3] runner/executor: Fix error handling at terminating test

[Kamil Konieczny <kamil.konieczny@linux.intel.com>]

Instead of just returning and leaking memory and file descriptors
inform user about an error which occurred and terminate
gracefully.

Signed-off-by: Kamil Konieczny <kamil.konieczny@linux.intel.com>
Reviewed-by: Jan Sokolowski <jan.sokolowski@intel.com>
Reviewed-by: Ryszard Knop <rk@dragonic.eu>
---
 runner/executor.c | 8 ++++++--
 1 file changed, 6 insertions(+), 2 deletions(-)

diff --git a/runner/executor.c b/runner/executor.c
index 16b12ba16..82e5ca5eb 100644
--- a/runner/executor.c
+++ b/runner/executor.c
@@ -1510,8 +1510,12 @@ static int monitor_output(pid_t child,
 			}
 
 			killed = next_kill_signal(killed);
-			if (!kill_child(killed, child))
-				return -1;
+			if (!kill_child(killed, child)) {
+				errf("Error at terminating test with %s, errno=%d\n",
+				     killed == SIGQUIT ? "SIGQUIT" : "SIGKILL", errno);
+				killed = -1;
+				break; /* while */
+			}
 			time_killed = time_now;
 		}
 	}
-- 
2.47.1