tree c272c186657e273aae81a34cf3f5f837e529ddc2
parent ca9f60c1ac4c792182b28ca601e9f8897b6eeed1
author Christopher Faulet <cfaulet@haproxy.com> 1613640168 +0100
committer Christopher Faulet <cfaulet@haproxy.com> 1613656723 +0100

BUG/MINOR: sample: Always consider zero size string samples as unsafe

smp_is_safe() function is used to be sure a sample may be safely
modified. For string samples, a test is performed to verify if there is a
null-terminated byte. If not, one is added, if possible. It means if the
sample is not const and if there is some free space in the buffer, after
data. However, we must not try to read the null-terminated byte if the
string sample is too long (data >= size) or if the size is equal to
zero. This last test was not performed. Thus it was possible to consider a
string sample as safe by testing a byte outside the buffer.

Now, a zero size string sample is always considered as unsafe and is
duplicated when smp_make_safe() is called.

This patch must be backported in all stable versions.