On Thu, 10 Dec 2009, Luiz Capitulino wrote:

> Return a QDict with server information. Connected clients are returned
> as a QList of QDicts.
> 
> The new functions (vnc_qdict_remote_addr(), vnc_qdict_local_addr() and
> put_addr_qdict()) are used to insert 'host' and 'service' information
> in the returned QDict.
> 
> This patch is big, but I don't see how to split it.
> 
> Signed-off-by: Luiz Capitulino <lcapitul...@redhat.com>
[..snip..]

> +
> +static int vnc_qdict_local_addr(QDict *qdict, int fd)
> +{
> +    struct sockaddr_storage sa;
> +    socklen_t salen;
> +
> +    salen = sizeof(sa);
> +    if (getsockname(fd, (struct sockaddr*)&sa, &salen) < 0)
> +        return -1;

Coding style violation, here and all over the place.

[..snip..]

-- 
mailto:av1...@comtv.ru


Reply via email to