Module Name: src
Committed By: drochner
Date: Tue Jan 26 19:11:01 UTC 2010
Modified Files:
src/dist/dhcp/dst: dst_api.c dst_internal.h prandom.c
Log Message:
rather than adding braces in the code, define the offending macro
to "(void)0". This is what similar code in bind9 (up to 9.5) does.
(native NetBSD code usually does "do { } while (/* CONSTCOND */ 0)"
in this case)
Anyway, I've checked the code, didn't find ambiguities due to
the empty statements, so the whole thing is harmless.
To generate a diff of this commit:
cvs rdiff -u -r1.6 -r1.7 src/dist/dhcp/dst/dst_api.c
cvs rdiff -u -r1.1 -r1.2 src/dist/dhcp/dst/dst_internal.h
cvs rdiff -u -r1.5 -r1.6 src/dist/dhcp/dst/prandom.c
Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.
Modified files:
Index: src/dist/dhcp/dst/dst_api.c
diff -u src/dist/dhcp/dst/dst_api.c:1.6 src/dist/dhcp/dst/dst_api.c:1.7
--- src/dist/dhcp/dst/dst_api.c:1.6 Mon Jan 25 20:33:57 2010
+++ src/dist/dhcp/dst/dst_api.c Tue Jan 26 19:11:00 2010
@@ -1,5 +1,5 @@
#ifndef LINT
-static const char rcsid[] = "$Header: /cvsroot/src/dist/dhcp/dst/dst_api.c,v 1.6 2010/01/25 20:33:57 drochner Exp $";
+static const char rcsid[] = "$Header: /cvsroot/src/dist/dhcp/dst/dst_api.c,v 1.7 2010/01/26 19:11:00 drochner Exp $";
#endif
/*
@@ -684,10 +684,9 @@
if (key_st->dk_func->from_dns_key(key_st, &rdata[start],
len - start) > 0)
return (key_st);
- } else {
+ } else
EREPORT(("dst_dnskey_to_public_key(): unsuppored alg %d\n",
alg));
- }
SAFE_FREE(key_st);
return (key_st);
@@ -746,10 +745,9 @@
return (enc_len + loc);
else
return (-1);
- } else {
+ } else
EREPORT(("dst_key_to_dnskey(): Unsupported ALG %d\n",
key->dk_alg));
- }
return (-1);
}
@@ -879,11 +877,10 @@
EREPORT(("dst_s_read_private_key_file(): Unknown keyfile %d.%d version for %s\n",
file_major, file_minor, name));
goto fail;
- } else if (file_major > major || file_minor > minor) {
+ } else if (file_major > major || file_minor > minor)
EREPORT((
"dst_s_read_private_key_file(): Keyfile %s version higher than mine %d.%d MAY FAIL\n",
name, file_major, file_minor));
- }
while (*p++ != '\n') ; /* skip to end of line */
Index: src/dist/dhcp/dst/dst_internal.h
diff -u src/dist/dhcp/dst/dst_internal.h:1.1 src/dist/dhcp/dst/dst_internal.h:1.2
--- src/dist/dhcp/dst/dst_internal.h:1.1 Fri Aug 3 11:35:34 2001
+++ src/dist/dhcp/dst/dst_internal.h Tue Jan 26 19:11:00 2010
@@ -60,7 +60,7 @@
#ifdef REPORT_ERRORS
#define EREPORT(str) printf str
#else
-#define EREPORT(str)
+#define EREPORT(str) (void)0
#endif
/* use our own special macro to FRRE memory */
Index: src/dist/dhcp/dst/prandom.c
diff -u src/dist/dhcp/dst/prandom.c:1.5 src/dist/dhcp/dst/prandom.c:1.6
--- src/dist/dhcp/dst/prandom.c:1.5 Mon Jan 25 20:33:57 2010
+++ src/dist/dhcp/dst/prandom.c Tue Jan 26 19:11:00 2010
@@ -1,5 +1,5 @@
#ifndef LINT
-static const char rcsid[] = "$Header: /cvsroot/src/dist/dhcp/dst/prandom.c,v 1.5 2010/01/25 20:33:57 drochner Exp $";
+static const char rcsid[] = "$Header: /cvsroot/src/dist/dhcp/dst/prandom.c,v 1.6 2010/01/26 19:11:00 drochner Exp $";
#endif
/*
* Portions Copyright (c) 1995-1998 by Trusted Information Systems, Inc.
@@ -850,9 +850,8 @@
i = dst_sign_data(SIG_MODE_ALL, my_key, NULL,
(u_char *) counter, hb_size,
semi_old, sizeof(semi_old));
- if (i != hb_size) {
+ if (i != hb_size)
EREPORT(("HMAC SIGNATURE FAILURE %d\n", i));
- }
cnt++;
if (size - out < i) /* Not all data is needed */
semi_loc = i = size - out;
