Pass -Wstrict-overflow.

This commit is contained in:
dholland 2012-10-12 10:38:53 +00:00
parent cfbd580d96
commit 6dd1299b8b
1 changed files with 7 additions and 7 deletions

View File

@ -1,4 +1,4 @@
/* $NetBSD: wizard.c,v 1.14 2009/08/25 06:56:52 dholland Exp $ */ /* $NetBSD: wizard.c,v 1.15 2012/10/12 10:38:53 dholland Exp $ */
/*- /*-
* Copyright (c) 1991, 1993 * Copyright (c) 1991, 1993
@ -39,7 +39,7 @@
#if 0 #if 0
static char sccsid[] = "@(#)wizard.c 8.1 (Berkeley) 6/2/93"; static char sccsid[] = "@(#)wizard.c 8.1 (Berkeley) 6/2/93";
#else #else
__RCSID("$NetBSD: wizard.c,v 1.14 2009/08/25 06:56:52 dholland Exp $"); __RCSID("$NetBSD: wizard.c,v 1.15 2012/10/12 10:38:53 dholland Exp $");
#endif #endif
#endif /* not lint */ #endif /* not lint */
@ -130,19 +130,19 @@ wizard(void)
void void
ciao(void) ciao(void)
{ {
char *c; char fname[80];
char fname[80]; size_t pos;
printf("What would you like to call the saved version?\n"); printf("What would you like to call the saved version?\n");
/* XXX - should use fgetln to avoid arbitrary limit */ /* XXX - should use fgetln to avoid arbitrary limit */
for (c = fname; c < fname + sizeof fname - 1; c++) { for (pos = 0; pos < sizeof(fname - 1); pos++) {
int ch; int ch;
ch = getchar(); ch = getchar();
if (ch == '\n' || ch == EOF) if (ch == '\n' || ch == EOF)
break; break;
*c = ch; fname[pos] = ch;
} }
*c = 0; fname[pos] = '\0';
if (save(fname) != 0) if (save(fname) != 0)
return; /* Save failed */ return; /* Save failed */
printf("To resume, say \"adventure %s\".\n", fname); printf("To resume, say \"adventure %s\".\n", fname);