- No need to copy our argument into a new buffer; it is writeable and will not be reused after we return.
- Instead of constructing the string "get path" and then splitting it into an argument vector, just construct the vector directly. This avoid potentially overrunning the buffer.
- Call settftpmode() just once, with either the default mode or the user-provided value we already validated.
- Use errx() instead of fprintf(stderr) + exit().
Reported by: Moyao, Minghao Fu
MFC after: 1 week