[libc-commits] [libc] [libc] Add the get and set sys/xattr.h entrypoints. (PR #224729)

[Alex Strelnikov via libc-commits]

================
@@ -0,0 +1,164 @@
+//===----------------------------------------------------------------------===//
+//
+// 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
+/// Unit tests for lsetxattr.
+///
+//===----------------------------------------------------------------------===//
+
+#include "hdr/sys_stat_macros.h"
+#include "hdr/sys_xattr_macros.h"
+#include "hdr/types/ssize_t.h"
+#include "src/__support/CPP/scope.h"
+#include "src/__support/OSUtil/linux/syscall.h"
+#include "src/__support/libc_errno.h"
+#include "src/fcntl/creat.h"
+#include "src/sys/xattr/lsetxattr.h"
+#include "src/unistd/close.h"
+#include "src/unistd/unlink.h"
+#include "test/UnitTest/ErrnoCheckingTest.h"
+#include "test/UnitTest/ErrnoSetterMatcher.h"
+#include "test/UnitTest/Test.h"
+#include <sys/syscall.h>
+
+namespace {
+
+using namespace LIBC_NAMESPACE::testing::ErrnoSetterMatcher;
+using LlvmLibcLsetxattrTest = LIBC_NAMESPACE::testing::ErrnoCheckingTest;
+using LIBC_NAMESPACE::cpp::scope_exit;
+using LIBC_NAMESPACE::cpp::string_view;
+
+int recreate_test_file(const char *path) {
+  LIBC_NAMESPACE::unlink(path);
+  LIBC_NAMESPACE::libc_errno = 0;
+  return LIBC_NAMESPACE::creat(path, S_IRWXU);
+}
+
+TEST_F(LlvmLibcLsetxattrTest, SetAttributeDefaultFlags) {
+  const LIBC_NAMESPACE::CString TEST_FILE_NAME =
+      libc_make_test_file_path("testdata/lsetxattr_default_flags.txt");
+
+  int fd = recreate_test_file(TEST_FILE_NAME);
+  ASSERT_ERRNO_SUCCESS();
+  ASSERT_THAT(LIBC_NAMESPACE::close(fd), Succeeds(0));
+  scope_exit cleanup([&] {
+    ASSERT_THAT(LIBC_NAMESPACE::unlink(TEST_FILE_NAME), Succeeds(0));
+  });
+
+  // Set an attribute through the test file name.
+  {
+    string_view XATTR_NAME = "user.test_attr_through_file";
+    string_view XATTR_VALUE = "test_value_through_file";
+    constexpr size_t BUFFER_SIZE = 32;
+    ASSERT_GE(BUFFER_SIZE, XATTR_VALUE.size());
+    char buffer[BUFFER_SIZE] = {};
+
+    ASSERT_THAT(LIBC_NAMESPACE::lsetxattr(TEST_FILE_NAME, XATTR_NAME.data(),
+                                          XATTR_VALUE.data(),
+                                          XATTR_VALUE.size(),
+                                          /* flags = */ 0),
+                Succeeds(0));
+
+    ASSERT_EQ(static_cast<ssize_t>(XATTR_VALUE.size()),
+              LIBC_NAMESPACE::syscall_impl<ssize_t>(
+                  SYS_getxattr, static_cast<const char *>(TEST_FILE_NAME),
+                  XATTR_NAME.data(), buffer, BUFFER_SIZE));
+    EXPECT_EQ(string_view(buffer, XATTR_VALUE.size()), XATTR_VALUE);
+  }
+
+  // TODO: Figure out if/how to test `lsetxattr` with a symlink path. The "user"
+  // namespace extended attributes appear restricted to regular files and
+  // directories. Attempting to set a "user" extended attribute on a symlink
+  // results in EPERM.
+}
+
+TEST_F(LlvmLibcLsetxattrTest, SetAttributeWithNonzeroFlags) {
+  const LIBC_NAMESPACE::CString TEST_FILE_NAME =
+      libc_make_test_file_path("testdata/lsetxattr_nonzero_flags.txt");
+  int fd = recreate_test_file(TEST_FILE_NAME);
----------------
strel-12 wrote:

Done

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