[Bug 709] refresh_progress_meter() appends null bytes

[bugzilla-daemon at mindrot.org]

http://bugzilla.mindrot.org/show_bug.cgi?id=709

           Summary: refresh_progress_meter() appends null bytes
           Product: Portable OpenSSH
           Version: -current
          Platform: All
        OS/Version: All
            Status: NEW
          Severity: minor
          Priority: P4
         Component: scp
        AssignedTo: openssh-bugs at mindrot.org
        ReportedBy: Robert.Dahlem at siemens.com


Since 3.7.1p1 refresh_progress_meter() appends a null byte every time the
progress meter is updated. 

This is quite disturbing on terminals which skip to the next line when a null
byte is written to the last column.

It is an off-by-one-bug: win_size is the string length including terminating
null byte, so atomicio() must not write more than win_size-1 bytes to stdout.
Perhaps strlen(buf) would be better than win_size-1 ...



------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.