projects
/
xscreensaver
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
From http://www.jwz.org/xscreensaver/xscreensaver-5.21.tar.gz
[xscreensaver]
/
driver
/
passwd-kerberos.c
diff --git
a/driver/passwd-kerberos.c
b/driver/passwd-kerberos.c
index cdb22b595d0ade04aa309caacd861e080f2bf9b7..202e0eb10982395a2f1c1186545c55e7ec4da38e 100644
(file)
--- a/
driver/passwd-kerberos.c
+++ b/
driver/passwd-kerberos.c
@@
-1,7
+1,6
@@
/* kpasswd.c --- verify kerberos passwords.
* written by Nat Lanza (magus@cs.cmu.edu) for
/* kpasswd.c --- verify kerberos passwords.
* written by Nat Lanza (magus@cs.cmu.edu) for
- * xscreensaver, Copyright (c) 1993-1997, 1998, 2000, 2003
- * Jamie Zawinski <jwz@jwz.org>
+ * xscreensaver, Copyright (c) 1993-2004 Jamie Zawinski <jwz@jwz.org>
*
* Permission to use, copy, modify, distribute, and sell this software and its
* documentation for any purpose is hereby granted without fee, provided that
*
* Permission to use, copy, modify, distribute, and sell this software and its
* documentation for any purpose is hereby granted without fee, provided that
@@
-26,6
+25,7
@@
#include <stdio.h>
#include <string.h>
#include <sys/types.h>
#include <stdio.h>
#include <string.h>
#include <sys/types.h>
+#include <sys/stat.h>
/* I'm not sure if this is exactly the right test...
Might __APPLE__ be defined if this is apple hardware, but not
/* I'm not sure if this is exactly the right test...
Might __APPLE__ be defined if this is apple hardware, but not
@@
-75,9
+75,13
@@
static char realm[REALM_SZ];
static char name[ANAME_SZ];
static char inst[INST_SZ];
static char realm[REALM_SZ];
static char name[ANAME_SZ];
static char inst[INST_SZ];
- static char *tk_file;
+ static c
onst c
har *tk_file;
#endif /* !HAVE_DARWIN */
#endif /* !HAVE_DARWIN */
+/* warning suppression: duplicated in passwd.c */
+extern Bool kerberos_lock_init (int argc, char **argv, Bool verbose_p);
+extern Bool kerberos_passwd_valid_p (const char *typed_passwd, Bool verbose_p);
+
/* Called at startup to grab user, instance, and realm information
from the user's ticketfile (remember, name.inst@realm). Since we're
/* Called at startup to grab user, instance, and realm information
from the user's ticketfile (remember, name.inst@realm). Since we're
@@
-192,6
+196,7
@@
kerberos_passwd_valid_p (const char *typed_passwd, Bool verbose_p)
C_Block mitkey;
Bool success;
char *newtkfile;
C_Block mitkey;
Bool success;
char *newtkfile;
+ int fh = -1;
/* temporarily switch to a new ticketfile.
I'm not using tmpnam() because it isn't entirely portable.
/* temporarily switch to a new ticketfile.
I'm not using tmpnam() because it isn't entirely portable.
@@
-199,7
+204,19
@@
kerberos_passwd_valid_p (const char *typed_passwd, Bool verbose_p)
newtkfile = malloc(80 * sizeof(char));
memset(newtkfile, 0, sizeof(newtkfile));
newtkfile = malloc(80 * sizeof(char));
memset(newtkfile, 0, sizeof(newtkfile));
- sprintf(newtkfile, "/tmp/xscrn-%i", getpid());
+ sprintf(newtkfile, "/tmp/xscrn-%i.XXXXXX", getpid());
+
+ if( (fh = mkstemp(newtkfile)) < 0)
+ {
+ free(newtkfile);
+ return(False);
+ }
+ if( fchmod(fh, 0600) < 0)
+ {
+ free(newtkfile);
+ return(False);
+ }
+
krb_set_tkt_string(newtkfile);
krb_set_tkt_string(newtkfile);
@@
-209,7
+226,7
@@
kerberos_passwd_valid_p (const char *typed_passwd, Bool verbose_p)
des_string_to_key(typed_passwd, mitkey);
if (krb_get_in_tkt(name, inst, realm, "krbtgt", realm, DEFAULT_TKT_LIFE,
des_string_to_key(typed_passwd, mitkey);
if (krb_get_in_tkt(name, inst, realm, "krbtgt", realm, DEFAULT_TKT_LIFE,
- key_to_key, NULL, mitkey) != 0) {
+ key_to_key, NULL,
(char *)
mitkey) != 0) {
success = False;
} else {
success = True;
success = False;
} else {
success = True;
@@
-222,6
+239,7
@@
kerberos_passwd_valid_p (const char *typed_passwd, Bool verbose_p)
krb_set_tkt_string(tk_file);
free(newtkfile);
memset(mitkey, 0, sizeof(mitkey));
krb_set_tkt_string(tk_file);
free(newtkfile);
memset(mitkey, 0, sizeof(mitkey));
+ close(fh); /* #### tom: should the file be removed? */
/* Did we verify successfully? */
/* Did we verify successfully? */