Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Live capture: fix corrupt state on Windows #37

Merged
merged 1 commit into from
Aug 4, 2024
Merged

Conversation

oshaked1
Copy link
Collaborator

@oshaked1 oshaked1 commented Aug 4, 2024

On Windows, Wireshark does not stop the capture program when using Wireshark's stop or restart button. If a capture is stopped that way and then restarted, the previous running capture prevents the new capture from being able to set up an SSH tunnel (for remote capture). To fix this we use a PID file and kill the previous capture before starting the new one. Additionally an already running Tracee container causes issues where the SSH tunnel forwarding thread gets a connection from Tracee before the reader thread starts which causes an error. To fix this we remove the existing Tracee container before we setup the new capture.

On Windows, Wireshark does not stop the capture program when using Wireshark's stop or restart button.
If a capture is stopped that way and then restarted, the previous running capture
prevents the new capture from being able to set up an SSH tunnel (for remote capture).
To fix this we use a PID file and kill the previous capture before starting the new one.
Additionally an already running Tracee container causes issues where the SSH tunnel forwarding thread gets a connection from Tracee before the reader thread starts which causes an error.
To fix this we remove the existing Tracee container before we setup the new capture.
@oshaked1 oshaked1 self-assigned this Aug 4, 2024
@oshaked1 oshaked1 added area/live capture bug Something isn't working labels Aug 4, 2024
@oshaked1 oshaked1 merged commit 3f88daa into main Aug 4, 2024
6 checks passed
@oshaked1 oshaked1 deleted the live_capture_fix branch August 4, 2024 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/live capture bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant