Changeset 7511bfa in mod_gnutls


Ignore:
Timestamp:
Jun 3, 2016, 3:09:41 PM (2 years ago)
Author:
Thomas Klute <thomas2.klute@…>
Branches:
debian/master, debian/stretch-backports, master, upstream
Children:
379a297
Parents:
8913410
Message:

Use ap_log_cerror() instead of ap_log_error() where reasonable

Probably old code from when ap_log_cerror() did not exist.

Location:
src
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • src/gnutls_hooks.c

    r8913410 r7511bfa  
    619619}
    620620
    621 static int vhost_cb(void *baton, conn_rec * conn __attribute__((unused)), server_rec * s) {
     621static int vhost_cb(void *baton, conn_rec *conn, server_rec * s)
     622{
    622623    mgs_srvconf_rec *tsc;
    623624    vhost_cb_rec *x = baton;
     
    637638        ret = gnutls_x509_crt_check_hostname(tsc->certs_x509_crt_chain[0], s->server_hostname);
    638639        if (0 == ret)
    639             ap_log_error(APLOG_MARK, APLOG_DEBUG, 0, s,
    640                          "GnuTLS: the certificate doesn't match requested hostname "
    641                          "'%s'", s->server_hostname);
     640            ap_log_cerror(APLOG_MARK, APLOG_DEBUG, 0, conn,
     641                          "GnuTLS: the certificate doesn't match requested "
     642                          "hostname '%s'", s->server_hostname);
    642643    } else {
    643         ap_log_error(APLOG_MARK, APLOG_INFO, 0, s,
    644                      "GnuTLS: SNI request for '%s' but no X.509 certs available at all",
    645                      s->server_hostname);
     644        ap_log_cerror(APLOG_MARK, APLOG_INFO, 0, conn,
     645                      "GnuTLS: SNI request for '%s' but no X.509 certs "
     646                      "available at all",
     647                      s->server_hostname);
    646648    }
    647649        return check_server_aliases(x, s, tsc);
     
    671673
    672674    if (sni_type != GNUTLS_NAME_DNS) {
    673         ap_log_error(APLOG_MARK, APLOG_CRIT, 0,
    674                 ctxt->c->base_server,
    675                 "GnuTLS: Unknown type '%d' for SNI: "
    676                 "'%s'", sni_type, sni_name);
     675        ap_log_cerror(APLOG_MARK, APLOG_CRIT, 0, ctxt->c,
     676                      "GnuTLS: Unknown type '%d' for SNI: '%s'",
     677                      sni_type, sni_name);
    677678        return NULL;
    678679    }
  • src/gnutls_io.c

    r8913410 r7511bfa  
    5959    case HTTP_BAD_REQUEST:
    6060        /* log the situation */
    61         ap_log_error(APLOG_MARK, APLOG_INFO, 0,
    62                      f->c->base_server,
    63                      "GnuTLS handshake failed: HTTP spoken on HTTPS port; "
    64                      "trying to send HTML error page");
     61        ap_log_cerror(APLOG_MARK, APLOG_INFO, 0, f->c,
     62                      "GnuTLS handshake failed: HTTP spoken on HTTPS port; "
     63                      "trying to send HTML error page");
    6564        mgs_srvconf_rec *sc = (mgs_srvconf_rec *)
    6665            ap_get_module_config(f->c->base_server->module_config,
     
    417416                || ret == GNUTLS_E_FATAL_ALERT_RECEIVED) {
    418417            errcode = gnutls_alert_get(ctxt->session);
    419             ap_log_error(APLOG_MARK, APLOG_INFO, 0,
    420                     ctxt->c->base_server,
    421                     "GnuTLS: Handshake Alert (%d) '%s'.",
    422                     errcode,
    423                     gnutls_alert_get_name(errcode));
     418            ap_log_cerror(APLOG_MARK, APLOG_INFO, 0, ctxt->c,
     419                          "GnuTLS: Handshake Alert (%d) '%s'.",
     420                          errcode, gnutls_alert_get_name(errcode));
    424421        }
    425422
    426423        if (!gnutls_error_is_fatal(ret)) {
    427             ap_log_error(APLOG_MARK, APLOG_INFO, 0,
    428                     ctxt->c->base_server,
    429                     "GnuTLS: Non-Fatal Handshake Error: (%d) '%s'",
    430                     ret, gnutls_strerror(ret));
     424            ap_log_cerror(APLOG_MARK, APLOG_INFO, 0, ctxt->c,
     425                          "GnuTLS: Non-Fatal Handshake Error: (%d) '%s'",
     426                          ret, gnutls_strerror(ret));
    431427            goto tryagain;
    432428        }
     
    470466    if (rv != 0) {
    471467        /* the client did not want to rehandshake. goodbye */
    472         ap_log_error(APLOG_MARK, APLOG_WARNING, 0,
    473                 ctxt->c->base_server,
    474                 "GnuTLS: Client Refused Rehandshake request.");
     468        ap_log_cerror(APLOG_MARK, APLOG_WARNING, 0, ctxt->c,
     469                      "GnuTLS: Client Refused Rehandshake request.");
    475470        return -1;
    476471    }
     
    722717                if (ret < 0) {
    723718                    /* error sending output */
    724                     ap_log_error(APLOG_MARK,
    725                             APLOG_INFO,
    726                             ctxt->output_rc,
    727                             ctxt->c->base_server,
    728                             "GnuTLS: Error writing data."
    729                             " (%d) '%s'",
    730                             (int) ret,
    731                             gnutls_strerror(ret));
     719                    ap_log_cerror(APLOG_MARK, APLOG_INFO, ctxt->output_rc,
     720                                  ctxt->c,
     721                                  "GnuTLS: Error writing data. (%d) '%s'",
     722                                  ret, gnutls_strerror(ret));
    732723                    if (ctxt->output_rc == APR_SUCCESS) {
    733724                        ctxt->output_rc =
Note: See TracChangeset for help on using the changeset viewer.