In the sudosh.c the termnew.c_lflag &= ~ECHO; seems useless

[wzis]

In the rawmode function, the termnew is declared as a static struct termios, and without any other assignment for the c_lflag, its value remains 0, then why do you need the termnew.c_lflag &= ~ECHO ?

[squash]

I assume it is unnecessary leftover from the original author. Generally changes to the code since I took over was to implement new features and compatibility with newer systems.