Hi, there are 2 valid issues spotted by smatch in debug logging message:
modules/http2/h2_proxy_session.c:339 on_data_chunk_recv() error: we previously assumed 'stream' could be null (see line 338) modules/http2/h2_proxy_session.c:425 stream_data_read() error: we previously assumed 'stream' could be null (see line 424)
I'll also give a closer look at it tonight if no one fix it in the meantime. CJ Le 12/05/2016 à 23:31, [email protected] a écrit :
Author: jim Date: Thu May 12 21:31:44 2016 New Revision: 1743577 URL: http://svn.apache.org/viewvc?rev=1743577&view=rev Log: Merge r1729208, r1735668, r1735668, r1735931, r1735935, r1735942 from trunk: let proxy handler forward ALPN protocol strings for ssl proxy connections Remove leftover comment Remove leftover comment APLOGNO update for mod_proxy_http2 fix APLOGNO at wrong place, me stupid h2_proxy_session: fill in missing APLOGNO()s. Submitted by: icing, jailletc36, jailletc36, icing, icing, ylavic Reviewed/backported by: jim Added: httpd/httpd/branches/2.4.x/docs/manual/mod/mod_proxy_http2.xml httpd/httpd/branches/2.4.x/modules/http2/NWGNUproxyht2 httpd/httpd/branches/2.4.x/modules/http2/h2_proxy_session.c httpd/httpd/branches/2.4.x/modules/http2/h2_proxy_session.h httpd/httpd/branches/2.4.x/modules/http2/mod_proxy_http2.c httpd/httpd/branches/2.4.x/modules/http2/mod_proxy_http2.h
