[compiler-rt] [sanitizer_common] Add tests for more stdio.h functions

[mgorny]

Summary

Add two new test cases that test the following stdio.h functions:

• clearerr()
• feof()
• ferror()
• fileno()
• fgetc()
• getc()
• ungetc()

[krytarowski]

test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc

7

Please move local variable declaration to the first line used (whenever possible)

test/sanitizer_common/TestCases/Posix/fgetc_ungetc_getc.cc

6

FILE *fp = fp = fopen(argv[0], "r");

[vitalybuka]

test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc

49

why does this close file only on success?
I assume normal test behavior is return 0;
So please replace all return N with asserts so when it fail we will have verbose output
e.g

assert(close(fd) != -1);
....
assert(ferror(fp));

[mgorny]

test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc

49

Will do. I've copied this from fgets.cc, fputs_puts.cc tests, and wrongly presumed it's expected coding style.

[mgorny]

Moved variable definitions, and added asserts.

[krytarowski]

test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc

39

assert(!fclose(fp));

test/sanitizer_common/TestCases/Posix/fgetc_ungetc_getc.cc

18

assert(!fclose(fp));

[krytarowski]

test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc

39

alternatively != EOF

[mgorny]

test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc

39

It will fail most likely, due to us closing the fd before. However, I'm not sure if we want to strictly rely on this error condition.

[mgorny]

(Do you have another idea how to reliably trigger ferror?)

[krytarowski]

test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc

39

Ah, OK. Please just add a comment that this call might fail.

[mgorny]

Added asserts for fclose().

test/sanitizer_common/TestCases/Posix/feof_fileno_ferror.cc

+// RUN: %clangxx -g %s -o %t && %run %t
+
+#include <stdio.h>
+#include <unistd.h>
+
+int main(int argc, char **argv) {
+  FILE *fp;
+  int fd;
+  char buf[BUFSIZ];
+
+  fp = fopen(argv[0], "r");
+  if (!fp)
+    return 1;
+
+  // file should be good upon opening
+  if (feof(fp) || ferror(fp))
+    return 2;
+
+  // read until EOF
+  while (fread(buf, 1, sizeof buf, fp) != 0) {}
+  if (!feof(fp))
+    return 3;
+
+  // clear EOF
+  clearerr(fp);
+  if (feof(fp) || ferror(fp))
+    return 4;
+
+  // get file descriptor
+  fd = fileno(fp);
+  if (fd == -1)
+    return 5;
+
+  // break the file by closing underlying descriptor
+  if (close(fd) == -1)
+    return 6;
+
+  // verify that an error is signalled
+  if (fread(buf, 1, sizeof buf, fp) != 0)
+    return 7;
+  if (!ferror(fp))
+    return 8;
+
+  // clear error
+  clearerr(fp);
+  if (feof(fp) || ferror(fp))
+    return 9;
+
+  fclose(fp);
+  return 0;
+}

test/sanitizer_common/TestCases/Posix/fgetc_ungetc_getc.cc

+// RUN: %clangxx -g %s -o %t && %run %t
+
+#include <stdio.h>
+
+int main(int argc, char **argv) {
+  FILE *fp;
+
+  fp = fopen(argv[0], "r");
+  if (!fp)
+    return 1;
+
+  if (fgetc(fp) == EOF)
+    return 2;
+
+  if (ungetc('X', fp) == EOF)
+    return 3;
+
+  if (getc(fp) != 'X')
+    return 4;
+
+  fclose(fp);
+  return 0;
+}