Cleanup in cons.saver

Pavel Roskin proski at gnu.org
Fri Jul 6 19:51:56 UTC 2001


Hello!

I have added support for /dev/vcc/a* in cons.saver. In order to implement
it safely and correctly I used the minor device number of the console
(which I called console_minor).

It turned out that using this variable eliminates a number of hacks and
makes cons.saver more secure. cons.saver no longer parses the argument -
instead it checks the minor number of the console and makes sure that the
console name is either /dev/ttyN or /dev/vc/N where N is console_minor as
defined by fstat(). That's much stricter that the old test, and much
simpler, too.

Using console_minor fixes the problem with /dev/tty10 and above on pre-1.2
kernels. I doubt that anybody ever encountered it, but it's better to have
clean code.

Finally, using strings to indicate errors has been disabled. It's a bad
style. Those strings were not visible to the user anyway.

The patch is not very readable, but everyone if welcome to check new
cons.saver.c - it looks better now.

-- 
Regards,
Pavel Roskin





More information about the mc-devel mailing list