As part of this continuing code review, I looked at http://cr.opensolaris.org/~tonyn/firewall13Jan2009/usr/src/cmd/svc/servinfo/servinfo.c
and I'm wondering if you have, in your latest workspace, a version of uaddr2port() that doesn't leak memory from the malloc() for port_str? The other alternative is to use alloca() instead. Darren
