Merge ncp_conn.c:1.28 from HEAD to RELENG_6:

Merge ncp_conn.c:1.28 from HEAD to RELENG_6:

  In ncp_sysctl_connstat(), the SLIST_FOREACH() logic to check 'error'
  resulted in deadcode, as 'error' could never be 0.  What this logic
  was originally meant to handle is not clear -- it's been this way
  (broken) since at least RELENG_4.

  Found with:     Coverity Prevent(tm)

Approved by:	re (scottl)