Refactor WriteAddressValue to simplify type handling

The code changes in the `WriteAddressValue` method within the `MemHackLib.PlatformImplementations` namespace in `MemHackLin.cs` remove the conversion of the `value` parameter to a byte array. Instead, the updated implementation directly assigns the `value` to the `data` variable based on its type. Specifically, it checks if `value` is of type `int`, `long`, `short`, or `byte` and assigns `data` as a new `nint` with the corresponding value. If `value` is of an unsupported type, it returns an error message "Unsupported value type." This refactor simplifies the process and eliminates the need for byte array conversion and pinning.

Author: vycdev

MemHackLib/PlatformImplementations/MemHackLin.cs

@@ -398,8 +398,6 @@ public List<nint> MemorySearch(uint processId, long desiredValue)
         // Write value to process memory
         public string WriteAddressValue(uint processId, nint targetPointer, long value)
         {
-            byte[] newValueBuffer = BitConverter.GetBytes((int)value);
-
             // Open the process memory using ptrace
             nint handle = OpenProcess((int)processId);
 
@@ -408,7 +406,19 @@ public string WriteAddressValue(uint processId, nint targetPointer, long value)
 
             // Use ptrace PTRACE_POKEDATA to write the memory value
             nint addr = new(targetPointer);
-            nint data = Marshal.UnsafeAddrOfPinnedArrayElement(newValueBuffer, 0);
+            nint data;
+
+            if(ValueType == typeof(int))
+                data = new((int)value);
+            else if (ValueType == typeof(long))
+                data = new((long)value);
+            else if (ValueType == typeof(short))
+                data = new((short)value);
+            else if (ValueType == typeof(byte))
+                data = new((byte)value);
+            else
+                return "Unsupported value type.";
+
             int result = ptrace(PTRACE_POKEDATA, (int)processId, addr, data);
 
             if (result == -1)