From 6941ebddf191e5469564041eef18835d645b3796 Mon Sep 17 00:00:00 2001 From: "W.C.A. Wijngaards" Date: Tue, 4 Feb 2014 11:20:33 +0100 Subject: [PATCH] Remove warnings about sprintf and strcat tests. --- src/test/check_getdns_dict_get_names.h | 7 ++- src/test/tests_dict.c | 54 ++++++++++----------- src/test/tests_list.c | 66 +++++++++++++------------- 3 files changed, 66 insertions(+), 61 deletions(-) diff --git a/src/test/check_getdns_dict_get_names.h b/src/test/check_getdns_dict_get_names.h index 6b7c0fcb..8074cdac 100644 --- a/src/test/check_getdns_dict_get_names.h +++ b/src/test/check_getdns_dict_get_names.h @@ -78,7 +78,12 @@ { ASSERT_RC(getdns_list_get_bindata(answer, i, &key), GETDNS_RETURN_GOOD, "Return code from getdns_list_get_bindata()"); - strcat(string_buffer, (char *)key->data); + /* concatenate strings, if enough buffer space */ + if(strlen(string_buffer) + strlen((char*)key->data) + 1 + < sizeof(string_buffer)) { + memmove(string_buffer+strlen(string_buffer), key->data, + strlen((char*)key->data)+1); + } } ck_assert_msg(strcmp(string_buffer, "elevententwelve") == 0, diff --git a/src/test/tests_dict.c b/src/test/tests_dict.c index 1e8de622..0ae162c5 100644 --- a/src/test/tests_dict.c +++ b/src/test/tests_dict.c @@ -70,25 +70,25 @@ tst_bindatasetget(void) tstmsg_case_msg("getdns_dict_get_bindata() empty dict"); retval = getdns_dict_get_bindata(NULL, key, &ans_bdata); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 1: getdns_dict_get_bindata(NULL, key, &ans_bdata),retval = %d", retval); tstmsg_case_msg(msg); retval = getdns_dict_get_bindata(dict, key, NULL); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 2: getdns_dict_get_bindata(dict, key, NULL),retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_bindata(dict, NULL, &ans_bindata)"); retval = getdns_dict_get_bindata(dict, NULL, &ans_bdata); - sprintf(msg, "test 3: getdns_dict_get_bindata,retval = %d", retval); + snprintf(msg, sizeof(msg), "test 3: getdns_dict_get_bindata,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_bindata(dict, key, &ans_bdata)"); retval = getdns_dict_get_bindata(dict, key, &ans_bdata); - sprintf(msg, "test 4: getdns_list_get_bindata,retval = %d", retval); + snprintf(msg, sizeof(msg), "test 4: getdns_list_get_bindata,retval = %d", retval); tstmsg_case_msg(msg); getdns_dict_destroy(dict); @@ -107,13 +107,13 @@ tst_bindatasetget(void) tstmsg_case_msg("getdns_dict_set_bindata(dict, key, bindata)"); retval = getdns_dict_set_bindata(dict, key, bindata); - sprintf(msg, "test 5: getdns_dict_set_bindata,retval=%d,key=%s", + snprintf(msg, sizeof(msg), "test 5: getdns_dict_set_bindata,retval=%d,key=%s", retval, key); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_bindata(dict, key, &ans_bdata)"); retval = getdns_dict_get_bindata(dict, key, &ans_bdata); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 6: getdns_dict_get_bindata,retval=%d,key=%s,data=%s", retval, key, ans_bdata->data); tstmsg_case_msg(msg); @@ -153,25 +153,25 @@ tst_dictsetget(void) tstmsg_case_msg("getdns_dict_get_dict() empty dict"); retval = getdns_dict_get_dict(NULL, key, &ansdict); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 7: getdns_dict_get_dict(NULL, key, &ansdict),retval = %d", retval); tstmsg_case_msg(msg); retval = getdns_dict_get_dict(dict, key, NULL); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 8: getdns_dict_get_dict(dict, key, NULL),retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_dict(dict, NULL, &ansdict)"); retval = getdns_dict_get_dict(dict, NULL, &ansdict); - sprintf(msg, "test 9: getdns_dict_get_dict,retval = %d", retval); + snprintf(msg, sizeof(msg), "test 9: getdns_dict_get_dict,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_dict(dict, key, &ansdict)"); retval = getdns_dict_get_dict(dict, key, &ansdict); - sprintf(msg, "test 10: getdns_list_get_dict,retval = %d", retval); + snprintf(msg, sizeof(msg), "test 10: getdns_list_get_dict,retval = %d", retval); tstmsg_case_msg(msg); getdns_dict_destroy(dict); @@ -189,7 +189,7 @@ tst_dictsetget(void) tstmsg_case_msg("getdns_dict_set_dict(dict, key, newdict)"); retval = getdns_dict_set_dict(dict, key, newdict); - sprintf(msg, "test 11: getdns_dict_set_dict,retval=%d,key=%s", retval, + snprintf(msg, sizeof(msg), "test 11: getdns_dict_set_dict,retval=%d,key=%s", retval, key); tstmsg_case_msg(msg); getdns_dict_destroy(newdict); @@ -198,7 +198,7 @@ tst_dictsetget(void) retval = getdns_dict_get_dict(dict, key, &ansdict); getdns_dict_get_int(ansdict, "foo", &int1); getdns_dict_get_int(ansdict, "bar", &int2); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 12: getdns_dict_get_dict,retval=%d,key=%s,int1=%d,int2=%d", retval, key, int1, int2); tstmsg_case_msg(msg); @@ -327,25 +327,25 @@ tst_listsetget(void) tstmsg_case_msg("getdns_dict_get_list() empty dict"); retval = getdns_dict_get_list(NULL, key, &anslist); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 13: getdns_dict_get_list(NULL, key, &anslist),retval = %d", retval); tstmsg_case_msg(msg); retval = getdns_dict_get_list(dict, key, NULL); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 14: getdns_dict_get_list(dict, key, NULL),retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_list(dict, NULL, &anslist)"); retval = getdns_dict_get_list(dict, NULL, &anslist); - sprintf(msg, "test 15: getdns_dict_get_list,retval = %d", retval); + snprintf(msg, sizeof(msg), "test 15: getdns_dict_get_list,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_list(dict, key, &anslist)"); retval = getdns_dict_get_list(dict, key, &anslist); - sprintf(msg, "test 16: getdns_list_get_list,retval = %d", retval); + snprintf(msg, sizeof(msg), "test 16: getdns_list_get_list,retval = %d", retval); tstmsg_case_msg(msg); getdns_dict_destroy(dict); @@ -365,7 +365,7 @@ tst_listsetget(void) tstmsg_case_msg("getdns_dict_set_list(dict, key, newlist)"); retval = getdns_dict_set_list(dict, key, newlist); - sprintf(msg, "test 17: getdns_dict_set_list,retval=%d,key=%s", + snprintf(msg, sizeof(msg), "test 17: getdns_dict_set_list,retval=%d,key=%s", retval, key); tstmsg_case_msg(msg); getdns_list_destroy(newlist); @@ -374,7 +374,7 @@ tst_listsetget(void) retval = getdns_dict_get_list(dict, key, &anslist); getdns_list_get_int(anslist, 0, &int1); getdns_list_get_int(anslist, 1, &int2); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 18: getdns_dict_get_list,retval=%d,key=%s,int1=%d,int2=%d", retval, key, int1, int2); tstmsg_case_msg(msg); @@ -411,25 +411,25 @@ tst_intsetget(void) tstmsg_case_msg("getdns_dict_get_int() empty dict"); retval = getdns_dict_get_int(NULL, key, &ans_int); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 19: getdns_dict_get_int(NULL, key, &ans_int),retval = %d", retval); tstmsg_case_msg(msg); retval = getdns_dict_get_int(dict, key, NULL); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 20: getdns_dict_get_int(dict, key, NULL),retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_int(dict, NULL, &ans_int)"); retval = getdns_dict_get_int(dict, NULL, &ans_int); - sprintf(msg, "test 21: getdns_dict_get_int,retval = %d", retval); + snprintf(msg, sizeof(msg), "test 21: getdns_dict_get_int,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_int(dict, key, &ans_int)"); retval = getdns_dict_get_int(dict, key, &ans_int); - sprintf(msg, "test 22: getdns_list_get_int,retval = %d", retval); + snprintf(msg, sizeof(msg), "test 22: getdns_list_get_int,retval = %d", retval); tstmsg_case_msg(msg); getdns_dict_destroy(dict); @@ -445,13 +445,13 @@ tst_intsetget(void) tstmsg_case_msg("getdns_dict_set_int(dict, key, newint)"); retval = getdns_dict_set_int(dict, key, newint); - sprintf(msg, "test 23: getdns_dict_set_int,retval=%d,key=%s,int=%d", + snprintf(msg, sizeof(msg), "test 23: getdns_dict_set_int,retval=%d,key=%s,int=%d", retval, key, newint); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_int(dict, key, &ans_int)"); retval = getdns_dict_get_int(dict, key, &ans_int); - sprintf(msg, "test 24: getdns_dict_get_int,retval=%d,key=%s,int=%d", + snprintf(msg, sizeof(msg), "test 24: getdns_dict_get_int,retval=%d,key=%s,int=%d", retval, key, ans_int); tstmsg_case_msg(msg); @@ -459,19 +459,19 @@ tst_intsetget(void) newint = 52; tstmsg_case_msg("getdns_dict_set_int(dict, key, newint)"); retval = getdns_dict_set_int(dict, key, newint); - sprintf(msg, "test 25: getdns_dict_set_int,retval=%d,key=%s,int=%d", + snprintf(msg, sizeof(msg), "test 25: getdns_dict_set_int,retval=%d,key=%s,int=%d", retval, key, newint); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_int(dict, key, &ans_int)"); retval = getdns_dict_get_int(dict, key, &ans_int); - sprintf(msg, "test 26: getdns_dict_get_int,retval=%d,key=%s,int=%d", + snprintf(msg, sizeof(msg), "test 26: getdns_dict_get_int,retval=%d,key=%s,int=%d", retval, key, ans_int); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_dict_get_data_type(dict, key, &dtype)"); retval = getdns_dict_get_data_type(dict, key, &dtype); - sprintf(msg, + snprintf(msg, sizeof(msg), "test 27: getdns_dict_get_data_type,retval=%d,key=%s,dtype=%d", retval, key, dtype); tstmsg_case_msg(msg); diff --git a/src/test/tests_list.c b/src/test/tests_list.c index ea65c67b..9177c692 100644 --- a/src/test/tests_list.c +++ b/src/test/tests_list.c @@ -67,36 +67,36 @@ tst_bindatasetget(void) tstmsg_case_msg("getdns_list_get_bindata() empty list"); retval = getdns_list_get_bindata(NULL, index, &ans_bindata); - sprintf(msg, + snprintf(msg, sizeof(msg), "getdns_list_get_bindata(NULL, index, &ans_bindata),retval = %d", retval); tstmsg_case_msg(msg); retval = getdns_list_get_bindata(list, index, NULL); - sprintf(msg, "getdns_list_get_bindata(list, index, NULL),retval = %d", + snprintf(msg, sizeof(msg), "getdns_list_get_bindata(list, index, NULL),retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_get_bindata(list, 0, &ans_bindata)"); retval = getdns_list_get_bindata(list, 0, &ans_bindata); - sprintf(msg, "getdns_list_get_bindata,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_get_bindata,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_get_bindata(list, 1, &ans_bindata)"); retval = getdns_list_get_bindata(list, 1, &ans_bindata); - sprintf(msg, "getdns_list_get_bindata,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_get_bindata,retval = %d", retval); tstmsg_case_msg(msg); /* test set function against empty list with bogus params */ tstmsg_case_msg("getdns_list_set_bindata(list, -1, ans_bindata)"); retval = getdns_list_set_bindata(list, -1, NULL); - sprintf(msg, "getdns_list_set_bindata,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_set_bindata,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_set_bindata(list, 1, ans_bindata)"); retval = getdns_list_set_bindata(list, 1, NULL); - sprintf(msg, "getdns_list_set_bindata,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_set_bindata,retval = %d", retval); tstmsg_case_msg(msg); /* test set and get legitimate use case */ @@ -108,7 +108,7 @@ tst_bindatasetget(void) getdns_list_set_bindata(list, index, new_bindata); retval = getdns_list_get_bindata(list, index, &ans_bindata); - sprintf(msg, + snprintf(msg, sizeof(msg), "getdns_list_set/get_bindata,retval = %d, bindata->data = %d,%s", retval, (int) ans_bindata->size, (char *) ans_bindata->data); tstmsg_case_msg(msg); @@ -144,35 +144,35 @@ tst_dictsetget(void) tstmsg_case_msg("getdns_list_get_dict() empty list"); retval = getdns_list_get_dict(NULL, index, &dict); - sprintf(msg, "getdns_list_get_dict(NULL, index, &dict),retval = %d", + snprintf(msg, sizeof(msg), "getdns_list_get_dict(NULL, index, &dict),retval = %d", retval); tstmsg_case_msg(msg); retval = getdns_list_get_dict(list, index, NULL); - sprintf(msg, "getdns_list_get_dict(list, index, NULL),retval = %d", + snprintf(msg, sizeof(msg), "getdns_list_get_dict(list, index, NULL),retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_get_dict(list, 0, &dict)"); retval = getdns_list_get_dict(list, 0, &dict); - sprintf(msg, "getdns_list_get_dict,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_get_dict,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_get_dict(list, 1, &dict)"); retval = getdns_list_get_dict(list, 1, &dict); - sprintf(msg, "getdns_list_get_dict,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_get_dict,retval = %d", retval); tstmsg_case_msg(msg); /* test int set function against empty list with bogus params */ tstmsg_case_msg("getdns_list_set_dict(list, 0, dict)"); retval = getdns_list_set_dict(list, -1, dict); - sprintf(msg, "getdns_list_set_dict,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_set_dict,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_set_dict(list, 1, dict)"); retval = getdns_list_set_dict(list, 1, dict); - sprintf(msg, "getdns_list_set_dict,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_set_dict,retval = %d", retval); tstmsg_case_msg(msg); /* test set and get legitimate use case */ @@ -181,7 +181,7 @@ tst_dictsetget(void) getdns_list_set_dict(list, index, dict); retval = getdns_list_get_dict(list, index, &ansdict); getdns_dict_get_int(ansdict, "foo", &ans_int); - sprintf(msg, "getdns_list_set/get_dict,retval=%d, ans=%d", retval, + snprintf(msg, sizeof(msg), "getdns_list_set/get_dict,retval=%d, ans=%d", retval, ans_int); tstmsg_case_msg(msg); @@ -216,36 +216,36 @@ tst_listsetget(void) tstmsg_case_msg("getdns_list_get_list() empty list"); retval = getdns_list_get_list(NULL, index, &ans_list); - sprintf(msg, + snprintf(msg, sizeof(msg), "getdns_list_get_list(NULL, index, &ans_list),retval = %d", retval); tstmsg_case_msg(msg); retval = getdns_list_get_list(list, index, NULL); - sprintf(msg, "getdns_list_get_list(list, index, NULL),retval = %d", + snprintf(msg, sizeof(msg), "getdns_list_get_list(list, index, NULL),retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_get_list(list, 0, &ans_list)"); retval = getdns_list_get_list(list, 0, &ans_list); - sprintf(msg, "getdns_list_get_list,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_get_list,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_get_list(list, 1, &ans_list)"); retval = getdns_list_get_list(list, 1, &ans_list); - sprintf(msg, "getdns_list_get_list,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_get_list,retval = %d", retval); tstmsg_case_msg(msg); /* test set function against empty list with bogus params */ tstmsg_case_msg("getdns_list_set_list(list, -1, ans_list)"); retval = getdns_list_set_list(list, -1, NULL); - sprintf(msg, "getdns_list_set_list,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_set_list,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_set_list(list, 1, ans_list)"); retval = getdns_list_set_list(list, 1, NULL); - sprintf(msg, "getdns_list_set_list,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_set_list,retval = %d", retval); tstmsg_case_msg(msg); /* test set and get legitimate use case */ @@ -256,7 +256,7 @@ tst_listsetget(void) getdns_list_set_list(list, index, new_list); retval = getdns_list_get_list(list, index, &ans_list); getdns_list_get_int(ans_list, 0, &ans_int); - sprintf(msg, "getdns_list_set/get_list,retval = %d, ans[0] = %d", + snprintf(msg, sizeof(msg), "getdns_list_set/get_list,retval = %d, ans[0] = %d", retval, ans_int); tstmsg_case_msg(msg); @@ -289,42 +289,42 @@ tst_intsetget(void) tstmsg_case_msg("getdns_list_get_int() empty list"); retval = getdns_list_get_int(NULL, index, &ans_int); - sprintf(msg, "getdns_list_get_int(NULL, index, &ans_int),retval = %d", + snprintf(msg, sizeof(msg), "getdns_list_get_int(NULL, index, &ans_int),retval = %d", retval); tstmsg_case_msg(msg); retval = getdns_list_get_int(list, index, NULL); - sprintf(msg, "getdns_list_get_int(list, index, NULL),retval = %d", + snprintf(msg, sizeof(msg), "getdns_list_get_int(list, index, NULL),retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_get_int(list, 0, &ans_int)"); retval = getdns_list_get_int(list, 0, &ans_int); - sprintf(msg, "getdns_list_get_int,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_get_int,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_get_int(list, 1, &ans_int)"); retval = getdns_list_get_int(list, 1, &ans_int); - sprintf(msg, "getdns_list_get_int,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_get_int,retval = %d", retval); tstmsg_case_msg(msg); /* test int set function against empty list with bogus params */ tstmsg_case_msg("getdns_list_set_int(list, -1, ans_int)"); retval = getdns_list_set_int(list, -1, ans_int); - sprintf(msg, "getdns_list_set_int,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_set_int,retval = %d", retval); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_set_int(list, 1, ans_int)"); retval = getdns_list_set_int(list, 1, ans_int); - sprintf(msg, "getdns_list_set_int,retval = %d", retval); + snprintf(msg, sizeof(msg), "getdns_list_set_int,retval = %d", retval); tstmsg_case_msg(msg); /* test set and get legitimate use case */ getdns_list_set_int(list, index, 42); retval = getdns_list_get_int(list, index, &ans_int); - sprintf(msg, "getdns_list_set/get_int,retval = %d, ans = %d", retval, + snprintf(msg, sizeof(msg), "getdns_list_set/get_int,retval = %d, ans = %d", retval, ans_int); tstmsg_case_msg(msg); @@ -370,7 +370,7 @@ tst_create(void) for (i = 0; i < GETDNS_LIST_BLOCKSZ + 2; i++) { retval = getdns_list_set_int(list, i, i); if (retval != GETDNS_RETURN_GOOD) { - sprintf(msg, "getdns_list_set_int,i=%d,retval = %d", + snprintf(msg, sizeof(msg), "getdns_list_set_int,i=%d,retval = %d", i, retval); tstmsg_case_msg(msg); } @@ -378,20 +378,20 @@ tst_create(void) tstmsg_case_msg("getdns_list_get_length(list)"); retval = getdns_list_get_length(list, &index); - sprintf(msg, "list length = %d", (int) index); + snprintf(msg, sizeof(msg), "list length = %d", (int) index); tstmsg_case_msg(msg); tstmsg_case_msg("getdns_list_get_length()"); retval = getdns_list_get_length(NULL, &index); - sprintf(msg, "NUll, %i, retval = %d", (int)index, retval); + snprintf(msg, sizeof(msg), "NUll, %i, retval = %d", (int)index, retval); tstmsg_case_msg(msg); retval = getdns_list_get_length(NULL, NULL); - sprintf(msg, "NUll, NULL, retval = %d", retval); + snprintf(msg, sizeof(msg), "NUll, NULL, retval = %d", retval); tstmsg_case_msg(msg); retval = getdns_list_get_length(list, NULL); - sprintf(msg, "list, NULL, retval = %d", retval); + snprintf(msg, sizeof(msg), "list, NULL, retval = %d", retval); tstmsg_case_msg(msg); getdns_list_destroy(list);