[PATCH] some signed/unsigned hickups

Jeremy Allison jra at samba.org
Tue Aug 23 04:48:41 UTC 2016


On Tue, Aug 23, 2016 at 06:41:39AM +0200, Volker Lendecke wrote:
> Hi!
> 
> Review appreciated!
> 
> Thanks, Volker

> From b04251559de42a7ff0f7afd7f994d17cdfae0e93 Mon Sep 17 00:00:00 2001
> From: Volker Lendecke <vl at samba.org>
> Date: Sun, 21 Aug 2016 17:46:16 +0200
> Subject: [PATCH] messaging_dgm: Fix signed/unsigned hickups
> 
> Signed-off-by: Volker Lendecke <vl at samba.org>
> ---
>  source3/lib/messages_dgm.c | 29 ++++++++++++++++++++++-------
>  1 file changed, 22 insertions(+), 7 deletions(-)
> 
> diff --git a/source3/lib/messages_dgm.c b/source3/lib/messages_dgm.c
> index 744e273..74879f9 100644
> --- a/source3/lib/messages_dgm.c
> +++ b/source3/lib/messages_dgm.c
> @@ -76,7 +76,10 @@ static int messaging_dgm_lockfile_create(struct messaging_dgm_context *ctx,
>  
>  	ret = snprintf(lockfile_name.buf, sizeof(lockfile_name.buf),
>  		       "%s/%u", ctx->lockfile_dir.buf, (unsigned)pid);
> -	if (ret >= sizeof(lockfile_name.buf)) {
> +	if (ret < 0) {
> +		return errno;
> +	}
> +	if ((unsigned)ret >= sizeof(lockfile_name.buf)) {
>  		return ENAMETOOLONG;
>  	}
>  
> @@ -272,7 +275,7 @@ static int messaging_dgm_context_destructor(struct messaging_dgm_context *c)
>  
>  		ret = snprintf(name.buf, sizeof(name.buf), "%s/%u",
>  			       c->lockfile_dir.buf, (unsigned)c->pid);
> -		if (ret >= sizeof(name.buf)) {
> +		if ((ret < 0) || ((size_t)ret >= sizeof(name.buf))) {
>  			/*
>  			 * We've checked the length when creating, so this
>  			 * should never happen
> @@ -312,7 +315,10 @@ int messaging_dgm_send(pid_t pid,
>  
>  	dst_pathlen = snprintf(dst.sun_path, sizeof(dst.sun_path),
>  			       "%s/%u", ctx->socket_dir.buf, (unsigned)pid);
> -	if (dst_pathlen >= sizeof(dst.sun_path)) {
> +	if (dst_pathlen == -1) {

Shouldn't the above be:

	if (dst_pathlen < 0) {

to match the other changes ?

With that change, Reviewed-by: Jeremy Allison <jra at samba.org>
Feel free to push !

> +		return errno;
> +	}
> +	if ((size_t)dst_pathlen >= sizeof(dst.sun_path)) {
>  		return ENAMETOOLONG;
>  	}
>  
> @@ -381,7 +387,10 @@ int messaging_dgm_get_unique(pid_t pid, uint64_t *unique)
>  
>  	ret = snprintf(lockfile_name.buf, sizeof(lockfile_name.buf),
>  		       "%s/%u", ctx->lockfile_dir.buf, (int)pid);
> -	if (ret >= sizeof(lockfile_name.buf)) {
> +	if (ret < 0) {
> +		return errno;
> +	}
> +	if ((size_t)ret >= sizeof(lockfile_name.buf)) {
>  		return ENAMETOOLONG;
>  	}
>  
> @@ -408,13 +417,19 @@ int messaging_dgm_cleanup(pid_t pid)
>  
>  	len = snprintf(socket_name.buf, sizeof(socket_name.buf), "%s/%u",
>  		       ctx->socket_dir.buf, (unsigned)pid);
> -	if (len >= sizeof(socket_name.buf)) {
> +	if (len < 0) {
> +		return errno;
> +	}
> +	if ((size_t)len >= sizeof(socket_name.buf)) {
>  		return ENAMETOOLONG;
>  	}
>  
>  	len = snprintf(lockfile_name.buf, sizeof(lockfile_name.buf), "%s/%u",
>  		       ctx->lockfile_dir.buf, (unsigned)pid);
> -	if (len >= sizeof(lockfile_name.buf)) {
> +	if (len < 0) {
> +		return errno;
> +	}
> +	if ((size_t)len >= sizeof(lockfile_name.buf)) {
>  		return ENAMETOOLONG;
>  	}
>  
> @@ -485,7 +500,7 @@ int messaging_dgm_wipe(void)
>  			 */
>  			continue;
>  		}
> -		if (pid == our_pid) {
> +		if ((pid_t)pid == our_pid) {
>  			/*
>  			 * fcntl(F_GETLK) will succeed for ourselves, we hold
>  			 * that lock ourselves.
> -- 
> 2.1.4
> 




More information about the samba-technical mailing list