iox-eclipse-iceoryx#2289 Add 'iox' prefix to entities in 'unistd.hpp'

iceoryx_hoofs/posix/auth/include/iox/posix_group.hpp

@@ -31,23 +31,23 @@ class PosixGroup
 {
   public:
     using groupName_t = string<platform::MAX_GROUP_NAME_LENGTH>;
-    explicit PosixGroup(const gid_t id) noexcept;
+    explicit PosixGroup(const iox_gid_t id) noexcept;
     explicit PosixGroup(const groupName_t& name) noexcept;
 
     bool operator==(const PosixGroup& other) const noexcept;
 
     groupName_t getName() const noexcept;
-    gid_t getID() const noexcept;
+    iox_gid_t getID() const noexcept;
 
     bool doesExist() const noexcept;
 
     static PosixGroup getGroupOfCurrentProcess() noexcept;
 
-    static optional<uid_t> getGroupID(const groupName_t& name) noexcept;
-    static optional<groupName_t> getGroupName(gid_t id) noexcept;
+    static optional<iox_uid_t> getGroupID(const groupName_t& name) noexcept;
+    static optional<groupName_t> getGroupName(iox_gid_t id) noexcept;
 
   private:
-    gid_t m_id;
+    iox_gid_t m_id;
     bool m_doesExist{false};
 };
 

iceoryx_hoofs/posix/auth/include/iox/posix_user.hpp

@@ -37,22 +37,22 @@ class PosixUser
 
     using userName_t = string<platform::MAX_USER_NAME_LENGTH>;
 
-    explicit PosixUser(const uid_t id) noexcept;
+    explicit PosixUser(const iox_uid_t id) noexcept;
     explicit PosixUser(const userName_t& name) noexcept;
 
     groupVector_t getGroups() const noexcept;
     userName_t getName() const noexcept;
-    uid_t getID() const noexcept;
+    iox_uid_t getID() const noexcept;
 
     bool doesExist() const noexcept;
 
     static PosixUser getUserOfCurrentProcess() noexcept;
 
-    static optional<uid_t> getUserID(const userName_t& name) noexcept;
-    static optional<userName_t> getUserName(uid_t id) noexcept;
+    static optional<iox_uid_t> getUserID(const userName_t& name) noexcept;
+    static optional<userName_t> getUserName(iox_uid_t id) noexcept;
 
   private:
-    uid_t m_id;
+    iox_uid_t m_id;
     bool m_doesExist{false};
 };
 

iceoryx_hoofs/posix/auth/source/posix_group.cpp

@@ -27,7 +27,7 @@
 
 namespace iox
 {
-PosixGroup::PosixGroup(gid_t id) noexcept
+PosixGroup::PosixGroup(iox_gid_t id) noexcept
     : m_id(id)
     , m_doesExist(getGroupName(id).has_value())
 {
@@ -43,7 +43,7 @@ PosixGroup::PosixGroup(const PosixGroup::groupName_t& name) noexcept
     else
     {
         IOX_LOG(ERROR, "Error: Group name not found");
-        m_id = std::numeric_limits<gid_t>::max();
+        m_id = std::numeric_limits<iox_gid_t>::max();
     }
 }
 
@@ -54,10 +54,10 @@ bool PosixGroup::operator==(const PosixGroup& other) const noexcept
 
 PosixGroup PosixGroup::getGroupOfCurrentProcess() noexcept
 {
-    return PosixGroup(getgid());
+    return PosixGroup(iox_getgid());
 }
 
-optional<gid_t> PosixGroup::getGroupID(const PosixGroup::groupName_t& name) noexcept
+optional<iox_gid_t> PosixGroup::getGroupID(const PosixGroup::groupName_t& name) noexcept
 {
     auto getgrnamCall = IOX_POSIX_CALL(getgrnam)(name.c_str()).failureReturnValue(nullptr).evaluate();
 
@@ -67,10 +67,10 @@ optional<gid_t> PosixGroup::getGroupID(const PosixGroup::groupName_t& name) noex
         return nullopt_t();
     }
 
-    return make_optional<gid_t>(getgrnamCall->value->gr_gid);
+    return make_optional<iox_gid_t>(getgrnamCall->value->gr_gid);
 }
 
-optional<PosixGroup::groupName_t> PosixGroup::getGroupName(gid_t id) noexcept
+optional<PosixGroup::groupName_t> PosixGroup::getGroupName(iox_gid_t id) noexcept
 {
     auto getgrgidCall = IOX_POSIX_CALL(getgrgid)(id).failureReturnValue(nullptr).evaluate();
 
@@ -94,7 +94,7 @@ PosixGroup::groupName_t PosixGroup::getName() const noexcept
     return groupName_t();
 }
 
