From dfd34e531fc5bbbdb8b6bc4efd7d19866ddba9c7 Mon Sep 17 00:00:00 2001 From: Branden Archer Date: Tue, 9 May 2017 22:29:19 -0400 Subject: [PATCH] Bail on error if cannot create pipe file There have been cases where a test will fail with "No messaging setup". This is believed to be a permissions issue, where the test process is unable to create a file in the folder selected for temporary files. To make this error more explicit, if creating a temporary file fails bail. --- src/check_msg.c | 8 ++++++++ 1 file changed, 8 insertions(+) diff --git a/src/check_msg.c b/src/check_msg.c index 9483654b..985eca0f 100644 --- a/src/check_msg.c +++ b/src/check_msg.c @@ -315,11 +315,19 @@ static void setup_pipe(void) if(send_file1 == NULL) { send_file1 = open_tmp_file(&send_file1_name); + if(send_file1 == NULL) + { + eprintf("Unable to create temporary file for communication; may not have permissions to do so", __FILE__, __LINE__ -3); + } return; } if(send_file2 == NULL) { send_file2 = open_tmp_file(&send_file2_name); + if(send_file2 == NULL) + { + eprintf("Unable to create temporary file for communication; may not have permissions to do so", __FILE__, __LINE__ -3); + } return; } eprintf("Only one nesting of suite runs supported", __FILE__, __LINE__);