diff options
author | Andy Green <andy@warmcat.com> | 2021-11-08 10:09:35 +0000 |
---|---|---|
committer | Andy Green <andy@warmcat.com> | 2021-11-08 11:05:28 +0000 |
commit | 1db45d17dfd6282ee7a88676ac01525714305cce (patch) | |
tree | d903e3b83f9de0ee64a279998dbc6c0cb1b3b5c3 | |
parent | b415f59bd27975e8e781526a8c5ee846f37fcb66 (diff) | |
download | libwebsockets-1db45d17dfd6282ee7a88676ac01525714305cce.tar.gz |
h2: coverity: check for OOM on dynamic table alloc
It can fail, we should check it.
-rw-r--r-- | lib/roles/http/server/server.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/lib/roles/http/server/server.c b/lib/roles/http/server/server.c index 150eeec5..57953cac 100644 --- a/lib/roles/http/server/server.c +++ b/lib/roles/http/server/server.c @@ -2437,8 +2437,9 @@ upgrade_h2c: lws_h2_settings(wsi, &wsi->h2.h2n->peer_set, (uint8_t *)tbuf, n); - lws_hpack_dynamic_size(wsi, (int)wsi->h2.h2n->peer_set.s[ - H2SET_HEADER_TABLE_SIZE]); + if (lws_hpack_dynamic_size(wsi, (int)wsi->h2.h2n->peer_set.s[ + H2SET_HEADER_TABLE_SIZE])) + return 1; strcpy(tbuf, "HTTP/1.1 101 Switching Protocols\x0d\x0a" "Connection: Upgrade\x0d\x0a" |