Skip to content

Commit f6a8eb6

Browse files
committed
Fixes for clang statix code analysis failures
1 parent 256cf6d commit f6a8eb6

File tree

3 files changed

+11
-9
lines changed

3 files changed

+11
-9
lines changed

rr.c

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -405,8 +405,8 @@ ldns_rr_new_frm_str_internal(ldns_rr **newrr, const char *str,
405405
}
406406

407407
pre_data_pos = ldns_buffer_position(rd_buf);
408-
if (-1 == (c = ldns_bget_token(
409-
rd_buf, rd, delimiters, LDNS_MAX_RDFLEN))) {
408+
if (-1 == ldns_bget_token(
409+
rd_buf, rd, delimiters, LDNS_MAX_RDFLEN)) {
410410

411411
done = true;
412412
(void)done; /* we're breaking, so done not read anymore */

sha1.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -97,7 +97,7 @@ ldns_sha1_transform(uint32_t state[5], const unsigned char buffer[LDNS_SHA1_BLOC
9797
state[4] += e;
9898
/* Wipe variables */
9999
a = b = c = d = e = 0;
100-
(void)a;
100+
(void)a; (void)b; (void)c; (void)d; (void)e;
101101
}
102102

103103

sha2.c

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -494,7 +494,8 @@ static void ldns_sha256_Transform(ldns_sha256_CTX* context,
494494

495495
/* Clean up */
496496
a = b = c = d = e = f = g = h = T1 = T2 = 0;
497-
(void)a;
497+
(void)a; (void)b; (void)c; (void)d; (void)e; (void)f; (void)g; (void)h;
498+
(void)T1; (void)T2;
498499
}
499500

500501
#endif /* SHA2_UNROLL_TRANSFORM */
@@ -528,7 +529,7 @@ void ldns_sha256_update(ldns_sha256_CTX* context, const sha2_byte *data, size_t
528529
context->bitcount += len << 3;
529530
/* Clean up: */
530531
usedspace = freespace = 0;
531-
(void)usedspace;
532+
(void)usedspace; (void)freespace;
532533
return;
533534
}
534535
}
@@ -546,7 +547,7 @@ void ldns_sha256_update(ldns_sha256_CTX* context, const sha2_byte *data, size_t
546547
}
547548
/* Clean up: */
548549
usedspace = freespace = 0;
549-
(void)usedspace;
550+
(void)usedspace; (void)freespace;
550551
}
551552

552553
typedef union _ldns_sha2_buffer_union {
@@ -809,7 +810,8 @@ static void ldns_sha512_Transform(ldns_sha512_CTX* context,
809810

810811
/* Clean up */
811812
a = b = c = d = e = f = g = h = T1 = T2 = 0;
812-
(void)a;
813+
(void)a; (void)b; (void)c; (void)d; (void)e; (void)f; (void)g; (void)h;
814+
(void)T1; (void)T2;
813815
}
814816

815817
#endif /* SHA2_UNROLL_TRANSFORM */
@@ -843,7 +845,7 @@ void ldns_sha512_update(ldns_sha512_CTX* context, const sha2_byte *data, size_t
843845
ADDINC128(context->bitcount, len << 3);
844846
/* Clean up: */
845847
usedspace = freespace = 0;
846-
(void)usedspace;
848+
(void)usedspace; (void)freespace;
847849
return;
848850
}
849851
}
@@ -861,7 +863,7 @@ void ldns_sha512_update(ldns_sha512_CTX* context, const sha2_byte *data, size_t
861863
}
862864
/* Clean up: */
863865
usedspace = freespace = 0;
864-
(void)usedspace;
866+
(void)usedspace; (void)freespace;
865867
}
866868

867869
static void ldns_sha512_Last(ldns_sha512_CTX* context) {

0 commit comments

Comments
 (0)