[PATCH 1/9] dmx: Fix a non-constant printf format string warning.
Alan Coopersmith
alan.coopersmith at oracle.com
Sat Sep 17 09:45:32 PDT 2011
On 09/17/11 01:22, Jamey Sharp wrote:
> Signed-off-by: Jamey Sharp<jamey at minilop.net>
> ---
> hw/dmx/input/lnx-keyboard.c | 4 ++--
> 1 files changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/hw/dmx/input/lnx-keyboard.c b/hw/dmx/input/lnx-keyboard.c
> index 5bfb881..5b92f3b 100644
> --- a/hw/dmx/input/lnx-keyboard.c
> +++ b/hw/dmx/input/lnx-keyboard.c
> @@ -429,11 +429,11 @@ static int kbdLinuxOpenVT(int vtno)
> {
> int fd = -1;
> int i;
> - const char *vcs[] = { "/dev/vc/%d", "/dev/tty%d", NULL };
> + const char *vcs[] = { "/dev/vc/", "/dev/tty", NULL };
> char name[64]; /* RATS: Only used in XmuSnprintf */
>
> for (i = 0; vcs[i]; i++) {
> - XmuSnprintf(name, sizeof(name), vcs[i], vtno);
> + XmuSnprintf(name, sizeof(name), "%s%d", vcs[i], vtno);
> if ((fd = open(name, O_RDWR | O_NONBLOCK, 0))>= 0) break;
> }
> if (fd< 0)
Reviewed-by: Alan Coopersmith <alan.coopersmith at oracle.com>
Though as Walter noted, XmuSnprintf() can be replaced by snprintf()
now. (It was a implementation X provided for it's libraries to use
in the days before all platforms we cared about had snprintf in libc.)
--
-Alan Coopersmith- alan.coopersmith at oracle.com
Oracle Solaris Platform Engineering: X Window System
More information about the xorg-devel
mailing list