-gid_t PosixGroup::getID() const noexcept
+iox_gid_t PosixGroup::getID() const noexcept
 {
     return m_id;
 }

iceoryx_hoofs/posix/auth/source/posix_user.cpp

@@ -20,6 +20,7 @@
 #include "iceoryx_platform/platform_correction.hpp"
 #include "iceoryx_platform/pwd.hpp"
 #include "iceoryx_platform/types.hpp"
+#include "iceoryx_platform/unistd.hpp"
 #include "iox/logging.hpp"
 #include "iox/posix_call.hpp"
 #include "iox/uninitialized_array.hpp"
@@ -28,7 +29,7 @@
 
 namespace iox
 {
-optional<uid_t> PosixUser::getUserID(const userName_t& name) noexcept
+optional<iox_uid_t> PosixUser::getUserID(const userName_t& name) noexcept
 {
     auto getpwnamCall = IOX_POSIX_CALL(getpwnam)(name.c_str()).failureReturnValue(nullptr).evaluate();
 
@@ -37,7 +38,7 @@ optional<uid_t> PosixUser::getUserID(const userName_t& name) noexcept
         IOX_LOG(ERROR, "Error: Could not find user '" << name << "'.");
         return nullopt_t();
     }
-    return make_optional<uid_t>(getpwnamCall->value->pw_uid);
+    return make_optional<iox_uid_t>(getpwnamCall->value->pw_uid);
 }
 
 optional<PosixUser::userName_t> PosixUser::getUserName(uid_t id) noexcept
@@ -67,8 +68,8 @@ PosixUser::groupVector_t PosixUser::getGroups() const noexcept
         return groupVector_t();
     }
 
-    gid_t userDefaultGroup = getpwnamCall->value->pw_gid;
-    UninitializedArray<gid_t, MAX_NUMBER_OF_GROUPS> groups{}; // groups is initialized in iox_getgrouplist
+    iox_gid_t userDefaultGroup = getpwnamCall->value->pw_gid;
+    UninitializedArray<iox_gid_t, MAX_NUMBER_OF_GROUPS> groups{}; // groups is initialized in iox_getgrouplist
     int numGroups = MAX_NUMBER_OF_GROUPS;
 
     auto getgrouplistCall =
@@ -96,7 +97,7 @@ PosixUser::groupVector_t PosixUser::getGroups() const noexcept
     return vec;
 }
 
-PosixUser::PosixUser(uid_t id) noexcept
+PosixUser::PosixUser(iox_uid_t id) noexcept
     : m_id(id)
     , m_doesExist(getUserName(id).has_value())
 {
@@ -112,7 +113,7 @@ PosixUser::PosixUser(const PosixUser::userName_t& name) noexcept
     else
     {
         IOX_LOG(ERROR, "Error: User name not found");
-        m_id = std::numeric_limits<gid_t>::max();
+        m_id = std::numeric_limits<iox_gid_t>::max();
     }
 }
 
@@ -127,7 +128,7 @@ PosixUser::userName_t PosixUser::getName() const noexcept
     return userName_t();
 }
 
-uid_t PosixUser::getID() const noexcept
+iox_uid_t PosixUser::getID() const noexcept
 {
     return m_id;
 }
@@ -139,7 +140,7 @@ bool PosixUser::doesExist() const noexcept
 
 PosixUser PosixUser::getUserOfCurrentProcess() noexcept
 {
-    return PosixUser(geteuid());
+    return PosixUser(iox_geteuid());
 }
 
 } // namespace iox

iceoryx_hoofs/posix/design/include/iox/file_management_interface.hpp

@@ -61,7 +61,7 @@ enum class FileSetPermissionError : uint8_t
 namespace details
 {
 expected<iox_stat, FileStatError> get_file_status(const int fildes) noexcept;
-expected<void, FileSetOwnerError> set_owner(const int fildes, const uid_t uid, const gid_t gid) noexcept;
+expected<void, FileSetOwnerError> set_owner(const int fildes, const iox_uid_t uid, const iox_gid_t gid) noexcept;
 expected<void, FileSetPermissionError> set_permissions(const int fildes, const access_rights perms) noexcept;
 } // namespace details
 
@@ -71,16 +71,16 @@ class Ownership
   public:
     /// @brief Acquire the user id
     /// @returns uid
-    uid_t uid() const noexcept;
+    iox_uid_t uid() const noexcept;
 
     /// @brief Acquire the group id
     /// @returns gid
-    gid_t gid() const noexcept;
+    iox_gid_t gid() const noexcept;
 
     /// @brief Constructs a ownership object from a uid and a gid.
     /// @returns If the user or group does not exist it returns 'iox::nullopt' otherwise an Ownership object
     ///             with existing user and group
