BUG/MINOR: buffer: fix debugging condition in b_peek_varint()

The BUG_ON_HOT() test condition added to b_peek_varint() by commit
8873b85bd ("DEBUG: buf: add BUG_ON_HOT() to most buffer management
functions") was wrong as <data> in this function is not b->data,
so that was triggering during live dumps of H2 traces on the CLI
when built with -DDEBUG_STRICT=2. No backport is needed.

diff --git a/include/haproxy/buf.h b/include/haproxy/buf.h
index 3f45ee849c0c5743bafda9b65b93193be95d4681..4ea4b73f13f1548f5a1c2cd31eef1b71379bee5a 100644 (file)
--- a/include/haproxy/buf.h
+++ b/include/haproxy/buf.h
@@ -913,7 +913,7 @@ static inline int b_peek_varint(struct buffer *b, size_t ofs, uint64_t *vptr)
        uint64_t v = 0;
        int bits = 0;
 
-       BUG_ON_HOT(ofs > data);
+       BUG_ON_HOT(ofs > b_data(b));
 
        if (data != 0 && (*head >= 0xF0)) {
                v = *head;