]> git.ipfire.org Git - thirdparty/haproxy.git/commitdiff
BUILD: http: stop gcc-4.1.2 from complaining about possibly uninitialized values
authorWilly Tarreau <w@1wt.eu>
Wed, 11 Apr 2012 15:28:40 +0000 (17:28 +0200)
committerWilly Tarreau <w@1wt.eu>
Sun, 29 Apr 2012 22:19:31 +0000 (00:19 +0200)
The three warnings below are totally wrong since the variables depend on another
one which is only turned on when the variables are initialized. Still this gcc-4.1.2
isn't able to see this and prefers to complain wrongly. So let's initialize the
variables to shut it up since we're not in the fast path.

src/proto_http.c: In function 'acl_fetch_any_cookie_cnt':
src/proto_http.c:8393: warning: 'val_end' may be used uninitialized in this function

src/proto_http.c: In function 'http_process_req_stat_post':
src/proto_http.c:2577: warning: 'st_next_param' may be used uninitialized in this function
src/proto_http.c:2577: warning: 'st_cur_param' may be used uninitialized in this function

src/proto_http.c

index 98386aaa68f9a4ac58e13686ce33582424ec3c36..ae300473eb25da4f06ee9e84ba8ece18889ab87a 100644 (file)
@@ -2574,7 +2574,8 @@ int http_process_req_stat_post(struct stream_interface *si, struct http_txn *txn
        int altered_servers = 0;
 
        char *first_param, *cur_param, *next_param, *end_params;
-       char *st_cur_param, *st_next_param;
+       char *st_cur_param = NULL;
+       char *st_next_param = NULL;
 
        first_param = req->data + txn->req.eoh + 2;
        end_params  = first_param + txn->req.body_len;
@@ -8395,7 +8396,7 @@ acl_fetch_any_cookie_cnt(struct proxy *px, struct session *l4, void *l7, char *s
        if (!txn)
                return 0;
 
-       val_beg = NULL;
+       val_end = val_beg = NULL;
        ctx.idx = 0;
        cnt = 0;