[SSSD] [PATCH] Fix warnings from -Wmissing-field-initializers

Sumit Bose sbose at redhat.com
Wed Mar 24 12:12:22 UTC 2010


Hi,

this patch fixes the warnings generated by -Wmissing-field-initializers
by adding the missing elements.

Please have a look at the changes to pysss.c because I added just NULL
to a element called 'doc', I do not know if it makes sense to put a
better description here.

bye,
Sumit
-------------- next part --------------
From b1b7d2dc705264850c9e2cdef2580c94f0d084ba Mon Sep 17 00:00:00 2001
From: Sumit Bose <sbose at redhat.com>
Date: Wed, 24 Mar 2010 13:03:19 +0100
Subject: [PATCH] Fix warnings from -Wmissing-field-initializers

This patch removes some tab-indentations from pamsrv.c, too.
---
 src/monitor/monitor.c          |    2 +-
 src/providers/ldap/ldap_auth.c |    4 +++-
 src/python/pysss.c             |    8 ++++----
 src/resolv/async_resolv.c      |    2 +-
 src/responder/nss/nsssrv.c     |    2 +-
 src/responder/pam/pamsrv.c     |   34 +++++++++++++++++-----------------
 src/tools/sss_useradd.c        |    2 +-
 7 files changed, 28 insertions(+), 26 deletions(-)

diff --git a/src/monitor/monitor.c b/src/monitor/monitor.c
index 58bef34..d086309 100644
--- a/src/monitor/monitor.c
+++ b/src/monitor/monitor.c
@@ -2189,7 +2189,7 @@ int main(int argc, const char *argv[])
          _("Run interactive (not a daemon)"), NULL}, \
         {"config", 'c', POPT_ARG_STRING, &opt_config_file, 0, \
          _("Specify a non-default config file"), NULL}, \
-        { NULL }
+        POPT_TABLEEND
     };
 
     pc = poptGetContext(argv[0], argc, argv, long_options, 0);
