Fix some unintialized variable warnings

fixed warning: expression which evaluates to zero treated as a null pointer constant of type 'struct addrinfo *'

Seems to be innocent, but clang is correct that this is strange. init_tun() expects two pointers, but options_string() tried to feed it two uint32_t values.

Signed-off-by Steffan Karger <[email protected]...>

---
 src/openvpn/options.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

src/openvpn/options.c +0 -0

index f536daa..84eb6ed 100644

Does not actually change behaviour, but fixes compiler warnings and properly initializing is good habit anyway.

Signed-off-by Steffan Karger <[email protected]...>

---
 src/openvpn/plugin.c | 2 +-
 src/openvpn/sig.c    | 2 +-
 src/openvpn/socket.c | 4 ++--
 3 files changed, 4 insertions(+), 4 deletions(-)

src/openvpn/plugin.c +1 -1

index 0948f23..54c5b52 100644
@@ -291,7 +291,7 @@ plugin_init_item (struct plugin *p, const struct plugin_option *o)
 static void
 plugin_vlog (openvpn_plugin_log_flags_t flags, const char *name, const char *format, va_list arglist)
 {
-  unsigned int msg_flags;
+  unsigned int msg_flags = 0;
 
   if (!format)
     return;

src/openvpn/sig.c +1 -1

index 90e39a4..a3d29de 100644
@@ -126,7 +126,7 @@ print_signal (const struct signal_info *si, const char *title, int msglevel)
     {
       const char *type = (si->signal_text ? si->signal_text : "");
       const char *t = (title ? title : "process");
-      const char *hs;
+      const char *hs = NULL;
       switch (si->source)
         {
         case SIG_SOURCE_SOFT:

src/openvpn/socket.c +0 -1

index 9e6bd10..c649d62 100644
@@ -2354,12 +2354,12 @@ print_sockaddr_ex (const struct sockaddr *sa,
 				   struct gc_arena *gc)
 {
   struct buffer out = alloc_buf_gc (128, gc);
-  bool addr_is_defined;