commit d0a0f72852d45db213886db606692e7b204207bc
author George Burgess IV <gbiv@google.com> Wed Sep 18 16:53:57 2019 -0700
committer George Burgess IV <gbiv@google.com> Mon Sep 30 13:00:35 2019 -0700
tree ff5c8932db010e582788a7d78d47f3d1df8be618
parent 82ee3a08e213d9b4ef98189a7d869048c7b48817

fortify(socket/stat): emit diagnostics regardless of API level

Bug: 141267932
Test: m checkbuild on internal-master

Change-Id: I7fe29184290e19ec080b204aa98d513ce59bc66b

libc/include/bits/fortify/socket.h

diff --git a/libc/include/bits/fortify/socket.h b/libc/include/bits/fortify/socket.h
index 35fad3d..cf5f189 100644
--- a/libc/include/bits/fortify/socket.h
+++ b/libc/include/bits/fortify/socket.h
@@ -37,35 +37,35 @@
 
 #if defined(__BIONIC_FORTIFY)
 
-#if __ANDROID_API__ >= __ANDROID_API_N__
 __BIONIC_FORTIFY_INLINE
 ssize_t recvfrom(int fd, void* const buf __pass_object_size0, size_t len, int flags, struct sockaddr* src_addr, socklen_t* addr_len)
     __overloadable
     __clang_error_if(__bos_unevaluated_lt(__bos0(buf), len),
                      "'recvfrom' called with size bigger than buffer") {
+#if __ANDROID_API__ >= __ANDROID_API_N__
   size_t bos = __bos0(buf);
 
-  if (__bos_trivially_not_lt(bos, len)) {
-    return __call_bypassing_fortify(recvfrom)(fd, buf, len, flags, src_addr, addr_len);
+  if (!__bos_trivially_ge(bos, len)) {
+    return __recvfrom_chk(fd, buf, len, bos, flags, src_addr, addr_len);
   }
-  return __recvfrom_chk(fd, buf, len, bos, flags, src_addr, addr_len);
-}
 #endif /* __ANDROID_API__ >= __ANDROID_API_N__ */
+  return __call_bypassing_fortify(recvfrom)(fd, buf, len, flags, src_addr, addr_len);
+}
 
-#if __ANDROID_API__ >= __ANDROID_API_N_MR1__
 __BIONIC_FORTIFY_INLINE
 ssize_t sendto(int fd, const void* const buf __pass_object_size0, size_t len, int flags, const struct sockaddr* dest_addr, socklen_t addr_len)
     __overloadable
     __clang_error_if(__bos_unevaluated_lt(__bos0(buf), len),
                      "'sendto' called with size bigger than buffer") {
+#if __ANDROID_API__ >= __ANDROID_API_N_MR1__
   size_t bos = __bos0(buf);
 
-  if (__bos_trivially_not_lt(bos, len)) {
-    return __call_bypassing_fortify(sendto)(fd, buf, len, flags, dest_addr, addr_len);
+  if (!__bos_trivially_ge(bos, len)) {
+    return __sendto_chk(fd, buf, len, bos, flags, dest_addr, addr_len);
   }
-  return __sendto_chk(fd, buf, len, bos, flags, dest_addr, addr_len);
-}
 #endif /* __ANDROID_API__ >= __ANDROID_API_N_MR1__ */
+  return __call_bypassing_fortify(sendto)(fd, buf, len, flags, dest_addr, addr_len);
+}
 
 __BIONIC_FORTIFY_INLINE
 ssize_t recv(int socket, void* const buf __pass_object_size0, size_t len, int flags)