Re: [RFC,PATCH] svc: REPOST - Fix the UDP address logic

[Greg Banks <gnb@sgi.com>]

On Wed, Oct 24, 2007 at 02:29:59PM -0400, Chuck Lever wrote:
> Tom Tucker wrote:
> > 
> >+static size_t svc_sockaddr_len(struct sockaddr *addr)
> >+{
> >+	switch (addr->sa_family) {
> >+	case AF_INET:
> >+		return sizeof(struct sockaddr_in);
> >+	case AF_INET6:
> >+		return sizeof(struct sockaddr_in6);
> >+	}
> >+	dprintk("svc: unrecognized address family %d\n", addr->sa_family);
> >+	return -ENOTSUPP;
> >+}
> >+
> 
> Nit: sa_family is probably an unsigned short, so use "%u" instead of
> "%d".  Also I'm using -EAFNOSUPPORT in similar error cases, but I may 
> have missed a stated preference.

EAFNOSUPPORT is the most appropriate error.

Also, you might want to make the return the signed ssize_t so that
the return value -EAFNOSUPPORT is actually negative.

Greg.
-- 
Greg Banks, R&D Software Engineer, SGI Australian Software Group.
Apparently, I'm Bedevere.  Which MPHG character are you?
I don't speak for SGI.

-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems?  Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
NFS maillist  -  NFS@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/nfs