Move root property from core to OS

Author: elicn

examples/netgear_6220_mips32el_linux.py

@@ -58,7 +58,7 @@ def my_bind(ql: Qiling):
 
 def my_netgear(path, rootfs):
     ql = Qiling(path, rootfs, verbose=QL_VERBOSE.DEBUG, profile="netgear_6220.ql", multithread=False)
-    ql.root = False
+    ql.os.root = False
 
     ql.add_fs_mapper('/proc', '/proc')
     ql.set_syscall(4004, my_syscall_write)

qiling/core.py

@@ -95,7 +95,6 @@ def __init__(
         self._patch_lib = []
         self._debug_stop = False
         self._debugger = None
-        self._root = False
 
         ###############################
         # Properties configured later #
@@ -574,19 +573,6 @@ def debugger(self) -> Union[str, bool]:
     def debugger(self, dbger):
         self._debugger = dbger
 
-    @property
-    def root(self) -> bool:
-        """ Whether run current program as root?
-
-            Type: bool
-            Examples: ql.root = True
-        """
-        return self._root
-
-    @root.setter
-    def root(self, root):
-        self._root = root
-
     @property
     def filter(self) -> str:
         """ Filter logs with regex.

qiling/os/os.py

@@ -123,6 +123,19 @@ def stdout(self, stream: TextIO) -> None:
     def stderr(self, stream: TextIO) -> None:
         self._stderr = stream
 
+    @property
+    def root(self) -> bool:
+        """An indication whether the process is running as root.
+        """
+
+        # for this to work the os derivative should override this property
+        # and implement the os logic. in case it is not, return False
+        return False
+
+    @root.setter
+    def root(self, enabled: bool) -> None:
+        raise NotImplementedError('Running as root is not implemented for this OS')
+
     def resolve_fcall_params(self, params: Mapping[str, Any]) -> Mapping[str, Any]:
         """Transform function call raw parameters values into meaningful ones, according to
         their assigned type.

qiling/os/posix/syscall/socket.py

@@ -301,7 +301,7 @@ def ql_syscall_bind(ql: Qiling, bind_fd, bind_addr, bind_addrlen):
     port, host = struct.unpack(">HI", data[2:8])
     host = ql_bin_to_ip(host)
 
-    if ql.root == False and port <= 1024:
+    if not ql.os.root and port <= 1024:
         port = port + 8000
 
     if sin_family == 1: