diff --git a/lib/isccfg/cfgmgr.c b/lib/isccfg/cfgmgr.c index 8264e9c5a6..8e2e8d732f 100644 --- a/lib/isccfg/cfgmgr.c +++ b/lib/isccfg/cfgmgr.c @@ -637,7 +637,7 @@ isc__cfgmgr_getval(const char *name, isc_cfgmgr_val_t *value) { if (value->type == ISC_CFGMGR_STRING) { value->string = ((char *)dbval.mv_data) + sizeof(value->type); } - INSIST(value->type != ISC_CFGMGR_UNKNOWN); + INSIST(value->type != ISC_CFGMGR_UNDEFINED); out: return result; @@ -667,7 +667,7 @@ isc__cfgmgr_setval(const char *name, const isc_cfgmgr_val_t *value, bool list) { REQUIRE(isc__cfgmgr_ctx.cursor != NULL); REQUIRE(isc__cfgmgr_ctx.readonly == false); REQUIRE(name != NULL); - REQUIRE((value != NULL && value->type != ISC_CFGMGR_UNKNOWN) || + REQUIRE((value != NULL && value->type != ISC_CFGMGR_UNDEFINED) || (value == NULL && list == false)); isc__cfgmgr_buildkey(name, false); diff --git a/lib/isccfg/include/isccfg/cfgmgr.h b/lib/isccfg/include/isccfg/cfgmgr.h index eef3253b5b..f4a449192f 100644 --- a/lib/isccfg/include/isccfg/cfgmgr.h +++ b/lib/isccfg/include/isccfg/cfgmgr.h @@ -20,7 +20,7 @@ * Supported data types for read/write operations from/to cfgmgr. */ typedef enum { - ISC_CFGMGR_UNKNOWN = 0, + ISC_CFGMGR_UNDEFINED = 0, ISC_CFGMGR_STRING, ISC_CFGMGR_BOOLEAN, ISC_CFGMGR_NONE, diff --git a/tests/isccfg/cfgmgr_test.c b/tests/isccfg/cfgmgr_test.c index b39d9cf372..3a9118cd4d 100644 --- a/tests/isccfg/cfgmgr_test.c +++ b/tests/isccfg/cfgmgr_test.c @@ -29,7 +29,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_rw) { isc_result_t result; isc_cfgmgr_val_t val1; - isc_cfgmgr_val_t val2 = { .type = ISC_CFGMGR_UNKNOWN }; + isc_cfgmgr_val_t val2 = { .type = ISC_CFGMGR_UNDEFINED }; result = isc_cfgmgr_init(mctx, TEST_DBPATH); assert_int_equal(result, ISC_R_SUCCESS); @@ -73,7 +73,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_rw) { result = isc_cfgmgr_setval("anotherprop", &val1); assert_int_equal(result, ISC_R_SUCCESS); - val2.type = ISC_CFGMGR_UNKNOWN; + val2.type = ISC_CFGMGR_UNDEFINED; result = isc_cfgmgr_getval("anotherprop", &val2); assert_int_equal(result, ISC_R_SUCCESS); assert_int_equal(val2.type, ISC_CFGMGR_BOOLEAN); @@ -378,7 +378,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_override) { result = isc_cfgmgr_setval("prop1", &val1); assert_int_equal(result, ISC_R_SUCCESS); - val2.type = ISC_CFGMGR_UNKNOWN; + val2.type = ISC_CFGMGR_UNDEFINED; result = isc_cfgmgr_getval("prop1", &val2); assert_int_equal(result, ISC_R_SUCCESS); assert_int_equal(val2.type, ISC_CFGMGR_BOOLEAN); @@ -393,7 +393,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_override) { ISC_RUN_TEST_IMPL(isc_cfgmgr_rw_string) { isc_result_t result; isc_cfgmgr_val_t val1; - isc_cfgmgr_val_t val2 = { .type = ISC_CFGMGR_UNKNOWN }; + isc_cfgmgr_val_t val2 = { .type = ISC_CFGMGR_UNDEFINED }; result = isc_cfgmgr_init(mctx, TEST_DBPATH); assert_int_equal(result, ISC_R_SUCCESS); @@ -571,7 +571,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_list) { ISC_RUN_TEST_IMPL(isc_cfgmgr_repeatable_clauses) { isc_result_t result; isc_cfgmgr_val_t val1; - isc_cfgmgr_val_t val2 = { .type = ISC_CFGMGR_UNKNOWN }; + isc_cfgmgr_val_t val2 = { .type = ISC_CFGMGR_UNDEFINED }; result = isc_cfgmgr_init(mctx, TEST_DBPATH); assert_int_equal(result, ISC_R_SUCCESS); @@ -614,7 +614,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_repeatable_clauses) { result = isc_cfgmgr_getval("p2", &val2); assert_int_equal(val2.type, ISC_CFGMGR_BOOLEAN); - val1.type = ISC_CFGMGR_UNKNOWN; + val1.type = ISC_CFGMGR_UNDEFINED; if (val2.boolean) { result = isc_cfgmgr_getval("p1", &val1); assert_int_equal(result, ISC_R_SUCCESS); @@ -630,12 +630,12 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_repeatable_clauses) { result = isc_cfgmgr_nextclause(); assert_int_equal(result, ISC_R_SUCCESS); - val2.type = ISC_CFGMGR_UNKNOWN; + val2.type = ISC_CFGMGR_UNDEFINED; result = isc_cfgmgr_getval("p2", &val2); assert_int_equal(result, ISC_R_SUCCESS); assert_int_equal(val2.type, ISC_CFGMGR_BOOLEAN); - val1.type = ISC_CFGMGR_UNKNOWN; + val1.type = ISC_CFGMGR_UNDEFINED; if (val2.boolean) { result = isc_cfgmgr_getval("p1", &val1); assert_int_equal(result, ISC_R_SUCCESS); @@ -699,7 +699,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_nested_clauses) { result = isc_cfgmgr_open("baz"); assert_int_equal(result, ISC_R_SUCCESS); - val.type = ISC_CFGMGR_UNKNOWN; + val.type = ISC_CFGMGR_UNDEFINED; result = isc_cfgmgr_getval("gee", &val); assert_int_equal(result, ISC_R_SUCCESS); assert_int_equal(val.type, ISC_CFGMGR_NONE); @@ -747,7 +747,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_nested_clauses) { result = isc_cfgmgr_open("foo"); assert_int_equal(result, ISC_R_SUCCESS); - val.type = ISC_CFGMGR_UNKNOWN; + val.type = ISC_CFGMGR_UNDEFINED; result = isc_cfgmgr_getval("propfoo", &val); assert_int_equal(result, ISC_R_SUCCESS); assert_int_equal(val.type, ISC_CFGMGR_STRING); @@ -792,7 +792,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_nested_clauses) { result = isc_cfgmgr_close(); assert_int_equal(result, ISC_R_SUCCESS); - val = (isc_cfgmgr_val_t){ .type = ISC_CFGMGR_UNKNOWN, .string = NULL }; + val = (isc_cfgmgr_val_t){ .type = ISC_CFGMGR_UNDEFINED, .string = NULL }; result = isc_cfgmgr_open("foo"); assert_int_equal(result, ISC_R_SUCCESS); @@ -813,7 +813,7 @@ ISC_RUN_TEST_IMPL(isc_cfgmgr_nested_clauses) { result = isc_cfgmgr_nextclause(); assert_int_equal(result, ISC_R_SUCCESS); - val = (isc_cfgmgr_val_t){ .type = ISC_CFGMGR_UNKNOWN, .string = NULL }; + val = (isc_cfgmgr_val_t){ .type = ISC_CFGMGR_UNDEFINED, .string = NULL }; result = isc_cfgmgr_getval("propsubclause", &val); assert_int_equal(result, ISC_R_SUCCESS); assert_int_equal(val.type, ISC_CFGMGR_STRING);