From 826c80c6fcce706733580096037d5dfd3c2c03c6 Mon Sep 17 00:00:00 2001
From: Ali Norouzi <ali.norouzi@keysight.com>
Date: Fri, 31 Jul 2026 11:47:39 +0200
Subject: [PATCH] tests: fix a bug in autorun.py that hung tests + Emulate
 interact test to work without IPython installed

AI-Assisted: yes (Claude Opus 4.7)
---
 scapy/autorun.py    | 12 +++++++++---
 scapy/main.py       |  1 +
 test/regression.uts | 37 +++++++++++++++++++++++++++++++++++++
 3 files changed, 47 insertions(+), 3 deletions(-)

diff --git a/scapy/autorun.py b/scapy/autorun.py
index 8908cb9cc9a..74ce94992cf 100644
--- a/scapy/autorun.py
+++ b/scapy/autorun.py
@@ -128,18 +128,24 @@ def autorun_commands_timeout(cmds, timeout=None, **kwargs):
     if timeout is None:
         return autorun_commands(cmds, **kwargs)
 
-    q = Queue()  # type: Queue[Any]
+    q = Queue()  # type: Queue[Tuple[bool, Any]]
 
     def _runner():
         # type: () -> None
-        q.put(autorun_commands(cmds, **kwargs))
+        try:
+            q.put((False, autorun_commands(cmds, **kwargs)))
+        except BaseException as e:
+            q.put((True, e))
     th = threading.Thread(target=_runner)
     th.daemon = True
     th.start()
     th.join(timeout)
     if th.is_alive():
         raise StopAutorunTimeout
-    return q.get()
+    is_exc, result = q.get()
+    if is_exc:
+        raise result
+    return result
 
 
 class StringWriter(StringIO):
diff --git a/scapy/main.py b/scapy/main.py
index 990f27f4ba3..924e6dc65cb 100644
--- a/scapy/main.py
+++ b/scapy/main.py
@@ -828,6 +828,7 @@ def ptpython_configure(repl):
         # repl.use_ui_colorscheme("scapy")
 
     # Extend banner text
+    banner = banner_text
     if conf.interactive_shell in ["ipython", "ptipython"]:
         import IPython
         if conf.interactive_shell == "ptipython":
diff --git a/test/regression.uts b/test/regression.uts
index 8a5dd3e460f..c0db14b6644 100644
--- a/test/regression.uts
+++ b/test/regression.uts
@@ -1133,6 +1133,43 @@ ret = autorun_get_text_interactive_session(cmds)
 ret
 assert "Daft Punk" in ret[0]
 
+= Test autorun timeout path surfaces exceptions from failing tests
+~ autorun
+
+# Regression for two bugs whose interaction hung UTScapy forever on:
+#   1. CPython gh-130250: traceback.print_last(file=...) passed `file`
+#      positionally into print_exception, binding it to `tb` and
+#      crashing with AttributeError on tb.tb_frame.
+#   2. scapy autorun_commands_timeout._runner swallowed exceptions from
+#      autorun_commands, leaving Queue.get() to block forever.
+
+out, res = autorun_get_text_interactive_session(
+    "assert False, 'the dead flag blues'\n", timeout=10,
+)
+assert res is False, "expected False from failing test, got %r" % (res,)
+assert "AssertionError" in out, "expected AssertionError in output, got:\n" + out
+assert "the dead flag blues" in out, "expected assertion message in output, got:\n" + out
+assert "tb_frame" not in out, "CPython gh-130250 leak: " + out
+
+= Test autorun timeout path propagates unexpected runner exceptions
+
+import scapy.autorun as _ar
+_orig = _ar.autorun_commands
+def _boom(*a, **kw):
+    raise RuntimeError("simulated internal failure")
+
+_ar.autorun_commands = _boom
+try:
+    raised = None
+    try:
+        _ = autorun_get_text_interactive_session("1+1\n", timeout=5)
+    except RuntimeError as e:
+        raised = e
+    assert raised is not None, "expected RuntimeError to propagate, got none"
+    assert "simulated internal failure" in str(raised)
+finally:
+    _ar.autorun_commands = _orig
+
 = Test utility TEX functions
 
 assert tex_escape("{scapy}\\^$~#_&%|><") == "{\\tt\\char123}scapy{\\tt\\char125}{\\tt\\char92}\\^{}\\${\\tt\\char126}\\#\\_\\&\\%{\\tt\\char124}{\\tt\\char62}{\\tt\\char60}"