-    static optional<Ownership> from_user_and_group(const uid_t uid, const gid_t gid) noexcept;
+    static optional<Ownership> from_user_and_group(const iox_uid_t uid, const iox_gid_t gid) noexcept;
 
     /// @brief Constructs a ownership object from a user name and a group name.
     /// @returns If the user or group does not exist it returns 'iox::nullopt' otherwise an Ownership object
@@ -93,11 +93,11 @@ class Ownership
   private:
     template <typename>
     friend struct FileManagementInterface;
-    Ownership(const uid_t uid, const gid_t gid) noexcept;
+    Ownership(const iox_uid_t uid, const iox_gid_t gid) noexcept;
 
   private:
-    uid_t m_uid{std::numeric_limits<uid_t>::max()};
-    gid_t m_gid{std::numeric_limits<gid_t>::max()};
+    iox_uid_t m_uid{std::numeric_limits<iox_uid_t>::max()};
+    iox_gid_t m_gid{std::numeric_limits<iox_gid_t>::max()};
 };
 
 /// @brief Abstract implementation to manage things common to all file descriptor

iceoryx_hoofs/posix/design/source/file_management_interface.cpp

@@ -52,7 +52,7 @@ expected<iox_stat, FileStatError> get_file_status(const int fildes) noexcept
     return ok(file_status);
 }
 
