• Nils Goroll's avatar
    Add Session Attribute workspace overflow handling · 287dc4a6
    Nils Goroll authored
    Notes:
    
    * for the acceptor, I think it makes sense to keep AN assertion (pun!)
      because varnish is not viable if the session workspace is too small
      to even hold the attributes initialized in the acceptor.
    
      If this was an issue, we should rather revisit the minimum values for
      the session workspace
    
    * for h1 and h2 session setup, I have used XXXAN() because I am not sure
      how we should best handle allocation failures.
    
    * The relevant bit, for now, is the proxy code which may allocate
      arbitrarily long TLV attributes, so this is the code for which we now
      actually handle errors and test that we do
    
    On the vtc: I added the test to o00005.vtc because there existed a
    previous overflow test from 267504b8,
    but that only tested for the one case of a WS overflow which was already
    handled.
    
    Fixes #3145
    287dc4a6
cache_http2_session.c 11.4 KB