Changeset c7710cf in mod_gnutls


Ignore:
Timestamp:
Dec 19, 2018, 1:09:13 PM (5 months ago)
Author:
Fiona Klute <fiona.klute@…>
Branches:
debian/master, master
Children:
08c48d7
Parents:
564f33f
Message:

Immediately return from proxy_conn_set_alpn() if there is no proxy note

This is easier to read than wrapping the rest of the function in an
"if" block with a negative check for NULL.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • src/gnutls_proxy.c

    r564f33f rc7710cf  
    323323    const char *proxy_alpn =
    324324        apr_table_get(ctxt->c->notes, PROXY_ALPN_NOTE);
    325     if (proxy_alpn != NULL)
    326     {
    327         // TODO: mod_ssl ssl_engine_io.c does some tokenization of
    328         // the input string, so it looks like the API allows
    329         // multiple protocols.
    330         gnutls_datum_t alpn_proto = {
    331             .data = (unsigned char *) apr_pstrdup(ctxt->c->pool, proxy_alpn),
    332             .size = strlen(proxy_alpn)
    333         };
    334         ap_log_cerror(APLOG_MARK, APLOG_DEBUG, APR_SUCCESS, ctxt->c,
    335                       "%s: proxy module requests ALPN proto '%s', "
    336                       "length %" APR_SIZE_T_FMT ".",
    337                       __func__, proxy_alpn, strlen(proxy_alpn));
    338         int ret = gnutls_alpn_set_protocols(ctxt->session,
    339                                             &alpn_proto,
    340                                             1 /* number of proposals */,
    341                                             0 /* flags */);
    342         if (ret != GNUTLS_E_SUCCESS)
    343             ap_log_cerror(APLOG_MARK, APLOG_ERR, ret, ctxt->c,
    344                           "Could not set ALPN proposal '%s' for proxy "
    345                           "connection: %s (%d)",
    346                           proxy_alpn, gnutls_strerror(ret), ret);
    347     }
     325    if (proxy_alpn == NULL)
     326        return;
     327
     328    // TODO: mod_ssl ssl_engine_io.c does some tokenization of
     329    // the input string, so it looks like the API allows
     330    // multiple protocols.
     331    gnutls_datum_t alpn_proto = {
     332        .data = (unsigned char *) apr_pstrdup(ctxt->c->pool, proxy_alpn),
     333        .size = strlen(proxy_alpn)
     334    };
     335    ap_log_cerror(APLOG_MARK, APLOG_DEBUG, APR_SUCCESS, ctxt->c,
     336                  "%s: proxy module requests ALPN proto '%s', "
     337                  "length %" APR_SIZE_T_FMT ".",
     338                  __func__, proxy_alpn, strlen(proxy_alpn));
     339    int ret = gnutls_alpn_set_protocols(ctxt->session,
     340                                        &alpn_proto,
     341                                        1 /* number of proposals */,
     342                                        0 /* flags */);
     343    if (ret != GNUTLS_E_SUCCESS)
     344        ap_log_cerror(APLOG_MARK, APLOG_ERR, ret, ctxt->c,
     345                      "Could not set ALPN proposal '%s' for proxy "
     346                      "connection: %s (%d)",
     347                      proxy_alpn, gnutls_strerror(ret), ret);
    348348}
    349349
Note: See TracChangeset for help on using the changeset viewer.