Public scan — anyone with this URL can view this analysis. Sign up to track your own repos privately, run scheduled re-scans, and get AI fix prompts via your dashboard.
Sign up free Scan another repo
174 of your 254 findings came from Repobility's proprietary detections. ✓ Repobility tags below mark them.

Scan timing: clone 3.4s · analysis 15.36s · 5.4 MB · GitHub API rate-limit (preflight)

commaai/openpilot

https://github.com/commaai/openpilot · scanned 2026-06-05 09:30 UTC (5 days, 16 hours ago) · 10 languages

531 raw signals (231 security + 300 graph) 57th percentile · Python · medium (20-100K LoC) System graph score 77 (lower by 9)

File as issue Image
UNIFIED Repobility · multi-layer engine · AI coders

Complete repo analysis

Last scanned 5 days, 16 hours ago · v2 · 210 actionable findings from 2 signal sources. 169 repeated signals grouped for readability. Security checks, system graph analysis, and verified AI-agent feedback are merged into one review queue.

JSON
Combined
67.4
/100
Security checks
58
97 findings
System graph
77
100.0% cov · 113 findings
Critical
5
click to filter
Agents
6
0 votes
Crowd
0
reported
Score breakdown â 2026-05-18-v5
Component Sub-score Weight Contribution
structure_score 60.0 0.15 9.00
security_score 45.7 0.25 11.43
testing_score 98.0 0.20 19.60
documentation_score 63.0 0.15 9.45
practices_score 84.0 0.15 12.60
code_quality 55.9 0.10 5.59
Overall 1.00 67.7
Severity distribution — click a segment to filter
Active filters: excluding tests × Reset all
Severity: Critical 5 High 16 Medium 31 Low 91 9-Layer: Software Security Quality Integrity Frontend Hardware Data Network Cicd Source: Security checks 97 System graph 113 Crowd 0 Layer: Quality 113 Security 20 Software 66 Cicd 8 Api 1 Frontend 2
Scan summary Quality grade B- (68/100). Dimensions: security 46, maintainability 60. 231 findings (48 security). 78,768 lines analyzed.

Showing 142 of 210 actionable findings. 379 raw detector signals were grouped into reader-sized issues. Click TP / FP to vote on a finding's accuracy — votes adjust the confidence weighting and improve detection across the platform.

