fix(yaffs): decided to only set timestamps, not ownership/permissions.

qkaiser · qkaiser · commit 1176c016073a · 2023-04-17T18:24:09.000+02:00
Setting permissions and ownerships leads to issues down the road when
trying to create files in directories we're not allowed to open.
diff --git a/unblob/handlers/filesystem/yaffs.py b/unblob/handlers/filesystem/yaffs.py
@@ -551,9 +551,7 @@ def extract_entry(self, entry: YAFFSEntry, outdir: Path):  # noqa: C901
                 )
                 return
             src_path.link_to(out_path)
-        # out_path.chmod(entry.st_mode)
-        # os.utime(out_path.as_posix(),(entry.st_atime,entry.st_mtime))
-        # os.chown(out_path.as_posix(), entry.st_uid, entry.st_gid)
+        os.utime(out_path.as_posix(), (entry.st_atime, entry.st_mtime))
 
 
 class YAFFS2Parser(YAFFSParser):

Original file line number	Diff line number	Diff line change
`@@ -551,9 +551,7 @@ def extract_entry(self, entry: YAFFSEntry, outdir: Path): # noqa: C901`
`551`	`551`	`)`
`552`	`552`	`return`
`553`	`553`	`src_path.link_to(out_path)`
`554`		`- # out_path.chmod(entry.st_mode)`
`555`		`- # os.utime(out_path.as_posix(),(entry.st_atime,entry.st_mtime))`
`556`		`- # os.chown(out_path.as_posix(), entry.st_uid, entry.st_gid)`
	`554`	`+ os.utime(out_path.as_posix(), (entry.st_atime, entry.st_mtime))`
`557`	`555`
`558`	`556`
`559`	`557`	`class YAFFS2Parser(YAFFSParser):`