Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 8 Jun 2009 20:24:29 +0000 (UTC)
From:      Ed Schouten <ed@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r193752 - head/sys/dev/kbdmux
Message-ID:  <200906082024.n58KOTXU042429@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: ed
Date: Mon Jun  8 20:24:29 2009
New Revision: 193752
URL: http://svn.freebsd.org/changeset/base/193752

Log:
  Use proper types in kbdmux_kbd_getc():
  
  - The return value should be a signed integer, because -1 means failure.
  - The c variable should be unsigned, to force it to be zero-extended
    when returned.
  
  Reported by:	Andreas Tobler <andreast-list fgznet ch>

Modified:
  head/sys/dev/kbdmux/kbdmux.c

Modified: head/sys/dev/kbdmux/kbdmux.c
==============================================================================
--- head/sys/dev/kbdmux/kbdmux.c	Mon Jun  8 20:08:20 2009	(r193751)
+++ head/sys/dev/kbdmux/kbdmux.c	Mon Jun  8 20:24:29 2009	(r193752)
@@ -181,10 +181,10 @@ kbdmux_kbd_putc(kbdmux_state_t *state, c
 	state->ks_inq_length++;
 }
 
-static char
+static int
 kbdmux_kbd_getc(kbdmux_state_t *state)
 {
-	char c;
+	unsigned char c;
 
 	if (state->ks_inq_length == 0)
 		return (-1);



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200906082024.n58KOTXU042429>