critical Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED030] Python Pickle Loads: pickle.loads() can execute arbitrary code via __reduce__.
Review and fix per the pattern semantics. See CWE-502 / for context.
3 files, 3 locations
selfdrive/debug/print_docs_diff.py:22
selfdrive/modeld/dmonitoringmodeld.py:33
selfdrive/modeld/get_model_metadata.py:44
critical Security checks quality Quality conf 1.00 ✓ Repobility [MINED102] Shell Injection Via F-string: Shell command built via f-string or .format with non-constant input — command injection. An attacker controlling any interpolated value can execute arbitrary shell commands.
Use the list form of subprocess (e.g. subprocess.run(["cmd", arg1, arg2])) with shell=False. Never combine shell=True with string interpolation.
system/timed.py:22
critical Security checks quality Quality conf 1.00 3 occurrences [SEC081] Python: pickle.loads / marshal.loads on untrusted data: pickle.load(s) and marshal.load(s) execute arbitrary code on untrusted input. Ported from dlint DUO103 / DUO120 (BSD-3).
Use json, msgpack, or protobuf for untrusted data. If pickle is required, sign the payload with HMAC.
3 files, 3 locations
selfdrive/debug/print_docs_diff.py:22
selfdrive/modeld/dmonitoringmodeld.py:33
selfdrive/modeld/get_model_metadata.py:44
critical Security checks security secrets conf 0.95 Identified a Private Key, which may compromise cryptographic security and sensitive data encryption.
Gitleaks detected a committed secret or credential pattern.
system/hardware/tici/id_rsa:1
high Security checks quality Quality conf 1.00 ✓ Repobility 6 occurrences Missing import: `struct` used but not imported
The file uses `struct.something(...)` but never imports `struct`. This raises NameError at runtime the first time the line executes.
6 files, 6 locations
cereal/messaging/__init__.py:38
selfdrive/debug/print_flags.py:7
system/hardware/tici/lpa.py:418
system/loggerd/uploader.py:190
system/proclogd.py:87
tools/cabana/dbc/generate_dbc_json.py:14
low Security checks cicd CI/CD security conf 0.35 ✓ Repobility Workflow references repository secrets in a pull_request workflow
Fork pull_request runs do not receive normal repository secrets on GitHub Actions. Review this as a reliability/intent signal, not as direct fork-secret exfiltration. Raise severity only for pull_request_target or another trusted-context path that runs untrusted PR code with secrets.
.github/workflows/docs.yaml:41 CI/CD securityworkflow secretsGitHub Actions
critical System graph security Secrets conf 1.00 Possible secret in system/ui/lib/wifi_manager.py
Detected pattern matching password_literal. Rotate the credential and move to a secret manager.
system/ui/lib/wifi_manager.py:35
low Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED006] Overcatch Baseexception: except BaseException: ... — prevents Ctrl+C and SystemExit from working.
Review and fix per the pattern semantics. See CWE-705 / for context.
3 files, 3 locations
selfdrive/debug/can_print_changes.py:63
selfdrive/debug/check_timings.py:28
selfdrive/debug/uiview.py:33
high Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED017] C System Call: system() invokes shell. command injection if any arg is dynamic.
Review and fix per the pattern semantics. See CWE-78 / for context.
3 files, 3 locations
common/util.h:100
selfdrive/ui/installer/installer.cc:79
tools/jotpluggler/util.cc:30
high Security checks quality Quality conf 1.00 ✓ Repobility [MINED021] Path Traversal Os Join: os.path.join(user_dir, filename) where filename can contain "../" — directory escape.
Review and fix per the pattern semantics. See CWE-22 / A01:2021 for context.
selfdrive/modeld/get_model_metadata.py:51
high Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED034] Python Subprocess Shell True: subprocess(..., shell=True) enables command injection.
Review and fix per the pattern semantics. See CWE-78 / for context.
3 files, 3 locations
system/hardware/tici/agnos.py:267
system/timed.py:22
system/tombstoned.py:40
high Security checks quality Quality conf 1.00 ✓ Repobility 3 occurrences [MINED036] Python Os System Call: os.system() invokes shell with no escaping.
Review and fix per the pattern semantics. See CWE-78 / for context.
3 files, 3 locations
common/utils.py:40
selfdrive/ui/layouts/settings/software.py:161
system/hardware/tici/agnos.py:281
high Security checks security path traversal conf 0.80 3 occurrences [SEC013] Path Traversal — User Input in File Path: User-controlled input used in file path without sanitization. Allows reading arbitrary files.
Use os.path.realpath() and verify the path starts with your expected base directory. Use secure_filename() for uploads.
3 files, 3 locations
selfdrive/car/card.py:122
selfdrive/modeld/helpers.py:11
selfdrive/ui/installer/installer.cc:216
high Security checks software Resource exhaustion conf 1.00 3 occurrences [SEC035] Unbounded Resource Allocation — DoS risk: Allocating resources (buffers, recursion stack, large ranges) based on user input without an upper bound. Attackers send `size=10000000` to exhaust memory, or trigger expensive computation. CWE-770/400. Examples: CVE-2023-44487 (HTTP/2 Rapid Reset), countless YAML/XML billion-laughs variants.
Cap user-controlled sizes BEFORE allocation: size = min(int(request.args.get('n', 100)), MAX_SIZE) Set framework-level limits: Flask: app.config['MAX_CONTENT_LENGTH'] = 10 * 1024 * 1024 FastAPI: use middleware to enforce request size Django: DATA_UPLOAD_MAX_MEMORY_SIZE in settings.py …
3 files, 3 locations
selfdrive/controls/plannerd.py:16
tools/joystick/joystickd.py:19
tools/lateral_maneuvers/lateral_maneuversd.py:93
high Security checks quality Quality conf 1.00 [SEC080] Python: tarfile.extractall without filter: tarfile.extract*() without filter='data' allows path-traversal (CVE-2007-4559, fixed via PEP 706 in 3.12). Ported from bandit B202 (Apache-2.0).
Add `filter='data'` (Python ≥ 3.12) or manually validate member paths against `os.path.abspath`.
system/updated/casync/tar.py:38
high Security checks quality Quality conf 1.00 ✓ Repobility 25 occurrences `self.close` used but never assigned in __init__
Method `__exit__` of class `SMBus` reads `self.close`, but no assignment to it exists in __init__ (and no class-level fallback). This raises AttributeError the first time the method runs against an instance.
7 files, 25 locations
common/i2c.py:44, 60, 62, 66, 69, 72, 78 (7 hits)
common/swaglog.py:37, 51, 56 (4 hits)
scripts/reporter.py:16, 19, 21 (4 hits)
common/text_window.py:35, 40, 43 (3 hits)
system/micd.py:102, 108, 111 (3 hits)
system/statsd.py:46, 55, 60 (3 hits)
system/version.py:82
high Security checks cicd CI/CD security conf 0.95 Docker final stage runs as root
The final runtime stage explicitly uses root. A compromised app process would have root inside the container.
Dockerfile.openpilot:37 CI/CD securitycontainers
high Security checks software dependencies conf 0.90 ✓ Repobility Dockerfile FROM `ubuntu:24.04` not pinned by digest
`FROM ubuntu:24.04` resolves the tag at build time. The registry CAN re-push a different image for the same tag, so every build is potentially different. Production images should pin to `image@sha256:...` for reproducibility + supply-chain integrity.
Dockerfile.openpilot:1
low Security checks cicd CI/CD security conf 0.90 ✓ Repobility 32 occurrences GitHub Action is tag-pinned rather than SHA-pinned
Action `actions/checkout` pinned to mutable ref `@v6` uses a mutable tag or branch. Pin external actions to a reviewed full commit SHA when the workflow is security-sensitive.
8 files, 32 locations
.github/workflows/tests.yaml:38, 68, 88, 105, 128, 145, 152, 160, +2 more (12 hits)
.github/workflows/jenkins-pr-trigger.yaml:14, 55, 75, 89 (8 hits)
.github/workflows/model_review.yaml:19, 23 (4 hits)
.github/workflows/docs.yaml:24, 37 (2 hits)
.github/workflows/stale.yaml:16, 37 (2 hits)
.github/workflows/ui_preview.yaml:65, 73 (2 hits)
.github/workflows/release.yaml:25
.github/workflows/repo-maintenance.yaml:17
CI/CD securitySupply chainGitHub Actions
medium Security checks cicd CI/CD security conf 0.90 ✓ Repobility 10 occurrences GitHub Action is tag-pinned rather than SHA-pinned
Action `lewagon/wait-on-check-action` pinned to mutable ref `@v1.3.4` uses a mutable tag or branch. Pin external actions to a reviewed full commit SHA when the workflow is security-sensitive.
3 files, 10 locations
.github/workflows/diff_report.yaml:20, 29, 40 (6 hits)
.github/workflows/ui_preview.yaml:42, 56, 167 (3 hits)
.github/workflows/docs.yaml:23
CI/CD securitySupply chainGitHub Actions
high System graph security security conf 1.00 Insecure pattern 'eval_used' in tools/jotpluggler/math_eval.py:44
Found a known-risky pattern (eval_used). Review and replace if possible.
tools/jotpluggler/math_eval.py:44 Eval used
high System graph security security conf 1.00 Insecure pattern 'exec_used' in tools/jotpluggler/math_eval.py:49
Found a known-risky pattern (exec_used). Review and replace if possible.
tools/jotpluggler/math_eval.py:49 Exec used
medium Security checks security auth conf 0.92 [AUC001] No Repobility access matrix policy found: The repository uses web/API frameworks but does not define .repobility/access.yml or equivalent authorization documentation.
The repository uses web/API frameworks but does not define .repobility/access.yml or equivalent authorization documentation.
low Security checks quality Error handling conf 1.00 3 occurrences [ERR001] Silent Exception Swallowing: Silently swallowing all exceptions hides bugs. Even in cleanup code, log at DEBUG level.
Log the error: `except Exception: logger.debug('cleanup failed', exc_info=True)`. Or handle specific exception types.
3 files, 3 locations
selfdrive/car/card.py:126
selfdrive/modeld/helpers.py:24
selfdrive/ui/mici/layouts/settings/firehose.py:65
low Security checks security Injection conf 0.50 3 occurrences [SEC005] Command Injection Risk: Unsafe shell execution or eval of user input.
Use subprocess with shell=False and a list of args. Never eval user input.
3 files, 3 locations
common/utils.py:40
system/ui/mici_reset.py:109
system/ui/tici_reset.py:48
low Security checks security Deserialization conf 1.00 3 occurrences [SEC007] Unsafe Deserialization: Unsafe deserialization can execute arbitrary code.
Use yaml.safe_load() instead of yaml.load(). Avoid pickle for untrusted data.
3 files, 3 locations
selfdrive/debug/print_docs_diff.py:22
selfdrive/modeld/dmonitoringmodeld.py:33
selfdrive/modeld/get_model_metadata.py:44
medium Security checks security path traversal conf 1.00 [SEC012] ZipSlip — Archive Path Traversal: Archive extraction without path validation allows writing files outside the target directory.
Validate extracted paths with os.path.realpath() and ensure they stay within the target directory.
tools/plotjuggler/juggle.py:62
medium Security checks security path traversal conf 1.00 [SEC012] ZipSlip — Archive Path Traversal: Archive extraction without path validation allows writing files outside the target directory.
Validate extracted paths with os.path.realpath() and ensure they stay within the target directory.
system/updated/casync/tar.py:38
low Security checks quality Quality conf 1.00 [SEC136] AI-typical over-broad exception handler swallowing all errors: Catch-all exception block that silently returns success or no-ops. AI agents reach for this pattern when a flaky test or an unfamiliar API throws — wrap, swallow, return success. Real bugs are masked, observability is destroyed, and callers think the operation worked. CWE-396 (improperly-generalized exception). Distinct from intentional fallback because there's no log line and the success value is fabricated.
Catch the specific exception type, log at error level with full exception info, and return a failure-shaped result. If the operation is genuinely best-effort, log at warning and document why in a comment so the next reader (or scanner) knows.
system/hardware/tici/iwlist.py:20
high Security checks quality Quality conf 0.72 Agent control bridge may listen on a network interface without visible auth
Agent, MCP, sidecar, and command bridge servers often start as local helpers. Binding them to 0.0.0.0 or a default all-interface listener without an authorization guard can expose tool execution or session data to the LAN.
system/webrtc/webrtcd.py:12
medium Security checks software dependencies conf 0.88 aiohttp: GHSA-hg6j-4rv6-33pg
AIOHTTP is vulnerable to cross-origin redirect with per-request cookies
uv.lock
medium Security checks software dependencies conf 0.88 aiohttp: GHSA-jg22-mg44-37j8
AIOHTTP is Vulnerable to Deserialization of Untrusted Data
uv.lock
low Security checks quality Error handling conf 0.55 ✓ Repobility 15 occurrences Broad exception handler needs review
This handler catches Exception/BaseException. It is actionable when it swallows errors without logging, re-raising, or returning a structured error. Handlers that intentionally convert exceptions into typed error results should not be treated as high risk.
11 files, 15 locations
common/gpio.py:10, 17, 25, 37 (4 hits)
common/utils.py:52, 263 (2 hits)
selfdrive/debug/filter_log_message.py:41
selfdrive/debug/max_lat_accel.py:93
selfdrive/ui/widgets/ssh_key.py:66
system/hardware/base.py:117
system/loggerd/uploader.py:187
system/qcomgpsd/nmeaport.py:116
Error handlingquality
medium Security checks cicd CI/CD security conf 0.76 Dockerfile copies broad context with incomplete .dockerignore
COPY . or ADD . is safer when .dockerignore excludes secrets, git history, keys, and generated artifacts.
Dockerfile.openpilot:29 CI/CD securitycontainers
medium Security checks quality Quality conf 0.78 Public web service has no security.txt
security.txt gives researchers and customers a safe disclosure channel. Public web apps and APIs should publish it under /.well-known/security.txt.
.well-known/security.txt
medium Security checks quality Quality conf 0.78 Suspicious implementation file appears unreferenced
A file created as a fixed/new/final/copy variant is not referenced by imports or path-like strings in the rest of the repository. This is a strong sign that an agent produced code beside the active application path.
selfdrive/debug/live_cpu_and_temp.py:1
medium System graph quality Integrity conf 1.00 `fetch()` without try/.catch or AbortSignal — tools/bodyteleop/static/js/webrtc.js:10
Bare `fetch(...)` will throw an unhandled rejection on network failure. Wrap in try/catch, attach a `.catch(...)`, or pass an AbortSignal with a timeout.
runtime safetyRobustness
medium System graph cicd CI/CD security conf 1.00 3 occurrences GitHub Actions workflow grants broad write permissions
CI tokens with write permissions increase blast radius when an action, dependency, or PR workflow is compromised. Prefer job-level least-privilege permissions.
3 files, 3 locations
.github/workflows/jenkins-pr-trigger.yaml
.github/workflows/prebuilt.yaml
.github/workflows/release.yaml
CI/CD securitySupply chainGithub actions
medium System graph security security conf 1.00 Insecure pattern 'subprocess_shell_true' in system/hardware/tici/agnos.py:267
Found a known-risky pattern (subprocess_shell_true). Review and replace if possible.
system/hardware/tici/agnos.py:267 Subprocess shell true
medium System graph security security conf 1.00 Insecure pattern 'subprocess_shell_true' in system/timed.py:22
Found a known-risky pattern (subprocess_shell_true). Review and replace if possible.
system/timed.py:22 Subprocess shell true
medium System graph security security conf 1.00 Insecure pattern 'subprocess_shell_true' in system/tombstoned.py:40
Found a known-risky pattern (subprocess_shell_true). Review and replace if possible.
system/tombstoned.py:40 Subprocess shell true
medium System graph security security conf 1.00 Insecure pattern 'subprocess_shell_true' in tools/bodyteleop/web.py:27
Found a known-risky pattern (subprocess_shell_true). Review and replace if possible.
tools/bodyteleop/web.py:27 Subprocess shell true
medium System graph security security conf 1.00 Insecure pattern 'subprocess_shell_true' in tools/plotjuggler/juggle.py:89
Found a known-risky pattern (subprocess_shell_true). Review and replace if possible.
tools/plotjuggler/juggle.py:89 Subprocess shell true
medium System graph security security conf 1.00 Insecure pattern 'weak_hash' in scripts/ci_results.py:75
Found a known-risky pattern (weak_hash). Review and replace if possible.
scripts/ci_results.py:75 Weak hash
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — scripts/ci_results.py:19
`subprocess.check_output(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — selfdrive/pandad/pandad.py:102
`subprocess.Popen(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — system/hardware/tici/agnos.py:98
`subprocess.check_output(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — system/hardware/tici/hardware.py:83
`subprocess.check_output(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — system/hardware/tici/modem.py:87
`subprocess.Popen(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — system/manager/process.py:233
`subprocess.Popen(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — system/ui/lib/wifi_manager.py:804
`subprocess.run(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — system/updated/casync/common.py:15
`subprocess.check_output(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — system/updated/updated.py:72
`subprocess.check_output(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — tools/clip/run.py:155
`subprocess.run(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — tools/lib/comma_car_segments.py:13
`requests.get(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — tools/lib/framereader.py:58
`subprocess.check_output(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — tools/plotjuggler/juggle.py:66
`subprocess.check_output(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
medium System graph quality Integrity conf 1.00 Network/subprocess call without timeout or try/except — tools/scripts/devsync.py:35
`subprocess.check_output(...)` here lacks both a `timeout=` arg and an enclosing try/except. This is exactly the class of bug that took down our git-clone earlier (HTTP/2 stream cancel surfaced as a fatal). Add a `timeout=` and wrap in try/except, or use a wrapper that retries.
runtime safetyRobustness
low Security checks cicd CI/CD security conf 0.72 .dockerignore misses sensitive defaults
.dockerignore exists but does not cover common secret or VCS patterns.
.dockerignore CI/CD securitycontainers
low Security checks software Race condition conf 1.00 [SEC124] TOCTOU file access (os.access then open): Check-then-use file pattern (access/exists then open) lets an attacker swap the file between check and use (symlink attack). `mktemp` is deprecated for the same reason.
Use `os.open(path, os.O_CREAT | os.O_EXCL | os.O_WRONLY)` for atomic create-only. Use `tempfile.NamedTemporaryFile()` (not `mktemp`). For locking, use `fcntl.flock`.
tools/scripts/extract_audio.py:39
low Security checks quality Quality conf 0.60 10 occurrences Duplicated implementation block across source files
Duplicate implementation blocks are maintenance debt. Keep them visible, but they are not a high-severity defect unless the duplicated logic is security-sensitive or drifting.
10 files, 10 locations
selfdrive/debug/read_dtc_status.py:10
selfdrive/selfdrived/selfdrived.py:405
selfdrive/ui/onroad/alert_renderer.py:70
selfdrive/ui/onroad/augmented_road_view.py:78
selfdrive/ui/onroad/cameraview.py:11
selfdrive/ui/onroad/model_renderer.py:10
system/sensord/sensors/lsm6ds3_gyro.py:23
system/ui/lib/multilang.py:43
duplicationquality
high Security checks quality Quality conf 0.62 Source file name looks like an AI patch artifact
Files named as final, fixed, copy, new, or backup are often temporary patch artifacts. They may be legitimate, but they deserve review before becoming production surface area.
system/sensord/sensors/lsm6ds3_temp.py:1
high Security checks quality Quality conf 0.62 Source file name looks like an AI patch artifact
Files named as final, fixed, copy, new, or backup are often temporary patch artifacts. They may be legitimate, but they deserve review before becoming production surface area.
selfdrive/debug/live_cpu_and_temp.py:1
low System graph quality Maintenance conf 1.00 219 TODO/FIXME markers
High count of TODO/FIXME/HACK markers — track them as issues so they're not forgotten.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: common/basedir.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: common/params.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: release/release_files.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: scripts/disable-powersave.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/car/docs.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/car/clear_dtc.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/car/disable_ecu.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/car/ecu_addrs.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/car/fw_versions.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/car/vin.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/check_freq.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/check_lag.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/check_timings.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/count_events.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/fuzz_fw_fingerprint.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/get_fingerprint.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/measure_torque_time_to_max.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/print_flags.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/read_dtc_status.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/run_process_on_route.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/set_car_params.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/test_fw_query_on_routes.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/touch_replay.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/debug/uiview.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/test/cpp_harness.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/ui/tests/body.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/ui/tests/cycle_offroad_alerts.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: selfdrive/ui/watch3.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: system/hardware/tici/precise_power_measure.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: tools/bodyteleop/static/js/jsmain.js
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: tools/car_porting/auto_fingerprint.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: tools/lib/openpilotcontainers.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: tools/scripts/fetch_image_from_route.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: tools/scripts/setup_ssh_keys.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph software Dead code candidate conf 1.00 File has no detected symbols: tools/scripts/ssh.py
Source file with no class/function declarations — possible config, dead code, or scratch file.
low System graph quality Integrity conf 1.00 17 occurrences Near-duplicate function bodies in 2 places
Functions with the same first-5-line body hash: selfdrive/test/helpers.py:wrap, selfdrive/test/helpers.py:wrap This is *the* AI-coder failure mode (4× more duplication in vibe-coded repos — see https://jw.hn/ai-code-hygiene). Consolidate or document why they're separate.
17 occurrences
repo-level (17 hits)
duplicatesduplication
low System graph quality Integrity conf 1.00 3 occurrences Near-duplicate function bodies in 3 places
Functions with the same first-5-line body hash: selfdrive/ui/widgets/offroad_alerts.py:refresh, selfdrive/ui/widgets/offroad_alerts.py:refresh, selfdrive/ui/widgets/offroad_alerts.py:refresh This is *the* AI-coder failure mode (4× more duplication in vibe-coded repos — see https://jw.hn/ai-code-hy…
3 occurrences
repo-level (3 hits)
duplicatesduplication
low System graph quality Integrity conf 1.00 Old/deprecated-named symbol `from_buffer_copy` in system/sensord/sensord.py:55
Names with suffixes like `_old`, `_v1`, `_deprecated` usually indicate replaced-but-not-removed code (typical AI-coder leftover). Confirm and delete, or rename if it's the active version.
old markerDead code
low System graph quality Integrity conf 1.00 Old/deprecated-named symbol `getDeprecated` in tools/jotpluggler/generate_event_extractors.py:243
Names with suffixes like `_old`, `_v1`, `_deprecated` usually indicate replaced-but-not-removed code (typical AI-coder leftover). Confirm and delete, or rename if it's the active version.
old markerDead code
low System graph quality Integrity conf 1.00 Old/deprecated-named symbol `last_parameters_data_old` in selfdrive/locationd/paramsd.py:204
Names with suffixes like `_old`, `_v1`, `_deprecated` usually indicate replaced-but-not-removed code (typical AI-coder leftover). Confirm and delete, or rename if it's the active version.
old markerDead code
low System graph quality Integrity conf 1.00 Old/deprecated-named symbol `model_v2` in selfdrive/controls/controlsd.py:89
Names with suffixes like `_old`, `_v1`, `_deprecated` usually indicate replaced-but-not-removed code (typical AI-coder leftover). Confirm and delete, or rename if it's the active version.
old markerDead code
low System graph software Dead code conf 1.00 Possibly dead Python function: confirm_callback_wrapper
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/widgets/dialog.py:101
low System graph software Dead code conf 1.00 Possibly dead Python function: decorator
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/onroad/torque_bar.py:23
low System graph software Dead code conf 1.00 Possibly dead Python function: excessive_actuation_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/widgets/offroad_alerts.py:99
low System graph software Dead code conf 1.00 Possibly dead Python function: github_username_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/developer.py:37
low System graph software Dead code conf 1.00 Possibly dead Python function: handle_language_selection
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/layouts/settings/device.py:82
low System graph software Dead code conf 1.00 Possibly dead Python function: inactivity_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/onboarding.py:111
low System graph software Dead code conf 1.00 Possibly dead Python function: max_severity
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/offroad_alerts.py:217
low System graph software Dead code conf 1.00 Possibly dead Python function: network_metered_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/network/network_layout.py:50
low System graph software Dead code conf 1.00 Possibly dead Python function: offroad_transition
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/device.py:183
low System graph software Dead code conf 1.00 Possibly dead Python function: perform_power_off
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/layouts/settings/device.py:179
low System graph software Dead code conf 1.00 Possibly dead Python function: perform_reboot
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/layouts/settings/device.py:167
low System graph software Dead code conf 1.00 Possibly dead Python function: power_off_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/device.py:295
low System graph software Dead code conf 1.00 Possibly dead Python function: reboot_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/device.py:298
low System graph software Dead code conf 1.00 Possibly dead Python function: reset_calibration_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/device.py:301
low System graph software Dead code conf 1.00 Possibly dead Python function: scrolling
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/offroad_alerts.py:220
low System graph software Dead code conf 1.00 Possibly dead Python function: show_confirm_dialog
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/widgets/dialog.py:236
low System graph software Dead code conf 1.00 Possibly dead Python function: snooze_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/widgets/offroad_alerts.py:94
low System graph software Dead code conf 1.00 Possibly dead Python function: ssh_keys_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/developer.py:55
low System graph software Dead code conf 1.00 Possibly dead Python function: tethering_password_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/network/network_layout.py:33
low System graph software Dead code conf 1.00 Possibly dead Python function: tethering_password_clicked
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/network/network_layout.py:39
low System graph software Dead code conf 1.00 Possibly dead Python function: tethering_toggle_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/network/network_layout.py:25
low System graph software Dead code conf 1.00 Possibly dead Python function: uninstall_openpilot_callback
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/device.py:310
low System graph software Dead code conf 1.00 Possibly dead Python function: update_apn
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/layouts/settings/network/network_layout.py:115
low System graph software Dead code conf 1.00 Possibly dead Python function: update_filter
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/onroad/confidence_ball.py:30
low System graph software Dead code conf 1.00 Possibly dead Python function: update_filter
No callers detected by AST scan in this repo. Could be exported for external callers or a framework handler.
selfdrive/ui/mici/onroad/torque_bar.py:157
low System graph frontend Frontend quality conf 1.00 Stray `console.log` in TS/JS — tools/bodyteleop/static/js/webrtc.js:34
Replace with the toast helper, an error boundary, or remove. `console.warn` / `console.error` are acceptable. Why: Hygiene — easy to leak debug output. Rule id: fq.console-leak
Fq console leak
low System graph quality Integrity conf 1.00 Stub function `apply_controls` (body is just `pass`/`return`) — tools/sim/lib/common.py:75
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `get_active_profile` (body is just `pass`/`return`) — system/hardware/tici/lpa.py:728
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `list_profiles` (body is just `pass`/`return`) — system/hardware/base.py:75
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `noop` (body is just `pass`/`return`) — selfdrive/test/helpers.py:109
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `prepare` (body is just `pass`/`return`) — system/manager/process.py:73
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `read_cameras` (body is just `pass`/`return`) — tools/sim/bridge/metadrive/metadrive_world.py:120
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `refresh` (body is just `pass`/`return`) — selfdrive/ui/widgets/offroad_alerts.py:129
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `run` (body is just `pass`/`return`) — system/webrtc/webrtcd.py:52
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `shutdown` (body is just `pass`/`return`) — system/sensord/sensors/lsm6ds3_temp.py:31
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `spawn_world` (body is just `pass`/`return`) — tools/sim/bridge/common.py:98
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
low System graph quality Integrity conf 1.00 Stub function `update` (body is just `pass`/`return`) — selfdrive/controls/lib/latcontrol.py:16
Likely an AI scaffold that was never filled in. Remove or implement.
Empty handlerDead code
For AI agents: Voting guide (TP/FP) MCP manifest Stdio wrapper SARIF Integrate Findings queue Vote TP/FP on findings to calibrate the engine.
For AI agents + API integrations
Email me when this repo regresses
Free. We re-scan periodically; new criticals → your inbox. No signup required for the scan itself.
API access

This page is publicly accessible at: https://repobility.com/scan/13523a63-0821-42b2-b202-a86fdc09d0c9/

To check status programmatically (no auth required): 

curl -s https://repobility.com/api/v1/public/scan/13523a63-0821-42b2-b202-a86fdc09d0c9/

Important — please don't re-submit the same URL repeatedly. The submission endpoint is idempotent: re-submitting the same git URL returns this same scan_token, not a new one. To re-scan this repo, sign up free and use the dashboard.