Commit 84eeb1a8 authored by Stefan Fritsch's avatar Stefan Fritsch
Browse files

Fix various "variable 'x' set but not used" warnings. 


git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1125877 13f79535-47bb-0310-9956-ffa450edef68
parent abb11be4

modules/aaa/mod_auth_digest.c

+2 −4
Changes for modules/aaa/mod_auth_digest.c: 2 added lines, 4 removed lines.
Original line number Diff line number Diff line
@@ -1100,7 +1100,6 @@ static const char *gen_nonce(apr_pool_t *p, apr_time_t now, const char *opaque, 
                             const digest_config_rec *conf)

{

    char *nonce = apr_palloc(p, NONCE_LEN+1);

    int len;

    time_rec t;



    if (conf->nonce_lifetime != 0) {
@@ -1116,7 +1115,7 @@ static const char *gen_nonce(apr_pool_t *p, apr_time_t now, const char *opaque, 
        /* XXX: WHAT IS THIS CONSTANT? */

        t.time = 42;

    }

    len = apr_base64_encode_binary(nonce, t.arr, sizeof(t.arr));

    apr_base64_encode_binary(nonce, t.arr, sizeof(t.arr));

    gen_nonce_hash(nonce+NONCE_TIME_LEN, nonce, opaque, server, conf);



    return nonce;
@@ -1515,7 +1514,6 @@ static int check_nonce(request_rec *r, digest_header_rec *resp, 
                       const digest_config_rec *conf)

{

    apr_time_t dt;

    int len;

    time_rec nonce_time;

    char tmp, hash[NONCE_HASH_LEN+1];
@@ -1529,7 +1527,7 @@ static int check_nonce(request_rec *r, digest_header_rec *resp, 


    tmp = resp->nonce[NONCE_TIME_LEN];

    resp->nonce[NONCE_TIME_LEN] = '\0';

    len = apr_base64_decode_binary(nonce_time.arr, resp->nonce);

    apr_base64_decode_binary(nonce_time.arr, resp->nonce);

    gen_nonce_hash(hash, resp->nonce, resp->opaque, r->server, conf);

    resp->nonce[NONCE_TIME_LEN] = tmp;

    resp->nonce_time = nonce_time.time;

modules/dav/fs/dbm.c

+5 −1
Changes for modules/dav/fs/dbm.c: 5 added lines, 1 removed line.
Original line number Diff line number Diff line
@@ -39,6 +39,8 @@ 


#include "mod_dav.h"

#include "repos.h"

#include "http_log.h"

#include "http_main.h"      /* for ap_server_conf */





struct dav_db {
@@ -487,7 +489,9 @@ static void dav_propdb_close(dav_db *db) 
        memcpy(db->ns_table.buf, &m, sizeof(m));



        err = dav_dbm_store(db, key, value);

        /* ### what to do with the error? */

        if (err != NULL)

            ap_log_error(APLOG_MARK, APLOG_WARNING, err->aprerr,

                         ap_server_conf, "Error writing propdb: %s", err->desc);

    }



    dav_dbm_close(db);

modules/examples/mod_example_hooks.c

+0 −6
Changes for modules/examples/mod_example_hooks.c: 0 added lines, 6 removed lines.
Original line number Diff line number Diff line
@@ -1113,11 +1113,8 @@ static int x_quick_handler(request_rec *r, int lookup_uri) 
 */

static int x_pre_connection(conn_rec *c, void *csd)

{

    x_cfg *cfg;

    char *note; 



    cfg = our_cconfig(c);



    /*

     * Log the call and exit.

     */
@@ -1138,9 +1135,6 @@ static int x_pre_connection(conn_rec *c, void *csd) 
 */

static int x_process_connection(conn_rec *c)

{

    x_cfg *cfg;

    cfg = our_cconfig(c);

    

    trace_connection(c, "x_process_connection()");

    

    return DECLINED;

modules/filters/mod_include.c

+7 −4
Changes for modules/filters/mod_include.c: 7 added lines, 4 removed lines.
Original line number Diff line number Diff line
@@ -1127,7 +1127,6 @@ static int parse_expr(include_ctx_t *ctx, const char *expr, int *was_error) 
    request_rec *rr = NULL;

    const char *error = "Invalid expression \"%s\" in file %s";

    const char *parse = expr;

    int was_unmatched = 0;

    unsigned regex = 0;



    *was_error = 0;
@@ -1144,14 +1143,18 @@ static int parse_expr(include_ctx_t *ctx, const char *expr, int *was_error) 
         */

        CREATE_NODE(ctx, new);



        was_unmatched = get_ptoken(ctx, &parse, &new->token,

        {

#ifdef DEBUG_INCLUDE

            int was_unmatched =

#endif

            get_ptoken(ctx, &parse, &new->token,

                       (current != NULL ? &current->token : NULL));

        if (!parse) {

            if (!parse)

                break;

        }



            DEBUG_DUMP_UNMATCHED(ctx, was_unmatched);

            DEBUG_DUMP_TOKEN(ctx, &new->token);

        }



        if (!current) {

            switch (new->token.type) {

modules/loggers/mod_log_config.c

+4 −2
Changes for modules/loggers/mod_log_config.c: 4 added lines, 2 removed lines.
Original line number Diff line number Diff line
@@ -1114,12 +1114,14 @@ static int config_log_transaction(request_rec *r, config_log_state *cls, 
        len += strl[i] = strlen(strs[i]);

    }

    if (!log_writer) {

        ap_log_rerror(APLOG_MARK, APLOG_ERR, APR_EGENERAL, r,

        ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r,

                "log writer isn't correctly setup");

         return HTTP_INTERNAL_SERVER_ERROR;

    }

    rv = log_writer(r, cls->log_writer, strs, strl, format->nelts, len);

    /* xxx: do we return an error on log_writer? */

    if (rv != APR_SUCCESS)

        ap_log_rerror(APLOG_MARK, APLOG_WARNING, rv, r, "Error writing to %s",

                      cls->fname);

    return OK;

}