minor changes

Author: DEntis-T

src/kernel/drivers/vmfs.cpp

@@ -51,10 +51,6 @@ namespace newasm::Drivers::FileSystem_V
     }
     FORCE_INLINE inline void SaveFileTable(DISK& disk, const FILE_TABLE& table)
     {
-        auto t = table;
-        std::sort(t.begin(), t.end(), [](const auto& a, const auto& b) -> bool {
-            return a.pos < b.pos;
-        });
         std::string buf(sizeof(FILE) * NewASM::Drivers::FileSystem_V::GetMaxFiles(), '\0');
         std::memcpy(&buf[0], t.data(), buf.size());
         disk.WRITE_DSK(0, buf);
@@ -97,13 +93,13 @@ namespace newasm::Drivers::FileSystem_V
         {
             std::vector<std::pair<DISK_POS, DISK_POS>> pairs;
             auto table = GetFileTable(disk);
-            std::sort(table.begin(), table.end(), [](const auto& a, const auto& b) -> bool {
-                return a.pos < b.pos;
-            });
             for(int i = 0; i < table.size(); ++i)
             {
                 pairs.push_back({table[i].pos, table[i].pos + table[i].size});
             }
+            std::sort(pairs.begin(), pairs.end(), [](const auto& a, const auto& b) -> bool {
+                return a.first < b.first;
+            });
             if(pairs.size() == 1)
             {
                 return pairs.front().second + 1;
@@ -152,6 +148,10 @@ namespace newasm::Drivers::FileSystem_V
     inline void MKFILE(DISK& disk, FILE_NAME name, const std::string& content)
     {
         using namespace NewASM::Drivers::FileSystem_V;
+        if(name.size() >= __newasm_MAX_FILENAME_LEN)
+        {
+            return;
+        }
         FILE_TABLE table = GetFileTable(disk);
         for(int i = 0; i < table.size(); ++i)
         {

src/kernel/syscall_handle.cpp

@@ -1374,6 +1374,7 @@ namespace newasm
                     if(!NewASM::header::functions::isalphanum(TLR))
                     {
                         NewASM::header::functions::err("Permission denied. File names have to be alphanumeric!");
+                        NewASM::terminate(NewASM::exit_codes::kernel_panic);
                         return 1;
                     }
                     if(newasm::ctl::data::path.size() > 1)
@@ -1398,6 +1399,13 @@ namespace newasm
                     if(EXISTS(NewASM::hardware::Disk, TLR))
                     {
                         NewASM::header::functions::err("File already exists.");
+                        NewASM::terminate(NewASM::exit_codes::kernel_panic);
+                        return 1;
+                    }
+                    if(TLR.size() >= __newasm_MAX_FILENAME_LEN)
+                    {
+                        NewASM::header::functions::err("Permission denied. File name exceeds a limit of 128 characters!");
+                        NewASM::terminate(NewASM::exit_codes::kernel_panic);
                         return 1;
                     }
                     MKFILE(NewASM::hardware::Disk, TLR, STL);
@@ -1436,6 +1444,7 @@ namespace newasm
                     if(!EXISTS(NewASM::hardware::Disk, TLR))
                     {
                         NewASM::header::functions::err("File does not exist.");
+                        NewASM::terminate(NewASM::exit_codes::kernel_panic);
                         return 1;
                     }
                     RMFILE(NewASM::hardware::Disk, TLR);
@@ -1492,6 +1501,7 @@ namespace newasm
                     if(!NewASM::header::functions::isalphanum(TLR))
                     {
                         NewASM::header::functions::err("Permission denied. File names have to be alphanumeric!");
+                        NewASM::terminate(NewASM::exit_codes::kernel_panic);
                         return 1;
                     }
 

src/newasm_setup.cpp

@@ -83,6 +83,7 @@ namespace newasm
         const int invalid_init = 55;
         const int linker_err = 56;
         const int jit_fail = 57;
+        const int kernel_panic = 58;
 
         const std::unordered_map<int, std::string> identifier = {
             {noterm_point, "NoTerminationPoint"},
@@ -142,7 +143,8 @@ namespace newasm
             {invalid_alloc, "InvalidAlloc"},
             {invalid_init, "InvalidVarInitializer"},
             {linker_err, "LinkerError"},
-            {jit_fail, "JITCompilerFailure"}
+            {jit_fail, "JITCompilerFailure"},
+            {kernel_panic, "KernelPanicOrSysCrash"}
         };
     }
     namespace cmp_results