diff --git a/src/providers/ldap/ldap_auth.c b/src/providers/ldap/ldap_auth.c
index 8c77e3a..c78f503 100644
--- a/src/providers/ldap/ldap_auth.c
+++ b/src/providers/ldap/ldap_auth.c
@@ -57,9 +57,11 @@ static errno_t check_pwexpire_kerberos(const char *expire_date, time_t now,
                                        enum sdap_result *result)
 {
     char *end;
-    struct tm tm = {0, 0, 0, 0, 0, 0, 0, 0, 0};
+    struct tm tm;
     time_t expire_time;
 
+    memset(&tm, 0, sizeof(tm));
+
     *result = SDAP_AUTH_FAILED;
 
     end = strptime(expire_date, "%Y%m%d%H%M%SZ", &tm);
diff --git a/src/python/pysss.c b/src/python/pysss.c
index 8011ed6..bc1cf6e 100644
--- a/src/python/pysss.c
+++ b/src/python/pysss.c
@@ -888,10 +888,10 @@ static PyMethodDef sss_local_methods[] = {
 
 static PyMemberDef sss_members[] = {
     { discard_const_p(char, "lock"), T_INT,
-      offsetof(PySssLocalObject, lock), RO },
+      offsetof(PySssLocalObject, lock), RO, NULL},
     { discard_const_p(char, "unlock"), T_INT,
-      offsetof(PySssLocalObject, unlock), RO },
-    {NULL} /* Sentinel */
+      offsetof(PySssLocalObject, unlock), RO, NULL},
+    {NULL, 0, 0, 0, NULL} /* Sentinel */
 };
 
 /*
@@ -913,7 +913,7 @@ static PyTypeObject pysss_local_type = {
  * Module methods
  */
 static PyMethodDef module_methods[] = {
-        {NULL}  /* Sentinel */
+        {NULL, NULL, 0, NULL}  /* Sentinel */
 };
 
 /*
diff --git a/src/resolv/async_resolv.c b/src/resolv/async_resolv.c
index d0c5b7a..28237a0 100644
--- a/src/resolv/async_resolv.c
+++ b/src/resolv/async_resolv.c
@@ -140,7 +140,7 @@ check_fd_timeouts(struct tevent_context *ev, struct tevent_timer *te,
 static void
 add_timeout_timer(struct tevent_context *ev, struct resolv_ctx *ctx)
 {
-    struct timeval tv = { 0 };
+    struct timeval tv = { 0, 0 };
     struct timeval *tvp;
 
     tvp = ares_timeout(ctx->channel, NULL, &tv);
diff --git a/src/responder/nss/nsssrv.c b/src/responder/nss/nsssrv.c
index 9785eeb..ea2dc51 100644
--- a/src/responder/nss/nsssrv.c
+++ b/src/responder/nss/nsssrv.c
@@ -409,7 +409,7 @@ int main(int argc, const char *argv[])
     struct poptOption long_options[] = {
         POPT_AUTOHELP
         SSSD_MAIN_OPTS
-        { NULL }
+        POPT_TABLEEND
     };
 
     pc = poptGetContext(argv[0], argc, argv, long_options, 0);
diff --git a/src/responder/pam/pamsrv.c b/src/responder/pam/pamsrv.c
index cb3429f..7903e34 100644
--- a/src/responder/pam/pamsrv.c
+++ b/src/responder/pam/pamsrv.c
@@ -168,24 +168,24 @@ int main(int argc, const char *argv[])
     struct main_context *main_ctx;
     int ret;
 
-	struct poptOption long_options[] = {
-		POPT_AUTOHELP
+    struct poptOption long_options[] = {
+        POPT_AUTOHELP
         SSSD_MAIN_OPTS
-		{ NULL }
-	};
-
-	pc = poptGetContext(argv[0], argc, argv, long_options, 0);
-	while((opt = poptGetNextOpt(pc)) != -1) {
-		switch(opt) {
-		default:
-			fprintf(stderr, "\nInvalid option %s: %s\n\n",
-				  poptBadOption(pc, 0), poptStrerror(opt));
-			poptPrintUsage(pc, stderr, 0);
-			return 1;
-		}
-	}
-
-	poptFreeContext(pc);
+        POPT_TABLEEND
+    };
+
+    pc = poptGetContext(argv[0], argc, argv, long_options, 0);
+    while((opt = poptGetNextOpt(pc)) != -1) {
+        switch(opt) {
+        default:
+            fprintf(stderr, "\nInvalid option %s: %s\n\n",
+                             poptBadOption(pc, 0), poptStrerror(opt));
+            poptPrintUsage(pc, stderr, 0);
+            return 1;
+        }
+    }
+
+    poptFreeContext(pc);
 
     /* set up things like debug, signals, daemonization, etc... */
     debug_log_file = "sssd_pam";
diff --git a/src/tools/sss_useradd.c b/src/tools/sss_useradd.c
index 077ac99..2d88e75 100644
--- a/src/tools/sss_useradd.c
+++ b/src/tools/sss_useradd.c
@@ -120,7 +120,7 @@ int main(int argc, const char **argv)
         { "groups", 'G', POPT_ARG_STRING, NULL, 'G', _("Groups"), NULL },
         { "create-home", 'm', POPT_ARG_NONE, NULL, 'm', _("Create user's directory if it does not exist"), NULL },
         { "no-create-home", 'M', POPT_ARG_NONE, NULL, 'M', _("Never create user's directory, overrides config"), NULL },
-        { "skel", 'k', POPT_ARG_STRING, &pc_skeldir, 0, _("Specify an alternative skeleton directory") },
+        { "skel", 'k', POPT_ARG_STRING, &pc_skeldir, 0, _("Specify an alternative skeleton directory"), NULL },
         POPT_TABLEEND
     };
     poptContext pc = NULL;
-- 
1.6.6.1



More information about the sssd-devel mailing list