[compiler-rt] Add a symbolizer testcase for closed stdin/stdout

[kubamracek]

Summary

This testcase just tests that the external symbolizer (llvm-symbolizer) can start and communicate correctly even when the program closed its stdin, stdout and stderr.

[samsonov]

Good idea

test/asan/TestCases/closed-fds.cc

18

Maybe, we should instead check fprintf return value?

25

Interleave CHECK lines with code...

30

to make this block actually close to the //BOOM line

[kubamracek]

Addressing review comments.

[samsonov]

LGTM

test/asan/TestCases/closed-fds.cc

19

CHECK-OUTPUT doesn't seem to be needed here at all.

[kubamracek]

Committed r231429.

test/asan/TestCases/closed-fds.cc

+// Check that when the program closed its std(in|out|err), running the external
+// symbolizer still works.
+
+// RUN: rm -f %t.log.*
+// RUN: %clangxx_asan -O0 %s -o %t 2>&1 && ASAN_OPTIONS=log_path=%t.log:verbosity=2 not %run %t 2>&1 | FileCheck %s --check-prefix=CHECK-OUTPUT
+// RUN: FileCheck %s --check-prefix=CHECK-FILE < %t.log.*
+
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <unistd.h>
+
+int main(int argc, char **argv) {
+  fprintf(stderr, "Closing streams.\n");
+  close(STDIN_FILENO);
+  close(STDOUT_FILENO);
+  close(STDERR_FILENO);
+  fprintf(stderr, "Can you hear me now?\n");
+  char *x = (char*)malloc(10 * sizeof(char));
+  free(x);
+  x[argc] = 'X';  // BOOM
+  return 0;
+}
+
+// CHECK-OUTPUT: Closing streams.
+// CHECK-OUTPUT-NOT: Can you hear me now?
+// CHECK-FILE: {{.*ERROR: AddressSanitizer: heap-use-after-free on address}}
+// CHECK-FILE:   {{0x.* at pc 0x.* bp 0x.* sp 0x.*}}
+// CHECK-FILE: {{WRITE of size 1 at 0x.* thread T0}}
+// CHECK-FILE: {{    #0 0x.* in main .*closed-fds.cc:}}[[@LINE-9]]