Add interceptors for the strtoi(3)/strtou(3) from NetBSD

[krytarowski]

Summary

strtoi/strtou converts string value to an intmax_t/uintmax_t integer.

Add a dedicated test.

Enable this API for NetBSD.

It's a reworked version of the original work by Yang Zheng.

[vitalybuka]

lib/sanitizer_common/sanitizer_common_interceptors.inc

7303 ↗

(On Diff #174619)

There is INTERCEPTORS_STRTO_BASE in msan
Probably we should try to move them into common, and than add new interceptors.
If this is hard to move, then just add strtoi there

[krytarowski]

lib/sanitizer_common/sanitizer_common_interceptors.inc

7303 ↗

(On Diff #174619)

Do you mean, to add strtoi for MSan only?

Yes, the interceptors shall be made generic.. however we are not able to do it in this iteration.

[krytarowski]

lib/sanitizer_common/sanitizer_common_interceptors.inc

7303 ↗

(On Diff #174619)

BTW. I will drop you a line in a mail.

[vitalybuka]

lib/sanitizer_common/sanitizer_common_interceptors.inc

7326 ↗

(On Diff #174619)

we need StrtolFixAndCheck and avoid reading entire string for strict_string_checks=0

[vitalybuka]

lib/sanitizer_common/sanitizer_common_interceptors.inc

7303 ↗

(On Diff #174619)

not sure what is long term is better
maybe adding this just into msan and later converting to _common would be easier

[krytarowski]

• rework the interceptor

[krytarowski]

• simplify

[vitalybuka]

test/sanitizer_common/TestCases/NetBSD/strtoi.cc

70 ↗

(On Diff #177109)

could you please make a single test function with arguments
now they are almost identical

[krytarowski]

test/sanitizer_common/TestCases/NetBSD/strtoi.cc

70 ↗

(On Diff #177109)

I will do it.

compiler-rt/trunk/lib/sanitizer_common/sanitizer_common_interceptors.inc

 }
 #define INIT_STATVFS1                  \
   COMMON_INTERCEPT_FUNCTION(statvfs1);  \
   COMMON_INTERCEPT_FUNCTION(fstatvfs1);
 #else
 #define INIT_STATVFS1
 #endif
 
+#if SANITIZER_INTERCEPT_STRTOI
+INTERCEPTOR(INTMAX_T, strtoi, const char *nptr, char **endptr, int base,
+            INTMAX_T low, INTMAX_T high, int *rstatus) {
+  void *ctx;
+  COMMON_INTERCEPTOR_ENTER(ctx, strtoi, nptr, endptr, base, low, high, rstatus);
+  char *real_endptr;
+  INTMAX_T ret = REAL(strtoi)(nptr, &real_endptr, base, low, high, rstatus);
+  StrtolFixAndCheck(ctx, nptr, endptr, real_endptr, base);
+  if (rstatus)
+    COMMON_INTERCEPTOR_WRITE_RANGE(ctx, rstatus, sizeof(*rstatus));
+  return ret;
+}
+
+INTERCEPTOR(UINTMAX_T, strtou, const char *nptr, char **endptr, int base,
+            UINTMAX_T low, UINTMAX_T high, int *rstatus) {
+  void *ctx;
+  COMMON_INTERCEPTOR_ENTER(ctx, strtou, nptr, endptr, base, low, high, rstatus);
+  char *real_endptr;
+  UINTMAX_T ret = REAL(strtou)(nptr, &real_endptr, base, low, high, rstatus);
+  StrtolFixAndCheck(ctx, nptr, endptr, real_endptr, base);
+  if (rstatus)
+    COMMON_INTERCEPTOR_WRITE_RANGE(ctx, rstatus, sizeof(*rstatus));
+  return ret;
+}
+#define INIT_STRTOI                                                            \
+  COMMON_INTERCEPT_FUNCTION(strtoi);                                           \
+  COMMON_INTERCEPT_FUNCTION(strtou)
+#else
+#define INIT_STRTOI
+#endif
+
 static void InitializeCommonInterceptors() {
   static u64 metadata_mem[sizeof(MetadataHashMap) / sizeof(u64) + 1];
   interceptor_metadata_map = new((void *)&metadata_mem) MetadataHashMap();
 
   INIT_MMAP;
   INIT_MMAP64;
   INIT_TEXTDOMAIN;
   INIT_STRLEN;
static void InitializeCommonInterceptors() {
   INIT_SYSCTL;
   INIT_ASYSCTL;
   INIT_SYSCTLGETMIBINFO;
   INIT_NL_LANGINFO;
   INIT_MODCTL;
   INIT_STRTONUM;
   INIT_FPARSELN;
   INIT_STATVFS1;
+  INIT_STRTOI;
 
   INIT___PRINTF_CHK;
 }

compiler-rt/trunk/lib/sanitizer_common/sanitizer_platform_interceptors.h

 #define SANITIZER_INTERCEPT_SYSCTL (SI_NETBSD || SI_FREEBSD)
 #define SANITIZER_INTERCEPT_ASYSCTL SI_NETBSD
 #define SANITIZER_INTERCEPT_SYSCTLGETMIBINFO SI_NETBSD
 #define SANITIZER_INTERCEPT_NL_LANGINFO (SI_NETBSD || SI_FREEBSD)
 #define SANITIZER_INTERCEPT_MODCTL SI_NETBSD
 #define SANITIZER_INTERCEPT_STRTONUM SI_NETBSD
 #define SANITIZER_INTERCEPT_FPARSELN SI_NETBSD
 #define SANITIZER_INTERCEPT_STATVFS1 SI_NETBSD
+#define SANITIZER_INTERCEPT_STRTOI SI_NETBSD
 
 #endif  // #ifndef SANITIZER_PLATFORM_INTERCEPTORS_H

compiler-rt/trunk/test/sanitizer_common/TestCases/NetBSD/strtoi.cc

+// RUN: %clangxx -O0 -g %s -o %t && %run %t 2>&1 | FileCheck %s
+
+#include <inttypes.h>
+#include <stdio.h>
+
+void test_strtoi(const char *nptr, int base, intmax_t lo, intmax_t hi) {
+  char *p;
+  int status;
+  intmax_t i = strtoi(nptr, &p, base, lo, hi, &status);
+  printf("strtoi: conversion of '%s' to a number %s, using %jd, p=%#" PRIx8
+         "\n",
+         nptr, status ? "failed" : "successful", i, *p);
+}
+
+void test_strtou(const char *nptr, int base, intmax_t lo, intmax_t hi) {
+  char *p;
+  int status;
+  uintmax_t i = strtou(nptr, &p, base, lo, hi, &status);
+  printf("strtou: conversion of '%s' to a number %s, using %ju, p=%#" PRIx8
+         "\n",
+         nptr, status ? "failed" : "successful", i, *p);
+}
+
+int main(void) {
+  printf("strtoi\n");
+
+  test_strtoi("100", 0, 1, 100);
+  test_strtoi("100", 0, 1, 10);
+  test_strtoi("100xyz", 0, 1, 100);
+  test_strtou("100", 0, 1, 100);
+  test_strtou("100", 0, 1, 10);
+  test_strtou("100xyz", 0, 1, 100);
+
+  // CHECK: strtoi
+  // CHECK: strtoi: conversion of '100' to a number successful, using 100, p=0
+  // CHECK: strtoi: conversion of '100' to a number failed, using 10, p=0
+  // CHECK: strtoi: conversion of '100xyz' to a number failed, using 10, p=0x78
+  // CHECK: strtou: conversion of '100' to a number successful, using 100, p=0
+  // CHECK: strtou: conversion of '100' to a number failed, using 10, p=0
+  // CHECK: strtou: conversion of '100xyz' to a number failed, using 10, p=0x78
+
+  return 0;
+}