[PATCH] Fix: transaction id usage in gisi/server.c

Rémi Denis-Courmont remi at remlab.net
Fri Apr 23 08:27:51 PDT 2010


On Fri, 23 Apr 2010 18:17:16 +0300, Pekka Pessi <Pekka.Pessi at nokia.com>
wrote:
> @@ -174,9 +172,11 @@ g_isi_server_add_name(GIsiServer *self)
>  			0, 0,
>  		};
> 
> -		if (sendto(self->fd, req, sizeof(req), 0, (void *)&spn,
> -				sizeof(spn)) != sizeof(spn))
> -			return;
> +		if (sendto(self->fd, req, sizeof(req), 0,
> +				(void *)&spn, sizeof(spn)) != sizeof(req)) {
> +			g_warning("%s: %s", "sendto(PN_NAMESERVICE)",
> +				  strerror(errno));
> +		}
>  	}

sendto() does not set errno if it returns less than sizeof(req) but more
than -1. I don't know if we care though.

-- 
Rémi Denis-Courmont
http://www.remlab.net
http://fi.linkedin.com/in/remidenis



More information about the ofono mailing list