kozorizki/haproxy
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

CLEANUP: qpack: properly use the QPACK macros not HPACK ones in debug code

Browse Source 
There were a few leftovers of DEBUG_HPACK and HPACK_SHT_SIZE instead of
their QPACK equivalent in the QPACK debug code. There's no harm anyway,
but it could lead to confusing results if the tables are not sized
equally.
This commit is contained in:
Willy Tarreau 2022-11-24 15:38:26 +01:00
parent f87bb23acb
commit 6c72fa3d18
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
src/qpack-tbl.c
View File

@ -144,7 +144,7 @@ const struct http_hdr qpack_sht[QPACK_SHT_SIZE] = {
struct pool_head *pool_head_qpack_tbl = NULL;
#ifdef DEBUG_HPACK
#ifdef DEBUG_QPACK
/* dump the whole dynamic header table */
void qpack_dht_dump(FILE *out, const struct qpack_dht *dht)
{
@ -152,8 +152,8 @@ void qpack_dht_dump(FILE *out, const struct qpack_dht *dht)
unsigned int slot;
char name[4096], value[4096];
for (i = HPACK_SHT_SIZE; i < HPACK_SHT_SIZE + dht->used; i++) {
slot = (qpack_get_dte(dht, i - HPACK_SHT_SIZE + 1) - dht->dte);
for (i = QPACK_SHT_SIZE; i < QPACK_SHT_SIZE + dht->used; i++) {
slot = (qpack_get_dte(dht, i - QPACK_SHT_SIZE + 1) - dht->dte);
fprintf(out, "idx=%u slot=%u name=<%s> value=<%s> addr=%u-%u\n",
i, slot,
istpad(name, qpack_idx_to_name(dht, i)).ptr,
@ -193,7 +193,7 @@ void qpack_dht_check_consistency(const struct qpack_dht *dht)
abort();
}
}
#endif // DEBUG_HPACK
#endif // DEBUG_QPACK
/* rebuild a new dynamic header table from <dht> with an unwrapped index and
* contents at the end. The new table is returned, the caller must not use the