Open
Conversation
- Zero-dependency Python 3 implementation using stdlib only - Multi-threaded TCP scanning with configurable thread count - Supports port ranges, comma-separated lists, and top-100 preset - Service identification for 80+ common ports - Optional banner grabbing for service fingerprinting - Color terminal output with scan summary https://claude.ai/code/session_01DGJrP8B7ohyiEG1h1khr4f
Limit concurrent socket connections to 50 via threading.Semaphore to prevent macOS from killing the process (Killed: 9) when too many file descriptors are opened simultaneously during a scan. https://claude.ai/code/session_01DGJrP8B7ohyiEG1h1khr4f
Lower semaphore from 50 to 30 and default thread count from 100 to 50 to stay well below macOS's default fd limit of 256 per process. https://claude.ai/code/session_01DGJrP8B7ohyiEG1h1khr4f
…fix worker race condition Three root causes fixed: 1. scan socket was still open while grab_banner() opened a second socket, doubling actual concurrent FDs; moved grab_banner() call after the `with socket` block so the scan socket is always closed first 2. worker() used `while not queue.empty()` + `queue.get()` — a TOCTOU race where threads could block indefinitely on get(); replaced with queue.get(block=False) + Empty exception 3. semaphore was a fixed global (30) independent of thread count; now created dynamically in run_scan() as min(threads, 20) so the limit always matches actual concurrency; default threads lowered 50→20 4. added _raise_fd_limit() to programmatically raise macOS fd soft limit to 2048 at startup https://claude.ai/code/session_01DGJrP8B7ohyiEG1h1khr4f
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
https://claude.ai/code/session_01DGJrP8B7ohyiEG1h1khr4f