Skip to content

Commit f5346e4

Browse files
committed
Args.c: drop 255 byte limit for -keyboard option
Could not find a reason to limit that string.
1 parent a447dc5 commit f5346e4

File tree

1 file changed

+2
-20
lines changed
  • nx-X11/programs/Xserver/hw/nxagent

1 file changed

+2
-20
lines changed

nx-X11/programs/Xserver/hw/nxagent/Args.c

+2-20
Original file line numberDiff line numberDiff line change
@@ -643,31 +643,13 @@ int ddxProcessArgument(int argc, char *argv[], int i)
643643
{
644644
if (++i < argc)
645645
{
646-
int size;
647-
648646
free(nxagentKeyboard);
649647
nxagentKeyboard = NULL;
650648

651-
/* FIXME: why limit to 256? */
652-
if ((size = strlen(argv[i])) < 256)
649+
if ((nxagentKeyboard = strdup(argv[i])) == NULL)
653650
{
654-
if ((nxagentKeyboard = strndup(argv[i], size)) == NULL)
655-
{
656-
FatalError("malloc for -keyboard failed");
657-
}
651+
FatalError("malloc for -keyboard failed");
658652
}
659-
#ifdef WARNING
660-
else
661-
{
662-
/*
663-
* it is useless to remember a kbtype
664-
* option that has just been truncated.
665-
*/
666-
667-
fprintf(stderr, "ddxProcessArgument: WARNING! Option [%s] too long. "
668-
"It will be ignored.\n", argv[i]);
669-
}
670-
#endif
671653
return 2;
672654
}
673655
return 0;

0 commit comments

Comments
 (0)