truss and procfs

Hiten Pandya hmp at backplane.com
Sun Oct 19 06:21:14 PDT 2003


Eirik Nygaard wrote:
truss(1) always looks for procfs in /proc, it also bails out with some not
very informative error messages if procfs is not mounted.
http://eirikn.kerneled.com/dragonfly/truss-procfs.diff checks where procfs
is mounted and if it is not present truss now tells the user what is
wrong.
	Hi Eirik,

	Hmm, I did a quick review, and the following type of changes
	freaked me out, as they are a No No:
	-  char buf[32];
	+  char buf[PATH_MAX + NAME_MAX];
	You shouldn't do PATH_MAX + NAME_MAX, because PATH_MAX is
	supposed to be the total maximum including the name.  Your
	defines will expand to 1024+255 which is scary. ;-)
	Regards,

--
Hiten Pandya
hmp at xxxxxxxxxxxxx





More information about the Submit mailing list