On 05/07/2015 06:15 PM, Bryce Harrington wrote: Extra call to strcpy here: >> wl->tty = open(tty, O_RDWR | O_NOCTTY); >> + strcpy(wl->tty_path, tty); >> + if (snprintf(wl->tty_path, PATH_MAX, "%s", tty) >= PATH_MAX) >> + error(1, 0, "tty filename too long"); >> + }