]> git.ipfire.org Git - thirdparty/shadow.git/commit
lib/port.c: getportent(): Make sure the aren't too many fields in the CSV
authorAlejandro Colomar <alx@kernel.org>
Tue, 2 Jul 2024 12:51:04 +0000 (14:51 +0200)
committerSerge Hallyn <serge@hallyn.com>
Sat, 6 Jul 2024 12:44:02 +0000 (07:44 -0500)
commita198054456728ae3ea8565e11e636c759616ef9c
tree97e085810efe13703fd8e8860c225b5d8b0a67bc
parentc3f97e251ef13f24538e4e52ffe4cba01ea0e840
lib/port.c: getportent(): Make sure the aren't too many fields in the CSV

Otherwise, the line is invalidly formatted, and we ignore it.

Detailed explanation:

There are two conditions on which we break out of the loops that precede
these added checks:

-  j is too big (we've exhausted the space in the static arrays)

$ grep -r -e PORT_TTY -e PORT_IDS lib/port.*
lib/port.c: static char *ttys[PORT_TTY + 1]; /* some pointers to tty names     */
lib/port.c: static char *users[PORT_IDS + 1]; /* some pointers to user ids     */
lib/port.c: for (cp = buf, j = 0; j < PORT_TTY; j++) {
lib/port.c: if ((',' == *cp) && (j < PORT_IDS)) {
lib/port.h: * PORT_IDS - Allowable number of IDs per entry.
lib/port.h: * PORT_TTY - Allowable number of TTYs per entry.
lib/port.h:#define PORT_IDS 64
lib/port.h:#define PORT_TTY 64

-  strpbrk(3) found a ':', which signals the end of the comma-sepatated
   list, and the start of the next colon-separated field.

If the first character in the remainder of the string is not a ':', it
means we've exhausted the array size, but the CSV list was longer, so
we'd be truncating it.  Consider the entire line invalid, and skip it.

Signed-off-by: Alejandro Colomar <alx@kernel.org>
lib/port.c