]> git.ipfire.org Git - thirdparty/haproxy.git/commit
BUG/MEDIUM: config: Reset outline buffer size on realloc error in readcfgfile()
authorChristopher Faulet <cfaulet@haproxy.com>
Wed, 18 May 2022 14:22:43 +0000 (16:22 +0200)
committerChristopher Faulet <cfaulet@haproxy.com>
Thu, 19 May 2022 08:12:38 +0000 (10:12 +0200)
commitdfe32c7e15f1fd8d6df51b807cb2b825fd2f85a0
tree7b669b28ce6c19c59104eead846d63d9a5d649df
parent8b87b15c22ffa04b27cc49c978061956519655d1
BUG/MEDIUM: config: Reset outline buffer size on realloc error in readcfgfile()

When the line parsing failed because outline buffer must be reallocated, if
my_realloc2() call fails, the buffer size must be reset. Indeed, in this case
the current line is skipped, a fatal error is reported and we jump to the next
line. At this stage the outline buffer is NULL. If the buffer size is not reset,
the next call to parse_line() crashes because we try to write in the buffer. We
fail to detect the outline buffer is too small to copy any character.

To fix the issue, outlinesize variable must be set to 0 when outline allocation
failed.

This patch should fix the issue #1563. It must be backported as far as 2.2.
src/cfgparse.c