Ezra Peisach [Mon, 12 Feb 2007 00:19:47 +0000 (00:19 +0000)]
In the resolve function - ensure on memory allocation error -
deref of null pointer does not happen.
Also include kt-int.h for prototypes.
ticket: 5411
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19153
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Mon, 5 Feb 2007 20:39:59 +0000 (20:39 +0000)]
krb5_kt_default_name should take an unsized length
The internal code to krb5_kt_default_name was casting the length to size_t.
Change prototype to take an unsigned - which makes sense in how the code uses
it.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19144
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Mon, 5 Feb 2007 15:32:13 +0000 (15:32 +0000)]
krb5_kt_get_type should return const char *.
The code returns a pointer to static structures. Just to enforce the
assumption that users can not change the returned data.
Change prototype of krb5_kt_get_type to return const char *. The other
changes are to clean up warnings - no change in code - usage assumed const.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19143
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sun, 4 Feb 2007 03:08:47 +0000 (03:08 +0000)]
Add a new program to perform various tests on the WRFILE: and MEMORY: keytabs
I developed this program to test functionality of the MEMORY keytab - which
resulted in the numerous fixes that have been committed recently.
Tests all functioanlity of keytabs except for krb5_kt_default() and
krb5_kt_read_service_key() - although essential functionality tested.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19142
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sun, 4 Feb 2007 02:49:54 +0000 (02:49 +0000)]
MEMORY keytab krb5_kt_get_entry sets enctype to 0 if unspecified
If one invokes krb5_kt_get_entry() for a memory keytab w/ an unspecified
enctype (i.e. 0) - the returned keytab has it's enctype set to 0 as well.
Also - in copying out the found keytab_entry - when a kvno is unspecified,
the last match found is used - not the one with the highest vno.
This was caused by copying out the variable "entry" and not "match".
Ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19141
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sun, 4 Feb 2007 02:03:48 +0000 (02:03 +0000)]
MEMORY keytab does not copy keytab_entry keyblock contents
In krb5_kt_add_entry: The MEMORY keytab does not make a copy of the
keytab_entry keyblock contents - but instead retains a pointer to the
incomming one.
In krb5_kt_get_entry and krb5_kt_get_next - a pointer to internal
keyblock contents memory is returned to the caller - which is subsequently
freed when tht caller invokes krb5_free_keytab_entry_contents.
Solution is to use krb5_copy_keyblock_contents() instead of simply copying
the structure.
Ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19140
dc483132-0cff-0310-8789-
dd5450dbe970
Russ Allbery [Sat, 3 Feb 2007 18:33:23 +0000 (18:33 +0000)]
The default K4 compatibility mode is now none, not preauth. Also document
that the valid values for v4_mode are the valid arguments to the -4 flag
to krb5kdc.
Ticket: 2724
Component: krb5-doc
Target_Version: 1.6.1
Tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19139
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sat, 3 Feb 2007 18:09:56 +0000 (18:09 +0000)]
Memory leak in new memory keytab
The memory keytab introduced in ticket 5411 suffers from a memory leak
in the Heimdal compatible kt_close() handling and thread destructor in which
the cursor->entry is not freed.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19138
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Thu, 1 Feb 2007 19:17:31 +0000 (19:17 +0000)]
buffer overflow in krb5_kt_get_name
krb5_kt_get_name() allows the called to specify the size of the buffer to copy
the name into. The size must be big enough for the tailing nul character.
If one specified a buffer length that is precisely the strlen w/o allowing for
the nul - the functions would copy one past the end of the buffer.
No code in our tree would be subject this problem - as buffers in use are 1024
or BUFSIZ....
The logic failure was:
strlen(p+1) vs. strlen(p)+1
The code is essentially duplicated in the three changed files.
Ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19137
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Thu, 1 Feb 2007 01:05:41 +0000 (01:05 +0000)]
Use krb5.h now for kdb errors, not kdb5_err.h
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19136
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Thu, 1 Feb 2007 00:09:07 +0000 (00:09 +0000)]
Don't need to initialize error tables
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19133
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Thu, 1 Feb 2007 00:02:36 +0000 (00:02 +0000)]
Don't need to register WRFILE: keytab type, it's already in the compiled-in
list.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19132
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Wed, 31 Jan 2007 00:53:11 +0000 (00:53 +0000)]
Fix merge botches:
restore krb5_get_init_creds_opt_set_change_password_prompt()
undo accidental reversion of preauth_tryagain change to
krb5_get_init_creds()
ticket: 5420
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19131
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Wed, 31 Jan 2007 00:52:59 +0000 (00:52 +0000)]
remove krb5_get_init_creds_opt_set_pkinit() for now
ticket: 5420
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19130
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Tue, 30 Jan 2007 23:37:36 +0000 (23:37 +0000)]
sort, remove duplicates
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19129
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Tue, 30 Jan 2007 21:40:20 +0000 (21:40 +0000)]
update def-check to look at preauth_plugin.h
ticket: 5420
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19128
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Tue, 30 Jan 2007 21:38:47 +0000 (21:38 +0000)]
get_init_creds_opt extensibility
r18922@cathode-dark-space: coffman | 2006-12-04 18:30:15 -0500
First cut at making the get_init_creds_opt structure extendable
and adding library functions to set options for preauthentication
plugins.
This does *not* include a compatibility function to work like
Heimdal's krb5_get_init_creds_opt_set_pkinit() function.
Hopefully, the test code that doesn't belong in kinit.c is
obvious.
r18929@cathode-dark-space: coffman | 2006-12-07 10:01:20 -0500
Remove extra "user_id" parameter.
Add function which duplicates the Heimdal interface (if we can agree on
what the matching attribute names should be).
r18934@cathode-dark-space: coffman | 2006-12-08 15:28:03 -0500
Update to use the simplified interface for krb5_get_init_creds_opt_set_pa()
Add code in kinit to process "-X" options as preauth options and pass
them along.
r18936@cathode-dark-space: coffman | 2006-12-11 12:04:26 -0500
Move prototypes for get_init_creds_opt_get_pa() and
krb5_get_init_creds_opt_free_pa() into the
preauth_plugin.h header rather than krb5.hin.
ticket: new
status: open
component: krb5-libs
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19127
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Tue, 30 Jan 2007 17:21:56 +0000 (17:21 +0000)]
This commit adds a thread safe MEMORY keytab implementation
that is compatible with Heimdal 0.7. Each successful resolve
returns a handle to a keytab and increases the internal
reference count. Each close invalidates the handle and
decreases the reference count. When the reference count hits
zero, the keytab is destroyed.
When a kt_destroy function is added, the automatic destruction
after close behavior will be removed.
ticket: 5411
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19126
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Tue, 30 Jan 2007 11:23:59 +0000 (11:23 +0000)]
Conditionalize ssize_t declaration
ticket: 5419
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19125
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Tue, 30 Jan 2007 11:22:45 +0000 (11:22 +0000)]
The fix for the command line race conditions
broke the ability to cancel and restart the
Obtain New Credentials dialog
ticket: 5414
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19124
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Tue, 30 Jan 2007 11:17:35 +0000 (11:17 +0000)]
Microsoft Windows Visual Studio does not define ssize_t
Microsoft defines size_t but not ssize_t.
Provide a definition in win-mac.h that follows the style
used by Microsoft providing a preprocessor macro
SSIZE_T_DEFINED which can be used for conditional
definition and testing in case Microsoft decides to add
it later.
ticket: new
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19123
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Tue, 30 Jan 2007 11:02:32 +0000 (11:02 +0000)]
KFW: 32-bit builds use the pismere krbv4w32.dll library
Only the 32-bit builds should use the pismere krbv4w32.dll
library. Pismere does not provide 64-bit support.
ticket: new
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19122
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 27 Jan 2007 08:46:09 +0000 (08:46 +0000)]
sort
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19119
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 27 Jan 2007 02:41:29 +0000 (02:41 +0000)]
sort
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19114
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Wed, 24 Jan 2007 00:09:13 +0000 (00:09 +0000)]
NIM Bug Fixes
Document User Interface Callbacks
Fix a race condition when performing renewal actions
triggered by command line parameters.
When importing credentials, kickoff a renewal after
the credentials after the API: ccache is created.
Another fix for identity expiration states. This one
fixes the behavior of the system tray icon.
ticket: new
component: windows
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19110
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Tue, 23 Jan 2007 23:15:36 +0000 (23:15 +0000)]
ideas for passing error msg info across kadm5 api
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19109
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Tue, 23 Jan 2007 20:44:06 +0000 (20:44 +0000)]
Add const to variable to cleanup warnings
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19108
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Tue, 23 Jan 2007 20:40:58 +0000 (20:40 +0000)]
Include k5-int.h for krb5_cc_register() prototype. Remove defintion of
krb5_cc_dfl_ops - as k5-int.h defines.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19107
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Tue, 23 Jan 2007 20:17:08 +0000 (20:17 +0000)]
Change second argument to krb5_cc_register to take a const krb5_cc_ops *. Makes
sense - when krb5_Xcc_ops currently defined as such - and assignment of value
assumes same.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19106
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Tue, 23 Jan 2007 02:57:32 +0000 (02:57 +0000)]
gss_krb5int_unseal_token_v3() - change bodysize argument to unsigned -
which is consistent with how it is invoked and used. Cleans up two
signed/unsigned warnings.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19105
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Tue, 23 Jan 2007 01:18:17 +0000 (01:18 +0000)]
kt_file.c memory leak on error in krb5_kt_resolve / krb5_kt_wresolve
The krb5_kt_resolve and krb5_kt_wresolve functions leak the 'data'
memory allocation if the lock cannot be initialized.
ticket: new
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19104
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Mon, 22 Jan 2007 14:32:15 +0000 (14:32 +0000)]
Include fake-addrinfo.h before krb5.h to get proper structure declarations
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19103
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sun, 21 Jan 2007 19:15:02 +0000 (19:15 +0000)]
Only declare cc_fcc_entry if USE_KEYRING_CCACHE is not defined
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19102
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Sat, 20 Jan 2007 22:21:13 +0000 (22:21 +0000)]
small fixes
ticket: 5408
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19101
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Sat, 20 Jan 2007 21:10:10 +0000 (21:10 +0000)]
Move files to their correct location
ticket: 5408
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19100
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 20:30:36 +0000 (20:30 +0000)]
Use intermediate target to prevent parallel invocations of 'make includes' in lib/krb4
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19099
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 19:11:34 +0000 (19:11 +0000)]
Use move-if-changed and a timestamp file to avoid re-running commands and
updating krb5.h when there are no textual changes.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19098
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 18:47:32 +0000 (18:47 +0000)]
Don't copy headers to .../include
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19097
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 18:44:55 +0000 (18:44 +0000)]
make depend
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19096
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 18:42:06 +0000 (18:42 +0000)]
Build krb5.h using the _err.h files from the error_tables directory, not
additional copies of them.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19095
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 18:32:03 +0000 (18:32 +0000)]
Don't explicitly include error table headers that are also in krb5.h
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19094
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 18:06:27 +0000 (18:06 +0000)]
Mark as C source for Emacs mode selection
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19093
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 17:23:36 +0000 (17:23 +0000)]
Move osconf input file to include directory instead of keeping it in its own
subdirectory.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19092
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 17:05:02 +0000 (17:05 +0000)]
* Makefile.in (osconf.h): Use move-if-changed script
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19091
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 17:04:25 +0000 (17:04 +0000)]
* pre.in (MOVEIFCHANGED): New variable
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19090
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 16:56:32 +0000 (16:56 +0000)]
Remove unused file
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19089
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 16:54:37 +0000 (16:54 +0000)]
Remove varargs.h paths, always use stdarg.h
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19088
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 12:52:34 +0000 (12:52 +0000)]
Include stdarg.h, and don't conditionalize declarations of functions with
va_list arguments.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19087
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 12:33:00 +0000 (12:33 +0000)]
make depend
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19086
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 12:13:15 +0000 (12:13 +0000)]
Handle function names immediately preceded by "*", like "*strdup"
in k5-int.h+krb5.h.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19085
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 12:07:41 +0000 (12:07 +0000)]
Mark as C source for Emacs mode selection
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19084
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 12:05:22 +0000 (12:05 +0000)]
Don't define KRB5_PRIVATE, since it's not tested any more
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19083
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 11:33:36 +0000 (11:33 +0000)]
Move all KRB5_PRIVATE declarations from krb5.h(in) to k5-int.h, and drop the
KRB5_PRIVATE preprocessor test.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19082
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 11:29:07 +0000 (11:29 +0000)]
* Makefile.in (private-and-public-decls): New target file, created from
k5-int.h and krb5.h.
(verify-calling-conventions-krb5): Use it instead of krb5.h.
(clean-unix): Get rid of it.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19081
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 11:02:58 +0000 (11:02 +0000)]
Include k5-int.h instead of krb5.h when 'private' functions are needed
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19080
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 10:59:19 +0000 (10:59 +0000)]
* pre.in (.et.h, .et.c): Use temporary files in the current directory with
fixed names, instead of incorporating the shell pid.
* post.in (clean-unix): Delete the temporary files.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19079
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Sat, 20 Jan 2007 05:50:41 +0000 (05:50 +0000)]
KFW MSI installer corrections
Add missing registry keys and values for
Network Identity Manager plug-ins.
Correct short names.
ticket: new
tags: pullup
component: windows
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19078
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Sat, 20 Jan 2007 05:42:30 +0000 (05:42 +0000)]
NIM - Context sensitive system tray menu and more
Adds context sensitive menus for renew and destroy
to the system tray menu. Select either all identities
or one of the identities with credentials.
Increases the API to 1.1.9 and adds a new interactive
callback mechanism.
ticket: new
component: windows
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19077
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 04:49:02 +0000 (04:49 +0000)]
Delete t_cccursor on cleanup. Add t_cc and t_cccursor to dependencies
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19076
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 20 Jan 2007 04:45:20 +0000 (04:45 +0000)]
Add t_cccursor to UNIX tests
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19075
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Thu, 18 Jan 2007 23:24:13 +0000 (23:24 +0000)]
Added KDC timesyncing support to the CCAPI ccache backend for CCAPI v5 and
later. v5 is the first version of the CCAPI to support the kdc timesyncing
API.
ticket: 5403
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19071
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Thu, 18 Jan 2007 15:28:07 +0000 (15:28 +0000)]
In addition to setting the kpasswd port after
searching for kadmind host addresses we must also
set the socket type according to the request.
ticket: 5393
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19070
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Thu, 18 Jan 2007 12:43:58 +0000 (12:43 +0000)]
NIM Kerberos 5 Provider corrections
When validating a Kerberos 5 principal name, the request
to the KDC should not request forwardable, renewable, or
proxiable options as these may be blocked by policy and
will result in the return of an error.
Always treat the Kerberos 5 principal name as valid
unless the KDC returns an error that clearly indicates that
the principal name does not exist.
Use a MEMORY: ccache for temporary storage instead of an
API: ccache.
Initialize pointer values with NULL instead of 0.
ticket: new
tags: pullup
component: windows
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19069
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Thu, 18 Jan 2007 12:39:06 +0000 (12:39 +0000)]
NIM Correct Visual Identity Expiration Status
The visual status for the identities in NIM 1.1 was based
upon the highest alert status of any credential that was
associated with the identity. The correct behavior that is
now implemented is that the identity status should be based
solely upon the expiration state of the credentials obtained
by the identity provider.
For example, the Kerberos v5 identity provider will based the
identity expiration status on the initial TGT. Service tickets
with short lifetimes that expire do not prevent the acquisition
of additional service tickets. Therefore, the identity should
not be listed as expired.
ticket: new
tags: pullup
component: windows
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19068
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Thu, 18 Jan 2007 12:33:47 +0000 (12:33 +0000)]
NIM Kerberos v4 configuration dialog
The Kerberos v4 options for individual identities
was never wired. The controls were visible but they
did not do anything. Implement them now for NIM 1.2.
ticket: new
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19067
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Thu, 18 Jan 2007 12:31:23 +0000 (12:31 +0000)]
NIM string tables
Update the string tables for NIM so that they are consistent.
Always use "Kerberos v5" or "Kerberos v4". Refer to credentials
instead of tickets. Do not abbreviate "Network Identity Manager".
Etc.
ticket: new
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19066
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Thu, 18 Jan 2007 11:35:33 +0000 (11:35 +0000)]
sendto_kdc.c: use of a variable index into a dynamically
allocated array to determine the sizeof() an object makes
it unclear what type of object is involved. It also requires
a runtime check instead of a compile time replacement.
Not to mention that it could lead to the evaluation of an
uninitialized variable as was done in this case. Replace
sizeof(array index variable) with sizeof(type).
memset() the correct data structure.
ticket: 5394
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19065
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Wed, 17 Jan 2007 00:01:56 +0000 (00:01 +0000)]
* t_cccursor.c (do_chk): Reverse sense of test of do_chk_one return value
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19064
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Tue, 16 Jan 2007 23:29:46 +0000 (23:29 +0000)]
Fix typo in checked-in version
ticket: 5349
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19063
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Tue, 16 Jan 2007 04:18:02 +0000 (04:18 +0000)]
This commit adds two new functions, krb5_server_decrypt_ticket_keyblock
(private) and krb5_server_decrypt_ticket_keytab (public). These
functions take a krb5_ticket as input and decrypt it using the provided
key data. The public function is useful for higher level application
protocols such a TLS-KRB5 and AFS RX-KRB5 which exchange a service
but do not use the AP-REQ/AP-REP messages.
This commit also adds new functionality to kvno which permits kvno
when provided a keytab as input to verify whether or not the keytab
contains a key that can successfully decrypt the obtains service ticket.
ticket: 5349
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19062
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Sun, 14 Jan 2007 08:04:46 +0000 (08:04 +0000)]
kfw wix installer - memory overwrite error
The custom handler allocates a buffer that is smaller
than is required to hold the input. Allocate the correct
sized buffer.
ticket: new
tags: pullup
component: windows
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19061
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Sat, 13 Jan 2007 17:08:28 +0000 (17:08 +0000)]
Update to KFW NSIS installer
Update copyright date, samples directory, and shortcut
to NetIDMgr documentation
ticket: new
component: windows
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19060
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Sat, 13 Jan 2007 17:04:44 +0000 (17:04 +0000)]
updated Windows README
Revise the readme text for Windows. Remove references
to old beta SDKs.
ticket: new
tags: pullup
component: windows
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19059
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Thu, 11 Jan 2007 00:13:46 +0000 (00:13 +0000)]
krb5-1.7-prerelease
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19056
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Tue, 9 Jan 2007 19:45:25 +0000 (19:45 +0000)]
MITKRB5-SA-2006-003: mechglue argument handling too lax
Fix mechglue argument checks so that output pointers are always
initialized regardless of whether the other arguments fail to validate
for some reason. This avoids freeing of uninitialized pointers.
Initialize the gss_buffer_descs in ovsec_kadmd.c.
ticket: new
target_version: 1.6
tags: pullup
component: krb5-libs
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19043
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Tue, 9 Jan 2007 19:45:10 +0000 (19:45 +0000)]
MITKRB5-SA-2006-002: svctcp_destroy() can call uninitialized function pointer
Explicitly null out xprt->xp_auth when AUTH_GSSAPI is being used, so
that svctcp_destroy() will not call through an uninitialized function
pointer after code in svc_auth_gssapi.c has destroyed expired state
structures. We can't unconditionally null it because the RPCSEC_GSS
implementation needs it to retrieve state.
ticket: new
target_version: 1.6
tags: pullup
component: krb5-libs
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19042
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 8 Jan 2007 22:51:23 +0000 (22:51 +0000)]
create KDC database directory
The default directory for the KDC database files doesn't exist after a
normal "make install". Reported by Jeff Blaine.
* Makefile.in (INSTALLMKDIRS): Add var (really localstatedir) and var/krb5kdc
to directories to be created at "make install" time, even though it'll be
empty on most machines (since most probably aren't KDCs).
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19039
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 8 Jan 2007 22:41:01 +0000 (22:41 +0000)]
crash creating db2 database in non-existent directory
* kdb_db2.c (krb5_db2_db_create): If the creation of the first database file
fails, return the error, instead of attempting to create the second (and using
a null pointer as an input string in formatting a filename).
Reported by Jeff Blaine.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19038
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 6 Jan 2007 08:30:46 +0000 (08:30 +0000)]
* lib.in (clean-libs): Delete darwin.exports and hpux10.exports.
* libnover.in (clean-libs): Use $(DYNOBJEXT) instead of $(SHLIBEXT) for
extension on object to delete. Also delete darwin.exports and hpux10.exports.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19037
dc483132-0cff-0310-8789-
dd5450dbe970
Russ Allbery [Fri, 5 Jan 2007 21:42:38 +0000 (21:42 +0000)]
Document what the kadmind ACL is for
Add a sentence documenting the purpose of the kadmind ACL to the node
explaining how to create it.
Ticket: 5279
Component: krb5-doc
Version_Reported: 1.5.1
Target_Version: 1.6
Tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19036
dc483132-0cff-0310-8789-
dd5450dbe970
Russ Allbery [Thu, 4 Jan 2007 22:32:41 +0000 (22:32 +0000)]
Document KDC behavior without stash file
After the discussion of the optional stash file, document the effects of
not creating a stash file.
ticket: new
Component: krb5-doc
Version_Reported: 1.5.1
Target_Version: 1.6
Tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19035
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Wed, 3 Jan 2007 23:54:25 +0000 (23:54 +0000)]
* ldap_principal.c (attributes_set): Swap first two elements
Also add comments indicating that this array and the KDB_*_ATTR macros
need to be in sync.
ticket: 5260
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19031
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Wed, 3 Jan 2007 23:15:55 +0000 (23:15 +0000)]
update ldap/Makefile.in for newer autoconf substitution requirements
The other makefile.in files have had the makefile-fragment
substitution lines updated to not have "#" at the front, because some
recent versions of autoconf require that the @-pattern start at the
beginning of the line. We missed plugins/kdb/ldap/Makefile.in at the
time.
Patch from Michael Calmer.
ticket: new
target_version: 1.6
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19030
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Wed, 3 Jan 2007 22:53:33 +0000 (22:53 +0000)]
* Makefile.in (install): Install kdb5_ldap_util.M. Based on patch from
Michael Calmer.
ticket: 3906
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19029
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Tue, 2 Jan 2007 23:53:36 +0000 (23:53 +0000)]
* kdc_util.h (CONVERT_INTO_DB, CONVERT_OUTOF_DB): Unused macros deleted
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19025
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sun, 31 Dec 2006 18:10:27 +0000 (18:10 +0000)]
Memory leak in tests/gssapi/t_imp_name.c
Memory leak by not releasing name_oid from gss_display_name().
(conditional on GSSAPI_V2 being defined).
ticket:new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19024
dc483132-0cff-0310-8789-
dd5450dbe970
Jeffrey Altman [Sun, 31 Dec 2006 16:24:54 +0000 (16:24 +0000)]
Repair broken links in NetIdMgr Help
A small number of links contained the wrong root directory.
ticket: new
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19023
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sat, 30 Dec 2006 06:09:25 +0000 (06:09 +0000)]
If gss_krb5int_unseal_token_v3() unwraps a message of length 0 - free
memory and return in message_buffer a NULL pointer for value. This
is consistant with gss_release_buffer in the mechglue implementation in which
memory is only freed if the buffer length != 0.
ticket: 5233
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19022
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sat, 30 Dec 2006 06:05:12 +0000 (06:05 +0000)]
memory leak if defective header present in gss_krb5int_unseal_token_v3
If after unsealing the message, the TOK_ID is not 05 04, free memory
before returning a defective token error.
ticket: new
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19021
dc483132-0cff-0310-8789-
dd5450dbe970
Russ Allbery [Thu, 28 Dec 2006 18:05:28 +0000 (18:05 +0000)]
Fix typo in user-guide.texinfo
Typo fix (network instead of netword). Thanks, Matt Zagrabelny.
Ticket: new
Component: krb5-doc
Version_Reported: 1.4.4
Target_Version: 1.6
Tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19020
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Thu, 28 Dec 2006 02:05:20 +0000 (02:05 +0000)]
* copy_data.c (krb5_copy_data): Use krb5int_copy_data_contents
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19019
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Thu, 28 Dec 2006 02:01:12 +0000 (02:01 +0000)]
* cc_memory.c (krb5_mcc_next_cred): Use krb5int_copy_creds_contents
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19018
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Thu, 28 Dec 2006 01:51:02 +0000 (01:51 +0000)]
* lib/krb5/krb/copy_creds.c (krb5int_copy_creds_contents): New function, split
out from krb5_copy_creds.
(krb5_copy_creds): Call it.
* include/k5-int.h (krb5int_copy_creds_contents): Declare.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19017
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Thu, 28 Dec 2006 01:36:49 +0000 (01:36 +0000)]
comment the various data structures
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19016
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Fri, 22 Dec 2006 01:26:59 +0000 (01:26 +0000)]
Some related changes were already in, and I found a couple more to make:
* ldap_realm.c (ldap_filter_correct): Change string argument to char *. Delete
length argument, which was always strlen of the string argument, and compute
it locally, using size_t instead of (unsigned) int for length-related values.
Update all calls.
* ldap_realm.h (ldap_filter_correct): Updated declaration.
* ldap_misc.c (remove_overlapping_subtrees): Add forward declaration. Make
static.
(is_principal_in_realm): Change local variable defrealmlen to size_t.
(store_tl_data): Change local variable curr to point to unsigned char, since
that's what the tl_data_contents array is declared as, and what the STORE16_INT
macro is happier with.
(krb5_ldap_get_reference_count): Make local variable i unsigned.
ticket: 4453
target_version: 1.6
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19009
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Fri, 22 Dec 2006 01:19:55 +0000 (01:19 +0000)]
* src/lib/krb5/krb/get_in_tkt.c (krb5_get_init_creds): Fix
ordering bug in previous patch.
ticket: 5123
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19007
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Thu, 21 Dec 2006 21:58:10 +0000 (21:58 +0000)]
don't split HTML output from makeinfo
* doc/Makefile (HTML): To avoid generating excessively long
filenames, don't split HTML output.
ticket: new
target_version: 1.6
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19002
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Wed, 20 Dec 2006 21:40:20 +0000 (21:40 +0000)]
misc cleanups in admin guide ldap sections
There are a bunch of instances of incorrect punctuation, inconsistent
use of @-commands with option names, typos in names of principal
flags, and a couple spelling errors. I only fixed what I noticed; I
haven't subjected the rest to careful review.
Also, the long section names for eDirectory-specific documentation
cause the tar files generated for snapshots (which include generated
html docs) to reach the 100-character limit for file names in
traditional tar format; GNU tar can create archives holding them, but
older tar implementations cannot read the archives properly. So,
several eDirectory-related section names have been shortened.
ticket: new
target: 1.6
tags: pullup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19001
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Wed, 20 Dec 2006 21:12:35 +0000 (21:12 +0000)]
Merge r18962 to trunk, with minor tweaks; ready to merge to 1.6 branch
Changes fix up some sample names used, remove some options described
from certain commands, and fix filling in man pages.
ticket: 5116
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@19000
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Tue, 19 Dec 2006 04:16:30 +0000 (04:16 +0000)]
use KRB5KRB_ERR_GENERIC, not KRB_ERR_GENERIC in preauth2.c
* src/lib/krb5/krb/preauth2.c (krb5_do_preauth_tryagain): Use
KRB5KRB_ERR_GENERIC, not KRB_ERR_GENERIC.
ticket: new
target_version: 1.6
tags: pullup
component: krb5-libs
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@18977
dc483132-0cff-0310-8789-
dd5450dbe970