xserver: fix an off-by-one error that lead to asprintf("%s", NULL)

reported by deraadt@, fix reviewed in X.Org by Keith Packard.
This commit is contained in:
matthieu
2016-08-03 19:31:23 +00:00
parent 7d876864d7
commit 185312ce1c

View File

@@ -395,8 +395,7 @@ autoConfigDevice(GDevPtr preconf_device)
/* for each other driver found, copy the first screen, insert it /* for each other driver found, copy the first screen, insert it
* into the list of screens and set the driver */ * into the list of screens and set the driver */
i = 0; for (i = 1; i < num_matches; i++) {
while (i++ < num_matches) {
if (!copyScreen(slp[0].screen, ptr, i, matches[i])) if (!copyScreen(slp[0].screen, ptr, i, matches[i]))
return NULL; return NULL;
} }