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

Kill telnetd during startup to prevent security vuln #17

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

ea-dot-codes
Copy link

No description provided.

@Xpndable
Copy link
Owner

Xpndable commented Apr 7, 2023

This will be pulled in shortly - wasn't an issue for the v1.0 release because there is currently no WiFi configuration capability in DotUI

@ericanderson
Copy link

I got my device and started reverse engineering for a day before heading on holiday. I wasn't sure if previously configuring wifi would keep this on

@anzz1
Copy link

anzz1 commented Apr 25, 2023

This is definitely a good idea and should be added when the wifi functionality is introduced.

However I'd make a tiny change, move the killall to launch.sh where the actual launch script is. No functional difference, just keeping in line with the current structure. Tidiness is next to godliness, they say 😄

# killall tee # NOTE: killing tee is somehow responsible for audioserver crashes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants