[PATCH] Fix comparison between two character arrays

Corinna Vinschen vinschen@redhat.com
Mon Jul 30 07:50:00 GMT 2018


On Jul 27 11:02, Jordi Sanfeliu wrote:
> 
> Hello,
> 
> The member 'id' in the 'utmp' struct is not a numeric but a character array,
> hence the strncmp() function is needed to compare two members.
> 
> diff --git a/newlib/libc/unix/getut.c b/newlib/libc/unix/getut.c
> index 89ed739..71a8c8f 100644
> --- a/newlib/libc/unix/getut.c
> +++ b/newlib/libc/unix/getut.c
> @@ -63,7 +63,7 @@ getutid (struct utmp *id)
>  	case LOGIN_PROCESS:
>  	case USER_PROCESS:
>  	case DEAD_PROCESS:
> -	  if (id->ut_id == utmp_data.ut_id)
> +	  if (!strncmp (id->ut_id, utmp_data.ut_id, sizeof (utmp_data.ut_id)))
>  	    return &utmp_data;
>  	  break;
>  	default:
> 
> Thanks.
> 
> --
> Jordi Sanfeliu
> FIBRANET Network Services Provider
> https://www.fibranet.cat

Pushed.


Thanks,
Corinna

-- 
Corinna Vinschen
Cygwin Maintainer
Red Hat
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://sourceware.org/pipermail/newlib/attachments/20180730/4c5df778/attachment.sig>


More information about the Newlib mailing list