login.c (destroy_tickets): Just use global kcontext instead
authorTheodore Tso <tytso@mit.edu>
Thu, 7 Nov 1996 17:07:03 +0000 (17:07 +0000)
committerTheodore Tso <tytso@mit.edu>
Thu, 7 Nov 1996 17:07:03 +0000 (17:07 +0000)
      of creating a new krb5 context for no good reason.

krshd.c (main):
krsh.c (main):
krlogind.c (main):
krlogin.c (main):
krcp.c(main): Check the error return from krb5_init_context(),
      and print an error message if necessary.

git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@9317 dc483132-0cff-0310-8789-dd5450dbe970

src/appl/bsd/ChangeLog
src/appl/bsd/krcp.c
src/appl/bsd/krlogin.c
src/appl/bsd/krlogind.c
src/appl/bsd/krsh.c
src/appl/bsd/krshd.c
src/appl/bsd/login.c

index 1cf3eb8c752ed6ce019d9530dad44e3718cf4af3..dd7acc02cb264343899fc0f0c73a1cce80958926 100644 (file)
@@ -1,3 +1,15 @@
+Wed Nov  6 11:32:36 1996  Theodore Ts'o  <tytso@rsts-11.mit.edu>
+
+       * login.c (destroy_tickets): Just use global kcontext instead
+               of creating a new krb5 context for no good reason.
+
+       * krshd.c (main): 
+       * krsh.c (main): 
+       * krlogind.c (main): 
+       * krlogin.c (main): 
+       * krcp.c(main): Check the error return from krb5_init_context(),
+               and print an error message if necessary.
+
 Fri Nov  1 23:11:21 1996  Sam Hartman  <hartmans@planet-zorp.MIT.EDU>
 
        * login.c: print out system error if -1 returns.
index 88b2b76b639887bd0b67f460f61ade2340960ab1..a353e55a03d43506800ea30dd74cf96856e843aa 100644 (file)
@@ -162,8 +162,11 @@ int main(argc, argv)
     int euid;
     char **orig_argv = save_argv(argc, argv);
 
-    krb5_init_context(&bsd_context);
-    krb5_init_ets(bsd_context);
+    status = krb5_init_context(&bsd_context);
+    if (status) {
+           com_err(argv[0], status, "while initializing krb5");
+           exit(1);
+    }
     desinbuf.data = des_inbuf;
     desoutbuf.data = des_outbuf;    /* Set up des buffers */
 #endif
index 2f0c348894199da235edc8ff46e2e8006861e480..0e51620ffb23a115484a3f67a48efff6cd460ee7 100644 (file)
@@ -472,8 +472,11 @@ main(argc, argv)
        exit(1);
     }
 #ifdef KERBEROS
-    krb5_init_context(&bsd_context);
-    krb5_init_ets(bsd_context);
+    status = krb5_init_context(&bsd_context);
+    if (status) {
+           com_err(argv[0], status, "while initializing krb5");
+           exit(1);
+    }
     desinbuf.data = des_inbuf;
     desoutbuf.data = des_outpkt+4;     /* Set up des buffers */
 #endif
index 2df7c80709501f9082ae7218da9cd8225d67b1e3..7773de5175eacf37686af237d40b091fd4ca8401 100644 (file)
@@ -337,17 +337,21 @@ int main(argc, argv)
 #define LOG_NDELAY 0
 #endif
     
-#ifdef KERBEROS
-    krb5_init_context(&bsd_context);
-    krb5_init_ets(bsd_context);
-#endif
-    
 #ifndef LOG_AUTH /* 4.2 syslog */
     openlog(progname, LOG_PID | LOG_NDELAY);
 #else
     openlog(progname, LOG_PID | LOG_NDELAY, LOG_AUTH);
 #endif /* 4.2 syslog */
     
+#ifdef KERBEROS
+    status = krb5_init_context(&bsd_context);
+    if (status) {
+           syslog(LOG_ERR, "Error initializing krb5: %s",
+                  error_message(status));
+           exit(1);
+    }
+#endif
+    
     /* Analyse parameters. */
     opterr = 0;
     while ((ch = getopt(argc, argv, ARGSTR)) != EOF)
index a1cca9c6049831a4a1527548d2775c1e481fa55b..a7fabc432481bd88c1ece0f7f347a4b86814eb05 100644 (file)
@@ -333,8 +333,11 @@ main(argc, argv0)
     }
 
 #ifdef KERBEROS
-    krb5_init_context(&bsd_context);
-    krb5_init_ets(bsd_context);
+    status = krb5_init_context(&bsd_context);
+    if (status) {
+           com_err(argv[0], status, "while initializing krb5");
+           exit(1);
+    }
     authopts = AP_OPTS_MUTUAL_REQUIRED;
 
     /* Piggy-back forwarding flags on top of authopts; */
index 4f064e9ee0c91e1d7bac1ddf30a23a26d88a863c..fdbaba8dcbe7514e788f996e078d63dc60d230c9 100644 (file)
@@ -270,11 +270,14 @@ int main(argc, argv)
 #endif /* 4.2 syslog */
     
 #ifdef KERBEROS
-    krb5_init_context(&bsd_context);
-    krb5_init_ets(bsd_context);
+    status = krb5_init_context(&bsd_context);
+    if (status) {
+           syslog(LOG_ERR, "Error initializing krb5: %s",
+                  error_message(status));
+           exit(1);
+    }
 #endif
     
-    
     /* Analyze parameters. */
     opterr = 0;
     while ((ch = getopt(argc, argv, ARGSTR)) != EOF)
index 26d56348b66b64da73efc1e7c5bc5f9278b4a174..dfcecc217703b1c3f5bc81fe725922702840bb17 100644 (file)
@@ -470,9 +470,14 @@ char realm[REALM_SZ];
 void k_init (ttyn)
     char *ttyn;
 {
-    krb5_init_context(&kcontext);
-    krb5_init_ets(kcontext);
-krb5_secure_config_files (kcontext);
+    krb5_error_code retval;
+    
+    retval = krb5_init_context(&kcontext);
+    if (retval) {
+       com_err("login", retval, "while initializing krb5");
+       exit(1);
+    }
+    krb5_secure_config_files (kcontext);
     login_get_kconf(kcontext);
 
     /* Set up the credential cache environment variable */
@@ -990,13 +995,14 @@ destroy_tickets()
 {
        krb5_context c;
        krb5_ccache cache;
+       krb5_error_code retval;
 
+#ifdef KRB5_GET_TICKETS
        if (login_krb5_get_tickets) {
-       krb5_init_context(&c);
-       krb5_secure_config_files (c);
-       if(!krb5_cc_default(c, &cache))
-               krb5_cc_destroy (c, cache);
+           if(!krb5_cc_default(kcontext, &cache))
+               krb5_cc_destroy (kcontext, cache);
        }
+#endif
 #ifdef KRB4_GET_TICKETS
        if (login_krb4_get_tickets||login_krb4_convert)
        dest_tkt();