summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorAmlal El Mahrouss <amlal@nekernel.org>2026-03-24 12:25:55 +0100
committerAmlal El Mahrouss <amlal@nekernel.org>2026-03-24 12:25:55 +0100
commit0c43e344918569474e1460876105c9f1eb43efa8 (patch)
tree4ea54404a0967e48147262a392fa52c2749bddc9
parent5b5081e679a0481356d1e026310546ac632b1c87 (diff)
[FEAT] Kernel: UTF-8 usage in UserMgr.
Signed-off-by: Amlal El Mahrouss <amlal@nekernel.org>
-rw-r--r--src/kernel/KernelKit/Semaphore.h7
-rw-r--r--src/kernel/KernelKit/User.h8
-rw-r--r--src/kernel/src/Property.cpp2
-rw-r--r--src/kernel/src/User.cpp13
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));
}
////////////////////////////////////////////////////////////