From b9ae8adc6f5b0d494d1e0b585e6f7aa6322d2e55 Mon Sep 17 00:00:00 2001 From: Arran Cudbard-Bell Date: Tue, 17 Jun 2014 14:27:43 +0100 Subject: [PATCH] Add newlines to CONSISTENCY CHECK errors --- src/lib/debug.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/src/lib/debug.c b/src/lib/debug.c index f5702a1..87875f8 100644 --- a/src/lib/debug.c +++ b/src/lib/debug.c @@ -811,14 +811,14 @@ inline void fr_verify_vp(char const *file, int line, VALUE_PAIR const *vp) if (!talloc_get_type(vp->data.ptr, uint8_t)) { fprintf(stderr, "CONSISTENCY CHECK FAILED %s[%u]: Attribute \"%s\" value buffer type should be " - "uint8_t but is %s", file, line, vp->da->name, talloc_get_name(vp->data.ptr)); + "uint8_t but is %s\n", file, line, vp->da->name, talloc_get_name(vp->data.ptr)); (void) talloc_get_type_abort(vp->data.ptr, uint8_t); } len = talloc_array_length(vp->vp_octets); if (vp->length > len) { fprintf(stderr, "CONSISTENCY CHECK FAILED %s[%u]: VALUE_PAIR length %zu does not equal uint8_t " - "value buffer length %zu", file, line, vp->length, len); + "value buffer length %zu\n", file, line, vp->length, len); fr_assert(0); fr_exit_now(1); } @@ -831,7 +831,7 @@ inline void fr_verify_vp(char const *file, int line, VALUE_PAIR const *vp) if (!talloc_get_type(vp->data.ptr, char)) { fprintf(stderr, "CONSISTENCY CHECK FAILED %s[%u]: Attribute \"%s\" value buffer type should be " - "char but is %s", file, line, vp->da->name, talloc_get_name(vp->data.ptr)); + "char but is %s\n", file, line, vp->da->name, talloc_get_name(vp->data.ptr)); (void) talloc_get_type_abort(vp->data.ptr, char); } @@ -873,7 +873,7 @@ void fr_verify_list(char const *file, int line, TALLOC_CTX *expected, VALUE_PAIR parent = talloc_parent(vp); if (expected && (parent != expected)) { fprintf(stderr, "CONSISTENCY CHECK FAILED %s[%u]: Expected VALUE_PAIR (%s) to be parented " - "by %p (%s), but parented by %p (%s)", + "by %p (%s), but parented by %p (%s)\n", file, line, vp->da->name, expected, talloc_get_name(expected), parent, parent ? talloc_get_name(parent) : "NULL"); -- 2.1.4