Changeset a314ec9 in mod_gnutls


Ignore:
Timestamp:
Apr 29, 2018, 6:01:12 AM (3 years ago)
Author:
Fiona Klute <fiona.klute@…>
Branches:
debian/master, master, proxy-ticket
Children:
d8d6b1e
Parents:
b94aee2
git-author:
Fiona Klute <fiona.klute@…> (04/29/18 05:56:04)
git-committer:
Fiona Klute <fiona.klute@…> (04/29/18 06:01:12)
Message:

Store config pointer in struct mgs_cache for logging

This way the store/fetch log messages will be correct with multiple
caches.

Location:
src
Files:
2 edited

Legend:

Unmodified
Added
Removed
  • src/gnutls_cache.c

    rb94aee2 ra314ec9  
    118118    if (rv != APR_SUCCESS)
    119119    {
    120         // TODO: clean up sc->cache_config reference
    121120        ap_log_error(APLOG_MARK, APLOG_DEBUG, rv, server,
    122121                     "error storing in cache '%s:%s'",
    123                      cache->prov->name, sc->cache_config);
     122                     cache->prov->name, sc->cache->config);
    124123        apr_pool_destroy(spool);
    125124        return -1;
    126125    }
    127126
    128     // TODO: clean up sc->cache_config reference
    129127    ap_log_error(APLOG_MARK, APLOG_TRACE1, rv, server,
    130128                 "stored %u bytes of data (%u byte key) in cache '%s:%s'",
    131129                 data.size, key.size,
    132                  cache->prov->name, sc->cache_config);
     130                 cache->prov->name, sc->cache->config);
    133131    apr_pool_destroy(spool);
    134132    return 0;
     
    169167// 4K should be enough for OCSP responses and sessions alike
    170168#define SOCACHE_FETCH_BUF_SIZE 4096
    171 gnutls_datum_t mgs_cache_fetch(mgs_cache_t cache __attribute__((unused)),
    172                                server_rec *server, gnutls_datum_t key,
    173                                apr_pool_t *pool)
     169gnutls_datum_t mgs_cache_fetch(mgs_cache_t cache, server_rec *server,
     170                               gnutls_datum_t key, apr_pool_t *pool)
    174171{
    175172    mgs_srvconf_rec *sc = (mgs_srvconf_rec *)
     
    196193    if (rv != APR_SUCCESS)
    197194    {
    198         // TODO: clean up sc->cache_config references
    199195        /* APR_NOTFOUND means there's no such object. */
    200196        if (rv == APR_NOTFOUND)
    201197            ap_log_error(APLOG_MARK, APLOG_TRACE1, rv, server,
    202198                         "requested entry not found in cache '%s:%s'.",
    203                          cache->prov->name, sc->cache_config);
     199                         cache->prov->name, sc->cache->config);
    204200        else
    205201            ap_log_error(APLOG_MARK, APLOG_WARNING, rv, server,
    206202                         "error fetching from cache '%s:%s'",
    207                          cache->prov->name, sc->cache_config);
     203                         cache->prov->name, sc->cache->config);
    208204        /* free unused buffer */
    209205        gnutls_free(data.data);
     
    213209    else
    214210    {
    215         // TODO: clean up sc->cache_config reference
    216211        ap_log_error(APLOG_MARK, APLOG_TRACE1, rv, server,
    217212                     "fetched %u bytes from cache '%s:%s'",
    218                      data.size, cache->prov->name, sc->cache_config);
     213                     data.size, cache->prov->name, sc->cache->config);
    219214    }
    220215    apr_pool_destroy(spool);
     
    287282                     ctxt->c->base_server,
    288283                     "error deleting from cache '%s:%s'",
    289                      ctxt->sc->cache->prov->name, ctxt->sc->cache_config);
     284                     ctxt->sc->cache->prov->name, ctxt->sc->cache->config);
    290285        return -1;
    291286    }
     
    302297    ap_log_error(APLOG_MARK, APLOG_DEBUG, APR_SUCCESS, s,
    303298                 "Cleaning up socache '%s:%s'",
    304                  sc->cache->prov->name, sc->cache_config);
     299                 sc->cache->prov->name, sc->cache->config);
    305300    sc->cache->prov->destroy(sc->cache->socache, s);
    306301    return APR_SUCCESS;
     
    342337    if (sc->cache->prov)
    343338    {
    344         /* Cache found; create it, passing anything beyond the colon. */
     339        /* Create and configure the cache instance. */
     340        sc->cache->config = sc->cache_config;
    345341        const char *err = sc->cache->prov->create(&sc->cache->socache,
    346                                                   sc->cache_config,
     342                                                  sc->cache->config,
    347343                                                  ptemp, pconf);
    348344        if (err != NULL)
     
    350346            ap_log_error(APLOG_MARK, APLOG_EMERG, APR_EGENERAL, s,
    351347                         "Creating cache '%s:%s' failed: %s",
    352                          sc->cache_type, sc->cache_config, err);
     348                         sc->cache_type, sc->cache->config, err);
    353349            return HTTP_INSUFFICIENT_STORAGE;
    354350        }
     
    363359            ap_log_error(APLOG_MARK, APLOG_EMERG, rv, s,
    364360                         "Initializing cache '%s:%s' failed!",
    365                          sc->cache_type, sc->cache_config);
     361                         sc->cache_type, sc->cache->config);
    366362            return HTTP_INSUFFICIENT_STORAGE;
    367363        }
    368364        ap_log_error(APLOG_MARK, APLOG_DEBUG, APR_SUCCESS, s,
    369365                     "%s: socache '%s:%s' created.", __func__,
    370                      sc->cache_type, sc->cache_config);
     366                     sc->cache_type, sc->cache->config);
    371367    }
    372368    else
  • src/gnutls_cache.h

    rb94aee2 ra314ec9  
    124124    /** The actual socache instance */
    125125    ap_socache_instance_t *socache;
     126    /** Cache configuration string (as passed to the socache create
     127     * function, for logging) */
     128    const char *config;
    126129    /** Mutex for cache access (used only if the cache type is not
    127130     * thread-safe) */
Note: See TracChangeset for help on using the changeset viewer.