From d5cbbe83973b8638210a6a90f250dfc0154898b3 Mon Sep 17 00:00:00 2001 From: Daniel Stenberg Date: Sat, 22 Sep 2018 22:53:05 +0200 Subject: unit1650: fix out of boundary access Fixes #2987 Closes #3035 --- tests/unit/unit1650.c | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/tests/unit/unit1650.c b/tests/unit/unit1650.c index a6278dd3c..b76936a97 100644 --- a/tests/unit/unit1650.c +++ b/tests/unit/unit1650.c @@ -268,12 +268,12 @@ UNITTEST_START rc = doh_decode((unsigned char *)full49, sizeof(full49)-1, DNS_TYPE_A, &d); fail_if(d.numaddr != 1, "missing address"); - a = &d.addr[i]; - snprintf((char *)buffer, sizeof(buffer), "%d.%d.%d.%d\n", + a = &d.addr[0]; + snprintf((char *)buffer, sizeof(buffer), "%d.%d.%d.%d", a->ip.v4 & 0xff, (a->ip.v4>>8) & 0xff, (a->ip.v4>>16) & 0xff, a->ip.v4 >>24); - if(rc && strcmp((char *)buffer, "127.0.0.1")) { - fprintf(stderr, "bad address decoded\n"); + if(rc || strcmp((char *)buffer, "127.0.0.1")) { + fprintf(stderr, "bad address decoded: %s, rc == %d\n", buffer, rc); return 7; } fail_if(d.numcname, "bad cname counter"); -- cgit v1.2.1