Add newlines to CONSISTENCY CHECK errors
authorArran Cudbard-Bell <a.cudbardb@freeradius.org>
Tue, 17 Jun 2014 13:27:43 +0000 (14:27 +0100)
committerArran Cudbard-Bell <a.cudbardb@freeradius.org>
Tue, 17 Jun 2014 13:27:43 +0000 (14:27 +0100)
src/lib/debug.c

index f5702a1..87875f8 100644 (file)
@@ -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");