From 1b5fac31896338aac4b25ea84a70e87a6fee3891 Mon Sep 17 00:00:00 2001
From: Kovid Goyal <kovid@kovidgoyal.net>
Date: Fri, 6 Jan 2023 14:25:21 +0530
Subject: [PATCH] Cleanup previous PR

---
 kittens/diff/main.py | 35 ++++++++---------------------------
 kitty/utils.py       | 19 +++++++++++++++++++
 2 files changed, 27 insertions(+), 27 deletions(-)

diff --git a/kittens/diff/main.py b/kittens/diff/main.py
index c24ee7263..879833133 100644
--- a/kittens/diff/main.py
+++ b/kittens/diff/main.py
@@ -14,16 +14,16 @@ from enum import Enum, auto
 from functools import partial
 from gettext import gettext as _
 from typing import (
-    Any, DefaultDict, Dict, Iterable, Iterator, List, Optional, Tuple, Union
+    Any, DefaultDict, Dict, Iterable, Iterator, List, Optional, Tuple, Union,
 )
 
-from kitty.cli import CONFIG_HELP, parse_args, CompletionSpec
+from kitty.cli import CONFIG_HELP, CompletionSpec, parse_args
 from kitty.cli_stub import DiffCLIOptions
 from kitty.conf.utils import KeyAction
 from kitty.constants import appname
 from kitty.fast_data_types import wcswidth
 from kitty.key_encoding import EventType, KeyEvent
-from kitty.utils import ScreenSize
+from kitty.utils import ScreenSize, extract_all_from_tarfile_safely
 
 from ..tui.handler import Handler
 from ..tui.images import ImageManager, Placement
@@ -32,21 +32,21 @@ from ..tui.loop import Loop
 from ..tui.operations import styled
 from . import global_data
 from .collect import (
-    Collection, add_remote_dir, create_collection, data_for_path,
-    lines_for_path, sanitize, set_highlight_data
+    Collection, add_remote_dir, create_collection, data_for_path, lines_for_path,
+    sanitize, set_highlight_data,
 )
 from .config import init_config
 from .options.types import Options as DiffOptions
 from .patch import Differ, Patch, set_diff_command, worker_processes
 from .render import (
-    ImagePlacement, ImageSupportWarning, Line, LineRef, Reference, render_diff
+    ImagePlacement, ImageSupportWarning, Line, LineRef, Reference, render_diff,
 )
 from .search import BadRegex, Search
 
 try:
     from .highlight import (
         DiffHighlight, get_highlight_processes, highlight_collection,
-        initialize_highlighter
+        initialize_highlighter,
     )
     has_highlighter = True
     DiffHighlight
@@ -630,26 +630,7 @@ def get_ssh_file(hostname: str, rpath: str) -> str:
         raise SystemExit(p.returncode)
     with tarfile.open(fileobj=io.BytesIO(raw), mode='r:') as tf:
         members = tf.getmembers()
-        def is_within_directory(directory, target):
-            
-            abs_directory = os.path.abspath(directory)
-            abs_target = os.path.abspath(target)
-        
-            prefix = os.path.commonprefix([abs_directory, abs_target])
-            
-            return prefix == abs_directory
-        
-        def safe_extract(tar, path=".", members=None, *, numeric_owner=False):
-        
-            for member in tar.getmembers():
-                member_path = os.path.join(path, member.name)
-                if not is_within_directory(path, member_path):
-                    raise Exception("Attempted Path Traversal in Tar File")
-        
-            tar.extractall(path, members, numeric_owner) 
-            
-        
-        safe_extract(tf, tdir)
+        extract_all_from_tarfile_safely(tf, tdir)
         if len(members) == 1:
             for root, dirs, files in os.walk(tdir):
                 if files:
diff --git a/kitty/utils.py b/kitty/utils.py
index dc568be54..4a9044f55 100644
--- a/kitty/utils.py
+++ b/kitty/utils.py
@@ -29,6 +29,7 @@ from .typing import AddressFamily, PopenType, Socket, StartupCtx
 if TYPE_CHECKING:
     from .fast_data_types import OSWindowSize
     from .options.types import Options
+    import tarfile
 else:
     Options = object
 
@@ -1103,3 +1104,21 @@ def sanitize_url_for_dispay_to_user(url: str) -> str:
     except Exception:
         url = 'Unpareseable URL: ' + url
     return url
+
+
+def extract_all_from_tarfile_safely(tf: 'tarfile.TarFile', dest: str) -> None:
+
+    def is_within_directory(directory: str, target: str) -> bool:
+        abs_directory = os.path.abspath(directory)
+        abs_target = os.path.abspath(target)
+        prefix = os.path.commonprefix((abs_directory, abs_target))
+        return prefix == abs_directory
+
+    def safe_extract(tar: 'tarfile.TarFile', path: str = ".", numeric_owner: bool = False) -> None:
+        for member in tar.getmembers():
+            member_path = os.path.join(path, member.name)
+            if not is_within_directory(path, member_path):
+                raise ValueError(f'Attempted path traversal in tar file: {member.name}')
+        tar.extractall(path, tar.getmembers(), numeric_owner=numeric_owner)
+
+    safe_extract(tf, dest)