]> git.ipfire.org Git - thirdparty/libarchive.git/commit
Issue #767: Buffer overflow printing a filename
authorTim Kientzle <kientzle@acm.org>
Sun, 21 Aug 2016 17:51:43 +0000 (10:51 -0700)
committerTim Kientzle <kientzle@acm.org>
Sun, 21 Aug 2016 17:57:20 +0000 (10:57 -0700)
commite37b620fe8f14535d737e89a4dcabaed4517bf1a
treea646d0875d64e41fcc7c169b04b36854c4c1adf7
parent36bb164e221a3a76488f2ceaf808db14de6f8ca4
Issue #767:  Buffer overflow printing a filename

The safe_fprintf function attempts to ensure clean output for an
arbitrary sequence of bytes by doing a trial conversion of the
multibyte characters to wide characters -- if the resulting wide
character is printable then we pass through the corresponding bytes
unaltered, otherwise, we convert them to C-style ASCII escapes.

The stack trace in Issue #767 suggest that the 20-byte buffer
was getting overflowed trying to format a non-printable multibyte
character.  This should only happen if there is a valid multibyte
character of more than 5 bytes that was unprintable.  (Each byte
would get expanded to a four-charcter octal-style escape of the form
"\123" resulting in >20 characters for the >5 byte multibyte character.)

I've not been able to reproduce this, but have expanded the conversion
buffer to 128 bytes on the belief that no multibyte character set
has a single character of more than 32 bytes.
tar/util.c