On 1/28/26 5:53 AM, Heinrich Schuchardt wrote:

Hello Heinrich,

static int do_zip(struct cmd_tbl *cmdtp, int flag, int argc, char *const argv[])
@@ -27,8 +28,10 @@ static int do_zip(struct cmd_tbl *cmdtp, int flag, int argc, 
char *const argv[])
                        return cmd_usage(cmdtp);
        }

-       if (gzip((void *) dst, &dst_len, (void *) src, src_len) != 0)
+       if (gzip(map_sysmem(dst, dst_len), &dst_len,
+                map_sysmem(src, src_len), src_len) != 0) {

Wasn't the idea of map_sysmem() with a non-zero length to have a matching 
unmap_sysmem()?

@Simon:

Does the sandbox really need the mappped areas or can we get rid of the length 
parameter and of unmap_sysmem()?

Best regards
Let me add the unmap_sysmem(), also for do_unzip() where it is missing too.

Reply via email to