Andrew Cann wrote:

Here's a very small patch that changes one of the arguments to
GNUNET_CONFIGURATION_get_value_choice. C++ doesn't allow you to cast
from a (char **) to a (const char **) because that enables overwriting
read-only memory. This patch makes the argument "more const" (which is
always a good thing) and makes it possible to call it from C++ passing
in a (char **).

https://canndrew.org/misc/change_GNUNET_CONFIGURATION_get_value_choice_argument.diff

1 files changed, 1 insertions, 1 deletions

diff --git a/src/util/configuration.c b/src/util/configuration.c
index 69e2168c1..a1150aa9b 100644
--- a/src/util/configuration.c
+++ b/src/util/configuration.c
@@ -964,7 +964,7 @@ GNUNET_CONFIGURATION_get_value_string (const struct GNUNET_CONFIGURATION_Handle
 int
 GNUNET_CONFIGURATION_get_value_choice (const struct GNUNET_CONFIGURATION_Handle
                                        *cfg, const char *section,
-                                       const char *option, const char **choices,
+                                       const char *option, const char *const *choices,
                                        const char **value)
 {
   struct ConfigEntry *e;