MINOR: config: Don't dump keywords if argument is NULL
Helper functions are used to dump bind, server or filter keywords. These
functions are used to report errors during the configuration parsing. To have a
coherent API, these functions are now prepared to handle a null pointer as
argument. If so, no action is performed and functions immediately return.
This patch should fix the issue #631. It is not a bug. There is no reason to
backport it.
diff --git a/src/filters.c b/src/filters.c
index e275050..07b9043 100644
--- a/src/filters.c
+++ b/src/filters.c
@@ -142,6 +142,9 @@
struct flt_kw_list *kwl;
int index;
+ if (!out)
+ return;
+
*out = NULL;
list_for_each_entry(kwl, &flt_keywords.list, list) {
for (index = 0; kwl->kw[index].kw != NULL; index++) {
diff --git a/src/listener.c b/src/listener.c
index 829dc99..01feee5 100644
--- a/src/listener.c
+++ b/src/listener.c
@@ -1255,6 +1255,9 @@
struct bind_kw_list *kwl;
int index;
+ if (!out)
+ return;
+
*out = NULL;
list_for_each_entry(kwl, &bind_keywords.list, list) {
for (index = 0; kwl->kw[index].kw != NULL; index++) {
diff --git a/src/server.c b/src/server.c
index 265106e..e4044cd 100644
--- a/src/server.c
+++ b/src/server.c
@@ -247,6 +247,9 @@
struct srv_kw_list *kwl;
int index;
+ if (!out)
+ return;
+
*out = NULL;
list_for_each_entry(kwl, &srv_keywords.list, list) {
for (index = 0; kwl->kw[index].kw != NULL; index++) {