cifs.upcall: make most functions static and silence compiler warnings
authorJeff Layton <jlayton@redhat.com>
Sat, 13 Sep 2008 01:13:54 +0000 (21:13 -0400)
committerJeff Layton <jlayton@redhat.com>
Sat, 13 Sep 2008 01:13:54 +0000 (21:13 -0400)
...to silence -Wmissing-prototypes and some uninit'ed variable
warnings. Thanks to GD for the extra-paranoid compiler flags.

Signed-off-by: Jeff Layton <jlayton@redhat.com>
(This used to be commit f0ee2c145fa031f91c28a69a44b7652f18eea0f0)

source3/client/cifs.upcall.c

index 7cb51660d7ea0a5f3978c17a789626697d2ab8bc..4110de35fde43dacc0eaa6300bd08922fb90223a 100644 (file)
@@ -32,6 +32,7 @@ create dns_resolver * * /usr/local/sbin/cifs.upcall %k
 const char *CIFSSPNEGO_VERSION = "1.2";
 static const char *prog = "cifs.upcall";
 typedef enum _secType {
 const char *CIFSSPNEGO_VERSION = "1.2";
 static const char *prog = "cifs.upcall";
 typedef enum _secType {
+       NONE = 0,
        KRB5,
        MS_KRB5
 } secType_t;
        KRB5,
        MS_KRB5
 } secType_t;
@@ -56,7 +57,8 @@ typedef enum _secType {
  *
  * ret: 0 - success, others - failure
 */
  *
  * ret: 0 - success, others - failure
 */
-int handle_krb5_mech(const char *oid, const char *principal,
+static int
+handle_krb5_mech(const char *oid, const char *principal,
                     DATA_BLOB * secblob, DATA_BLOB * sess_key)
 {
        int retval;
                     DATA_BLOB * secblob, DATA_BLOB * sess_key)
 {
        int retval;
@@ -88,7 +90,8 @@ int handle_krb5_mech(const char *oid, const char *principal,
 #define DKD_HAVE_UID           32
 #define DKD_MUSTHAVE_SET (DKD_HAVE_HOSTNAME|DKD_HAVE_VERSION|DKD_HAVE_SEC)
 
 #define DKD_HAVE_UID           32
 #define DKD_MUSTHAVE_SET (DKD_HAVE_HOSTNAME|DKD_HAVE_VERSION|DKD_HAVE_SEC)
 
-int decode_key_description(const char *desc, int *ver, secType_t * sec,
+static int
+decode_key_description(const char *desc, int *ver, secType_t * sec,
                           char **hostname, uid_t * uid)
 {
        int retval = 0;
                           char **hostname, uid_t * uid)
 {
        int retval = 0;
@@ -151,7 +154,8 @@ int decode_key_description(const char *desc, int *ver, secType_t * sec,
        return retval;
 }
 
        return retval;
 }
 
-int cifs_resolver(const key_serial_t key, const char *key_descr)
+static int
+cifs_resolver(const key_serial_t key, const char *key_descr)
 {
        int c;
        struct addrinfo *addr;
 {
        int c;
        struct addrinfo *addr;
@@ -203,7 +207,7 @@ int cifs_resolver(const key_serial_t key, const char *key_descr)
        return 0;
 }
 
        return 0;
 }
 
-void
+static void
 usage(void)
 {
        syslog(LOG_WARNING, "Usage: %s [-c] [-v] key_serial", prog);
 usage(void)
 {
        syslog(LOG_WARNING, "Usage: %s [-c] [-v] key_serial", prog);
@@ -215,12 +219,12 @@ int main(const int argc, char *const argv[])
        struct cifs_spnego_msg *keydata = NULL;
        DATA_BLOB secblob = data_blob_null;
        DATA_BLOB sess_key = data_blob_null;
        struct cifs_spnego_msg *keydata = NULL;
        DATA_BLOB secblob = data_blob_null;
        DATA_BLOB sess_key = data_blob_null;
-       secType_t sectype;
+       secType_t sectype = NONE;
        key_serial_t key = 0;
        size_t datalen;
        long rc = 1;
        key_serial_t key = 0;
        size_t datalen;
        long rc = 1;
-       uid_t uid;
-       int kernel_upcall_version;
+       uid_t uid = 0;
+       int kernel_upcall_version = 0;
        int c, use_cifs_service_prefix = 0;
        char *buf, *hostname = NULL;
        const char *oid;
        int c, use_cifs_service_prefix = 0;
        char *buf, *hostname = NULL;
        const char *oid;