| commit | 18795d48a9bb09aedc57e547029828a56322e49d | [log] [tgz] |
|---|---|---|
| author | Tim Duesterhus <tim@bastelstu.be> | Sun Aug 29 00:58:22 2021 +0200 |
| committer | Willy Tarreau <w@1wt.eu> | Mon Aug 30 06:14:50 2021 +0200 |
| tree | d336a1751528bc2bed5b8fce8ca40af2278d8cbf | |
| parent | 1f269c12dc31bb63db31559cb44c187ab91abb64 [diff] |
BUG/MINOR: tools: Fix loop condition in dump_text() The condition should first check whether `bsize` is reached, before dereferencing the offset. Even if this always works fine, due to the string being null-terminated, this certainly looks odd. Found using GitHub's CodeQL scan. This bug traces back to at least 97c2ae13bc0d7961a348102d6719fbcaf34d46d5 (1.7.0+) and this patch should be backported accordingly.