https://bugzilla.mindrot.org/show_bug.cgi?id=2651
--- Comment #6 from don fong <[email protected]> --- the patch(es) that i submitted have some advantages over this fix. * this fix errors out when the line length is exactly 4095 including newline. in this case, the line is not "too long" to fit in the buffer, so the error message is somewhat misleading. my patch correctly handles the case when the line exactly fits in the buffer. * when the error happens, my patch prints a more helpful error message, telling the user what the maximum line length is. * my patch has a regression test. * my patch also documents (in the man page) the fact that there is a limit on the line length. * my patch uses a symbolic constant for the maximum line length. this is a better practice than a hard-coded constant. it is also needed to tie together the code, the regression test, and the documentation. -- You are receiving this mail because: You are watching the assignee of the bug. You are watching someone on the CC list of the bug. _______________________________________________ openssh-bugs mailing list [email protected] https://lists.mindrot.org/mailman/listinfo/openssh-bugs
