Fix a bug with __getitem__ for pointer types

Author: mrexodia

src/dumpulator/ntprimitives.py

@@ -71,7 +71,12 @@ def __getitem__(self, index):
             return self.arch.read_ptr(self.ptr + index * self.arch.ptr_size())
         else:
             assert index == 0  # TODO: sizeof() not yet implemented
-            return self.type(self)
+            sizeof = self.arch.ptr_size()
+            ptr = self.ptr + index * sizeof
+            if issubclass(self.type, PVOID):
+                return self.type(self.arch.read_ptr(ptr), self.arch)
+            else:
+                return self.type(PVOID(ptr, self.arch))
 
     def __int__(self):
         return self.ptr
@@ -98,7 +103,7 @@ def read_ptr(self):
 
     def deref(self):
         assert self.type is not None
-        return self.type(self)
+        return self[0]
 
 def P(t):
     class P(PVOID):

src/dumpulator/ntsyscalls.py

@@ -2446,8 +2446,8 @@ def ZwProtectVirtualMemory(dp: Dumpulator,
                            NewProtect: ULONG,
                            OldProtect: P(ULONG)
                            ):
-    base = BaseAddress[0] & 0xFFFFFFFFFFFFF000
-    size = round_to_pages(RegionSize[0])
+    base = BaseAddress.read_ptr() & 0xFFFFFFFFFFFFF000
+    size = round_to_pages(RegionSize.read_ptr())
 
     print(f"protect {base:x}[{size:x}] = {NewProtect:x}")
     dp.protect(base, size, NewProtect)