[PATCH weston] logind-util: Use %u for unsigned int in snprintf

Bryce Harrington bryce at osg.samsung.com
Fri Jul 17 14:36:54 PDT 2015


On Fri, Jul 17, 2015 at 03:25:55PM -0500, Derek Foreman wrote:
> Looks fine to me.
> 
> Reviewed-By: Derek Foreman <derekf at osg.samsung.com>

Thanks, pushed:
To ssh://git.freedesktop.org/git/wayland/weston
   27135af..c683623  master -> master

 
> On 19/06/15 06:28 PM, Bryce Harrington wrote:
> > Addresses this warning found by Denis Denisov:
> > 
> >   [src/logind-util.c:702]: (warning) %d in format string (no. 1)
> >   requires 'int' but the argument type is 'unsigned int'.
> > 
> > Signed-off-by: Bryce Harrington <bryce at osg.samsung.com>
> > ---
> >  src/logind-util.c | 2 +-
> >  1 file changed, 1 insertion(+), 1 deletion(-)
> > 
> > diff --git a/src/logind-util.c b/src/logind-util.c
> > index 5c95874..006a1ae 100644
> > --- a/src/logind-util.c
> > +++ b/src/logind-util.c
> > @@ -702,7 +702,7 @@ weston_logind_setup_vt(struct weston_logind *wl)
> >  	sigset_t mask;
> >  	struct wl_event_loop *loop;
> >  
> > -	snprintf(buf, sizeof(buf), "/dev/tty%d", wl->vtnr);
> > +	snprintf(buf, sizeof(buf), "/dev/tty%u", wl->vtnr);
> >  	buf[sizeof(buf) - 1] = 0;
> >  
> >  	wl->vt = open(buf, O_RDWR|O_CLOEXEC|O_NONBLOCK);
> > 


More information about the wayland-devel mailing list