Properly initialize sockaddr_in struct in fast and dhcp plugins

diff --git a/src/libcharon/plugins/dhcp/dhcp_socket.c b/src/libcharon/plugins/dhcp/dhcp_socket.c
index 104c71c01e43590f6358d6aaa2ab5f753fc54fed..f469c5a35f94f395cfffeac19394995aac2e4661 100644 (file)
--- a/src/libcharon/plugins/dhcp/dhcp_socket.c
+++ b/src/libcharon/plugins/dhcp/dhcp_socket.c
@@ -648,7 +648,13 @@ METHOD(dhcp_socket_t, destroy, void,
 dhcp_socket_t *dhcp_socket_create()
 {
        private_dhcp_socket_t *this;
-       struct sockaddr_in src;
+       struct sockaddr_in src = {
+               .sin_family = AF_INET,
+               .sin_port = htons(DHCP_CLIENT_PORT),
+               .sin_addr = {
+                       .s_addr = INADDR_ANY,
+               },
+       };
        int on = 1;
        struct sock_filter dhcp_filter_code[] = {
                BPF_STMT(BPF_LD+BPF_B+BPF_ABS,
@@ -738,9 +744,6 @@ dhcp_socket_t *dhcp_socket_create()
                destroy(this);
                return NULL;
        }
-       src.sin_family = AF_INET;
-       src.sin_port = htons(DHCP_CLIENT_PORT);
-       src.sin_addr.s_addr = INADDR_ANY;
        if (bind(this->send, (struct sockaddr*)&src, sizeof(src)) == -1)
        {
                DBG1(DBG_CFG, "unable to bind DHCP send socket: %s", strerror(errno));

diff --git a/src/libfast/smtp.c b/src/libfast/smtp.c
index 4118c74a6e3a1b10e97f4cb00eaa6adeccb3d280..1375c2944a1890f5e2a23ffd044047cb38aba7db 100644 (file)
--- a/src/libfast/smtp.c
+++ b/src/libfast/smtp.c
@@ -136,7 +136,13 @@ METHOD(smtp_t, destroy, void,
 smtp_t *smtp_create()
 {
        private_smtp_t *this;
-       struct sockaddr_in addr;
+       struct sockaddr_in addr = {
+               .sin_family = AF_INET,
+               .sin_port = htons(25),
+               .sin_addr = {
+                       .s_addr = htonl(INADDR_LOOPBACK),
+               },
+       };
        int s;
 
        INIT(this,
@@ -153,9 +159,6 @@ smtp_t *smtp_create()
                free(this);
                return NULL;
        }
-       addr.sin_family = AF_INET;
-       addr.sin_addr.s_addr = htonl(INADDR_LOOPBACK);
-       addr.sin_port = htons(25);
        if (connect(s, (struct sockaddr*)&addr, sizeof(addr)) < 0)
        {
                DBG1(DBG_LIB, "connecting to SMTP server failed: %s", strerror(errno));