empire/empserver
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

(options_cmd): login_opts[] lacked sentinel. Potential crash-bug.

Browse source 
Simplify.
This commit is contained in:
Markus Armbruster 2006-06-14 18:07:37 +00:00
parent f940413c67
commit bb2a725b0d
1 changed files with 3 additions and 2 deletions
Download patch file Download diff file Expand all files Collapse all files

5
src/lib/player/login.c
View file

@ -246,7 +246,8 @@ options_cmd(void)
int val; int val;
}; };
static struct logoptstr login_opts[] = { static struct logoptstr login_opts[] = {
{ "utf-8", PF_UTF8 } { "utf-8", PF_UTF8 },
{ NULL, 0 }
}; };
char **ap; char **ap;
char *p; char *p;
@ -254,7 +255,7 @@ options_cmd(void)
unsigned i; unsigned i;
if (!player->argp[1]) { if (!player->argp[1]) {
for (i = 0; i < sizeof(login_opts) / sizeof(*login_opts); ++i) for (i = 0; login_opts[i].name; ++i)
pr_id(player, C_DATA, "%s=%d\n", pr_id(player, C_DATA, "%s=%d\n",
login_opts[i].name, login_opts[i].name,
(player->flags & login_opts[i].val) != 0); (player->flags & login_opts[i].val) != 0);