-expected<void, FileSetOwnerError> set_owner(const int fildes, const uid_t uid, const gid_t gid) noexcept
+expected<void, FileSetOwnerError> set_owner(const int fildes, const iox_uid_t uid, const iox_gid_t gid) noexcept
 {
     auto result = IOX_POSIX_CALL(iox_fchown)(fildes, uid, gid).failureReturnValue(-1).evaluate();
 
@@ -118,17 +118,17 @@ expected<void, FileSetPermissionError> set_permissions(const int fildes, const a
 
 } // namespace details
 
-uid_t Ownership::uid() const noexcept
+iox_uid_t Ownership::uid() const noexcept
 {
     return m_uid;
 }
 
-gid_t Ownership::gid() const noexcept
+iox_gid_t Ownership::gid() const noexcept
 {
     return m_gid;
 }
 
-optional<Ownership> Ownership::from_user_and_group(const uid_t uid, const gid_t gid) noexcept
+optional<Ownership> Ownership::from_user_and_group(const iox_uid_t uid, const iox_gid_t gid) noexcept
 {
     if (!PosixUser(uid).doesExist() || !PosixGroup(gid).doesExist())
     {
@@ -156,7 +156,7 @@ Ownership Ownership::from_process() noexcept
     return Ownership(PosixUser::getUserOfCurrentProcess().getID(), PosixGroup::getGroupOfCurrentProcess().getID());
 }
 
-Ownership::Ownership(const uid_t uid, const gid_t gid) noexcept
+Ownership::Ownership(const iox_uid_t uid, const iox_gid_t gid) noexcept
     : m_uid{uid}
     , m_gid{gid}
 {

iceoryx_hoofs/posix/filesystem/source/file.cpp

@@ -203,7 +203,7 @@ void File::close_fd() noexcept
 
 expected<bool, FileAccessError> File::does_exist(const FilePath& file) noexcept
 {
-    auto result = IOX_POSIX_CALL(iox_access)(file.as_string().c_str(), F_OK).failureReturnValue(-1).evaluate();
+    auto result = IOX_POSIX_CALL(iox_access)(file.as_string().c_str(), IOX_F_OK).failureReturnValue(-1).evaluate();
 
     if (!result.has_error())
     {

iceoryx_hoofs/posix/ipc/source/posix_shared_memory.cpp

@@ -129,7 +129,7 @@ expected<PosixSharedMemory, PosixSharedMemoryError> PosixSharedMemoryBuilder::cr
 
     if (hasOwnership)
     {
-        auto result = IOX_POSIX_CALL(ftruncate)(sharedMemoryFileHandle, static_cast<int64_t>(m_size))
+        auto result = IOX_POSIX_CALL(iox_ftruncate)(sharedMemoryFileHandle, static_cast<int64_t>(m_size))
                           .failureReturnValue(PosixSharedMemory::INVALID_HANDLE)
                           .evaluate();
         if (result.has_error())

iceoryx_hoofs/posix/utility/source/system_configuration.cpp

@@ -29,7 +29,7 @@ uint64_t pageSize() noexcept
 {
     // sysconf fails when one provides an invalid name parameter. _SC_PAGESIZE
     // is a valid name parameter therefore it should never fail.
-    return static_cast<uint64_t>(IOX_POSIX_CALL(sysconf)(_SC_PAGESIZE)
+    return static_cast<uint64_t>(IOX_POSIX_CALL(iox_sysconf)(IOX_SC_PAGESIZE)
                                      .failureReturnValue(-1)
                                      .evaluate()
                                      .or_else([](auto& r) {

iceoryx_hoofs/test/moduletests/test_posix_filesystem_posix_acl.cpp

@@ -60,7 +60,7 @@ struct PwUidResult
     char buff[BUFFER_SIZE];
 };
 
-std::unique_ptr<PwUidResult> iox_getpwuid(const uid_t uid)
+std::unique_ptr<PwUidResult> iox_getpwuid(const iox_uid_t uid)
 {
     passwd* resultPtr = nullptr;
     std::unique_ptr<PwUidResult> result = std::make_unique<PwUidResult>();
@@ -126,7 +126,7 @@ TEST_F(PosixAcl_test, writeSpecialUserPermissions)
     // no name specified
     EXPECT_FALSE(entryAdded);
 
-    auto name = iox_getpwuid(geteuid());
+    auto name = iox_getpwuid(iox_geteuid());
     ASSERT_TRUE(name);
     PosixUser::userName_t currentUserName(iox::TruncateToCapacity, name->pwd.pw_name);
 
@@ -209,11 +209,11 @@ TEST_F(PosixAcl_test, writeSpecialPermissionsWithID)
 {
     ::testing::Test::RecordProperty("TEST_ID", "ef0c7e17-de0e-4cfb-aafa-3e68580660e5");
 
-    auto name = iox_getpwuid(geteuid());
+    auto name = iox_getpwuid(iox_geteuid());
     ASSERT_TRUE(name);
     std::string currentUserName(name->pwd.pw_name);
-    uid_t currentUserId(name->pwd.pw_uid);
-    gid_t groupId = 0; // root
+    iox_uid_t currentUserId(name->pwd.pw_uid);
+    iox_gid_t groupId = 0; // root
 
     bool entryAdded = m_accessController.addPermissionEntry(
         PosixAcl::Category::SPECIFIC_USER, PosixAcl::Permission::READWRITE, currentUserId);
@@ -254,7 +254,7 @@ TEST_F(PosixAcl_test, writeSpecialPermissionsWithID)
 TEST_F(PosixAcl_test, addNameInWrongPlace)
 {
     ::testing::Test::RecordProperty("TEST_ID", "2d2dbb0d-1fb6-4569-8651-d341a4525ea6");
-    auto name = iox_getpwuid(geteuid());
+    auto name = iox_getpwuid(iox_geteuid());
     ASSERT_TRUE(name);
 
     m_accessController.addPermissionEntry(PosixAcl::Category::GROUP, PosixAcl::Permission::READ);

iceoryx_platform/freertos/include/iceoryx_platform/grp.hpp

@@ -19,11 +19,13 @@
 #ifndef IOX_HOOFS_FREERTOS_PLATFORM_GRP_HPP
 #define IOX_HOOFS_FREERTOS_PLATFORM_GRP_HPP
 
+#include "iceoryx_platform/types.hpp"
+
 struct group
 {
     const char* gr_name;
     const char* gr_passwd;
-    gid_t gr_gid;
+    iox_gid_t gr_gid;
     const char** gr_mem;
 };
 
@@ -39,7 +41,7 @@ inline struct group* getgrnam(const char*)
     return &dummy;
 }
 
-inline struct group* getgrgid(gid_t)
+inline struct group* getgrgid(iox_gid_t)
 {
     static const char* groupName = "iceoryx_freertos_group";
     static const char* groupPasswd = "iceoryx_freertos_passwd";
@@ -51,7 +53,7 @@ inline struct group* getgrgid(gid_t)
     return &dummy;
 }
 
-inline int iox_getgrouplist(const char*, gid_t, gid_t* groups, int* ngroups)
+inline int iox_getgrouplist(const char*, iox_gid_t, iox_gid_t* groups, int* ngroups)
 {
     groups[0] = 0;
     *ngroups = 1;

iceoryx_platform/freertos/include/iceoryx_platform/types.hpp

@@ -19,4 +19,9 @@
 #ifndef IOX_HOOFS_FREERTOS_PLATFORM_TYPES_HPP
 #define IOX_HOOFS_FREERTOS_PLATFORM_TYPES_HPP
 
+#include <sys/types.h>
+
+using iox_gid_t = gid_t;
+using iox_uid_t = uid_t;
+
 #endif // IOX_HOOFS_FREERTOS_PLATFORM_TYPES_HPP