[libc-commits] [libc] [libc] Port remaining socket functions to syscall_wrappers (PR #198463)

[Jeff Bailey via libc-commits]

================
@@ -1,32 +1,33 @@
-//===-- Linux implementation of bind --------------------------------------===//
+//===----------------------------------------------------------------------===//
 //
 // Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
 // See https://llvm.org/LICENSE.txt for license information.
 // SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
 //
 //===----------------------------------------------------------------------===//
-
+///
+/// \file
+/// Linux implementation of bind.
+///
+//===----------------------------------------------------------------------===//
 #include "src/sys/socket/bind.h"
-
-#include "src/__support/OSUtil/syscall.h" // For internal syscall function.
+#include "hdr/types/socklen_t.h"
+#include "hdr/types/struct_sockaddr.h"
+#include "src/__support/OSUtil/linux/syscall_wrappers/bind.h"
 #include "src/__support/common.h"
-
 #include "src/__support/libc_errno.h"
-#include "src/__support/macros/config.h"
-
-#include <sys/syscall.h> // For syscall numbers.
 
 namespace LIBC_NAMESPACE_DECL {
 
 LLVM_LIBC_FUNCTION(int, bind,
                    (int socket, const struct sockaddr *address,
                     socklen_t address_len)) {
-  int ret = syscall_impl<int>(SYS_bind, socket, address, address_len);
-  if (ret < 0) {
-    libc_errno = -ret;
+  auto result = linux_syscalls::bind(socket, address, address_len);
+  if (!result.has_value()) {
----------------
kaladron wrote:

We should use LIBC_CRASH_ON_NULLPTR on the pointer arguments (for all functions)


https://github.com/llvm/llvm-project/pull/198463