Quoth Tony Nguyen on Thu, Jun 05, 2008 at 01:21:01PM -0700:
> Webrev: http://cr.opensolaris.org/~tonyn/6710856/

- s/shouldn not/should not/ in the bug synopsis.

cmd/svc/configd/rc_node.c
  7207: Please add a comment explaining that the name is already being
    tracked, so we don't have to do anything.

  7208: This will leak f.

  7218: I haven't investigated this too much, but it's not clear to me
    that we shouldn't do this if we're already watching the name.  Do
    you know why?


David

Reply via email to