Initialize local value boxes for some coercion results (CIDs below)

CIDs: #1503917, #1503948, #1503o5o, #1503989

These are in calc_{string, octet}(), where coercion results stored
in locals must be cleared before returning. Currently coverity claims
that the value box's type is not set. One would think that modeling
fr_value_box_cast() to say it writes the destination on success would
suffice, but that might not be the case, so we will initialize the
value boxes just as we have done with some local buffers used in
sbuffs.

diff --git a/src/lib/util/calc.c b/src/lib/util/calc.c
index 8cec6ca9386573c0675d4876e984b29b1c6731d8..7861e9be3915ea99cc5c1c9754e0dbb17b7378ed 100644 (file)
--- a/src/lib/util/calc.c
+++ b/src/lib/util/calc.c
@@ -845,7 +845,8 @@ static int calc_octets(TALLOC_CTX *ctx, fr_value_box_t *dst, fr_value_box_t cons
 {
        uint8_t *buf;
        size_t len;
-       fr_value_box_t one, two;
+       fr_value_box_t one = (fr_value_box_t){};
+       fr_value_box_t two = (fr_value_box_t){};
 
        fr_assert(dst->type == FR_TYPE_OCTETS);
 
@@ -983,7 +984,8 @@ static int calc_string(TALLOC_CTX *ctx, fr_value_box_t *dst, fr_value_box_t cons
 {
        char *buf;
        size_t len;
-       fr_value_box_t one, two;
+       fr_value_box_t one = (fr_value_box_t){};
+       fr_value_box_t two = (fr_value_box_t){};
 
        fr_assert(dst->type == FR_TYPE_STRING);
 
@@ -1085,9 +1087,7 @@ static int calc_string(TALLOC_CTX *ctx, fr_value_box_t *dst, fr_value_box_t cons
                return ERR_INVALID;     /* invalid operator */
        }
 
-       /* coverity[uninit_use_in_call] */
        if (a == &one) fr_value_box_clear_value(&one);
-       /* coverity[uninit_use_in_call] */
        if (b == &two) fr_value_box_clear_value(&two);
 
        return 0;