diff options
| author | Amlal El Mahrouss <amlal@nekernel.org> | 2026-03-24 12:25:55 +0100 |
|---|---|---|
| committer | Amlal El Mahrouss <amlal@nekernel.org> | 2026-03-24 12:25:55 +0100 |
| commit | 0c43e344918569474e1460876105c9f1eb43efa8 (patch) | |
| tree | 4ea54404a0967e48147262a392fa52c2749bddc9 /src | |
| parent | 5b5081e679a0481356d1e026310546ac632b1c87 (diff) | |
[FEAT] Kernel: UTF-8 usage in UserMgr.
Signed-off-by: Amlal El Mahrouss <amlal@nekernel.org>
Diffstat (limited to 'src')
| -rw-r--r-- | src/kernel/KernelKit/Semaphore.h | 7 | ||||
| -rw-r--r-- | src/kernel/KernelKit/User.h | 8 | ||||
| -rw-r--r-- | src/kernel/src/Property.cpp | 2 | ||||
| -rw-r--r-- | src/kernel/src/User.cpp | 13 |
4 files changed, 19 insertions, 11 deletions
diff --git a/src/kernel/KernelKit/Semaphore.h b/src/kernel/KernelKit/Semaphore.h index bb6b9ba9..4dc9a454 100644 --- a/src/kernel/KernelKit/Semaphore.h +++ b/src/kernel/KernelKit/Semaphore.h @@ -29,6 +29,7 @@ using SemaphoreArr = UInt64[kSemaphoreCount]; /// @brief Checks if the semaphore is valid. inline bool rtl_sem_is_valid(const SemaphoreArr& sem, const UInt64& owner = 0) { + if (!sem) return false; return sem[kSemaphoreOwnerIndex] == owner && sem[kSemaphoreCountIndex] > 0; } @@ -36,6 +37,8 @@ inline bool rtl_sem_is_valid(const SemaphoreArr& sem, const UInt64& owner = 0) { /// @param sem /// @return inline bool rtl_sem_release(SemaphoreArr& sem) { + if (!sem) return false; + sem[kSemaphoreOwnerIndex] = 0; sem[kSemaphoreCountIndex] = 0; @@ -47,6 +50,8 @@ inline bool rtl_sem_release(SemaphoreArr& sem) { /// @param owner the owner to set, could be anything identifitable. /// @return inline bool rtl_sem_acquire(SemaphoreArr& sem, const UInt64& owner) { + if (!sem) return false; + if (!owner) { err_global_get() = kErrorInvalidData; return false; // Invalid owner, return false and set KPC. @@ -65,6 +70,8 @@ inline bool rtl_sem_acquire(SemaphoreArr& sem, const UInt64& owner) { /// @return inline bool rtl_sem_wait(SemaphoreArr& sem, const UInt64& owner, const UInt64& timeout, bool& condition) { + if (!sem) return false; + if (!rtl_sem_is_valid(sem, owner)) { return false; } diff --git a/src/kernel/KernelKit/User.h b/src/kernel/KernelKit/User.h index bae51180..6711bce7 100644 --- a/src/kernel/KernelKit/User.h +++ b/src/kernel/KernelKit/User.h @@ -42,16 +42,16 @@ enum struct UserRingKind : Int32 { kRingCount = 3, }; -using UserPublicKey = Char*; -using UserPublicKeyType = Char; +using UserPublicKey = Char8*; +using UserPublicKeyType = Char8; /// @brief System User class. class User final { public: User() = delete; - User(const Int32& sel, const Char* username); - User(const UserRingKind& kind, const Char* username); + User(const Int32& sel, const UserPublicKeyType* username); + User(const UserRingKind& kind, const UserPublicKeyType* username); ~User(); diff --git a/src/kernel/src/Property.cpp b/src/kernel/src/Property.cpp index 6db475e8..6da6ae70 100644 --- a/src/kernel/src/Property.cpp +++ b/src/kernel/src/Property.cpp @@ -6,6 +6,7 @@ #include <CFKit/Property.h> namespace Kernel::CF { + /***********************************************************************************/ /// @brief Destructor. /***********************************************************************************/ @@ -37,4 +38,5 @@ KBasicString<>& Property::GetKey() { PropertyId& Property::GetValue() { return fValue; } + } // namespace Kernel::CF diff --git a/src/kernel/src/User.cpp b/src/kernel/src/User.cpp index 3f5bb66a..daaf27a4 100644 --- a/src/kernel/src/User.cpp +++ b/src/kernel/src/User.cpp @@ -23,7 +23,7 @@ namespace Detail { /// \param password password to hash. /// \return the hashed password //////////////////////////////////////////////////////////// - STATIC UInt64 user_fnv_generator(const Char* password, User* user) { + STATIC UInt64 user_fnv_generator(const UserPublicKeyType* password, User* user) { kout << "user_fnv_generator: Try hashing user password...\r"; if (!password || !user) return 0; @@ -48,18 +48,17 @@ namespace Detail { //////////////////////////////////////////////////////////// /// @brief User ring constructor. //////////////////////////////////////////////////////////// -User::User(const Int32& sel, const Char* user_name) : mUserRing((UserRingKind) sel) { +User::User(const Int32& sel, const UserPublicKeyType* user_name) : mUserRing((UserRingKind) sel) { MUST_PASS(sel >= 0); - rt_copy_memory_safe((VoidPtr) user_name, this->mUserName, rt_string_len(user_name), - kMaxUserNameLen); + urt_copy_memory((VoidPtr) user_name, this->mUserName, urt_string_len(user_name)); } //////////////////////////////////////////////////////////// /// @brief User ring constructor. //////////////////////////////////////////////////////////// -User::User(const UserRingKind& ring_kind, const Char* user_name) : mUserRing(ring_kind) { - rt_copy_memory_safe((VoidPtr) user_name, this->mUserName, rt_string_len(user_name), - kMaxUserNameLen); +User::User(const UserRingKind& ring_kind, const UserPublicKeyType* user_name) + : mUserRing(ring_kind) { + urt_copy_memory((VoidPtr) user_name, this->mUserName, urt_string_len(user_name)); } //////////////////////////////////////////////////////////// |
