Ken Raeburn [Mon, 24 Nov 2008 19:19:14 +0000 (19:19 +0000)]
Fix memory leak caught by valgrind
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21179
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 17 Nov 2008 22:19:26 +0000 (22:19 +0000)]
Note kprop doesn't do IPv6 yet either
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21135
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 17 Nov 2008 21:34:48 +0000 (21:34 +0000)]
Set krb4 and priocntl flags via site.exp instead of command line
Update set of files to clean out at test suite startup, or when
deleting the database.
Improve support for setting up slave test environment. Don't set
KRB5_KDC_PROFILE except in KDC master and slave environments. Create
distinct env.sh and env.csh files for different configurations. Move
kpropd setup proc into common initialization.
Add incremental propagation test: Create new kiprop/$host principal,
update kproplog test for the new data. "Propagate" the master
database to the slave, add a new principal, start up kpropd, watch for
the "OK" message, and check to see if the new principal exists on the
slave.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21134
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 17 Nov 2008 21:28:04 +0000 (21:28 +0000)]
Fix a memory management bug
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21133
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 17 Nov 2008 21:24:29 +0000 (21:24 +0000)]
Don't stick '<null>' in log messages. Run kdb5_util dump with input explicitly from /dev/null
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21132
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 17 Nov 2008 21:14:03 +0000 (21:14 +0000)]
Log more info if ulog_replay fails. Correctly use supplied keytab name
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21131
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 17 Nov 2008 21:12:01 +0000 (21:12 +0000)]
Use correct update log file name in messages
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21130
dc483132-0cff-0310-8789-
dd5450dbe970
Zhanna Tsitkov [Mon, 17 Nov 2008 21:04:06 +0000 (21:04 +0000)]
Lite Client - the following calls are server-side functions:
decode_krb5_authenticator,
krb5_auth_con_getauthenticator,
krb5_copy_authenticator,
krb5_ser_authenticator_init
Take them out for the Lite CLient.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21129
dc483132-0cff-0310-8789-
dd5450dbe970
Zhanna Tsitkov [Mon, 17 Nov 2008 19:28:24 +0000 (19:28 +0000)]
PERF: Introduced a new function krb5_is_permitted_enctype_ext to replace multiple calls to krb5_is_permitted_enctype
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21128
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Wed, 12 Nov 2008 22:11:43 +0000 (22:11 +0000)]
Fix test bug: kpropd -S exits after success
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21079
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Wed, 12 Nov 2008 21:49:35 +0000 (21:49 +0000)]
Add basic kprop test, and a little more debugging support
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21078
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 10 Nov 2008 23:34:50 +0000 (23:34 +0000)]
Delete replay cache before exiting with success indication
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21068
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 10 Nov 2008 22:43:21 +0000 (22:43 +0000)]
Generate separate master-KDC and slave-KDC config files, with
different names for the database files. (Slave config files unused as
yet.) Ensure that the master-KDC environment is used when running
kadmin.local or kdb5_util. Define and use a new proc for deleting all
KDC database and keytab files.
Set KPASSWD in default.exp.
Run kadmin, pwchange, pwhist, gssftp, telnet, v4gssftp tests only once each.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21067
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Mon, 10 Nov 2008 18:45:17 +0000 (18:45 +0000)]
Fix a krsh bug introduced in r21001 which could cause garbage at the
beginning of the command string.
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21065
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 10 Nov 2008 18:27:42 +0000 (18:27 +0000)]
Delete the pass (one of twelve) that does all KDC exchanges with TCP;
add a test case that sets that up and runs kinit.
Add a new support proc that allows running a test only once despite
multiple passes; use it for the tcp and iprop tests.
According to one totally unscientific measurement, this reduces the
tests/dejagnu tests run from ~4200 to ~3800, and cuts over 8% off the
run time of those tests, without IMNSHO reducing the effectiveness of
the testing.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21064
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Mon, 10 Nov 2008 17:18:56 +0000 (17:18 +0000)]
In pa_sam(), free the outer krb5_data structure returned by
encode_krb5_sam_response.
ticket: 6211
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21063
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Mon, 10 Nov 2008 16:59:59 +0000 (16:59 +0000)]
Properly free sam_challenge in pa_sam()
ticket: 6210
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21062
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Fri, 7 Nov 2008 23:51:42 +0000 (23:51 +0000)]
Re-add repaired iprop (really, update-log creation) test case
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21044
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Fri, 7 Nov 2008 23:12:18 +0000 (23:12 +0000)]
Back out iprop test; it's not working right after all
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21043
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Fri, 7 Nov 2008 21:56:40 +0000 (21:56 +0000)]
Test that an iprop update log gets properly created and updated
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21042
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Fri, 7 Nov 2008 19:44:10 +0000 (19:44 +0000)]
fix typo in 'file delete' change
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21041
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Fri, 7 Nov 2008 00:15:50 +0000 (00:15 +0000)]
Use 'file delete' instead of spawning an 'rm' process
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21026
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Thu, 6 Nov 2008 23:54:16 +0000 (23:54 +0000)]
Recognize 'ticket expired' when trying to get a root shell
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21025
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Thu, 6 Nov 2008 15:49:00 +0000 (15:49 +0000)]
Fix a kadmin bug introduced in a recent set of string handling
conversions.
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21019
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Wed, 5 Nov 2008 17:47:00 +0000 (17:47 +0000)]
Only look for IPv4 addresses for the kpasswd server. This is just a
workaround for other parts of the code failing to cope with IPv6
addresses, and won't work in an IPv6-only environment; the problem
should still be fixed for real.
ticket: 5595
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21004
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Wed, 5 Nov 2008 17:08:47 +0000 (17:08 +0000)]
Rename krb5int_buf_cstr to krb5int_buf_data, since k5bufs can be used
for binary data as well as C string data. The buffer will always have
a null byte at krb5int_buf_len bytes regardless of whether it contains
C string data.
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21003
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Wed, 5 Nov 2008 16:47:24 +0000 (16:47 +0000)]
Add more svn:ignore entries for generated files
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21002
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Wed, 5 Nov 2008 16:19:01 +0000 (16:19 +0000)]
Convert many uses of strcpy/strcat (and sometimes sprintf) to accepted
string-handling functions.
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21001
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Wed, 5 Nov 2008 16:09:22 +0000 (16:09 +0000)]
Replace strcpy/strcat/sprintf uses in a couple of sample code files
with strncpy/strncat. Since this is sample code, we can't rely on
build system support for asprintf/strlcpy/strlcat.
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@21000
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Tue, 4 Nov 2008 01:50:02 +0000 (01:50 +0000)]
Check in Nalin's patch, and a test case for changing passwords via kinit
when +needchange is set. Update dependencies.
ticket: 5867
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20966
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Tue, 4 Nov 2008 01:46:06 +0000 (01:46 +0000)]
Don't build dependencies for fakeka.c
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20965
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Tue, 4 Nov 2008 01:19:59 +0000 (01:19 +0000)]
Don't build dependencies for v4rcp.c.
Rebuild dependencies for k5-buf.h, and without krb4 support.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20964
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Mon, 3 Nov 2008 22:50:08 +0000 (22:50 +0000)]
Add _with_password credential acquisition functions to KIM API
Needed for kinit password option.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20963
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Mon, 3 Nov 2008 22:47:10 +0000 (22:47 +0000)]
BIND_8_COMPAT no longer needed in Leopard
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20962
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Mon, 3 Nov 2008 21:05:25 +0000 (21:05 +0000)]
Apply Apple patch to null out key->contents after freeing on failure,
eliminating the possibility that the pointer will be used after free.
ticket: 6247
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20961
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 3 Nov 2008 20:50:42 +0000 (20:50 +0000)]
Fix conditionals from last change
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20960
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 3 Nov 2008 20:48:04 +0000 (20:48 +0000)]
Maybe include printf.o in t_k5buf build
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20959
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Mon, 3 Nov 2008 19:47:40 +0000 (19:47 +0000)]
Apply a patch from Apple to correct a few memory leaks
ticket: 6201
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20958
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Mon, 3 Nov 2008 18:41:33 +0000 (18:41 +0000)]
If we're not making asn1buf_insert_octet an inline function, then make
asn1buf_size, asn1buf_ensure_space, and asn1buf_expand static in
asn1buf.c, for better optimization.
Recode asn1buf_ensure_space to directly return the result of asn1buf_expand.
Don't check for NULL before malloc/realloc in asn1buf_expand.
Fix a couple minor signedness warnings.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20957
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Mon, 3 Nov 2008 18:22:18 +0000 (18:22 +0000)]
Apply an adapted Sun patch to fix error string leaks in kadmind
ticket: 6202
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20956
dc483132-0cff-0310-8789-
dd5450dbe970
Justin Anderson [Fri, 31 Oct 2008 19:37:18 +0000 (19:37 +0000)]
KerberosAgent bindings causing crashes
Attempting to fix a hard to reproduce bug with bindings by handling not applicable keys a little better.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20945
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 31 Oct 2008 19:35:22 +0000 (19:35 +0000)]
No . at end of strings
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20944
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Fri, 31 Oct 2008 18:35:28 +0000 (18:35 +0000)]
Add a few safeties to the k5buf code, to make static analysis tools happier
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20943
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 31 Oct 2008 17:11:47 +0000 (17:11 +0000)]
Avoid duplicate identical dialogs in KIM
Added KIM_DUPLICATE_UI_REQUEST_ERR error so UI can indicate it cancelled
its dialog due to getting multiple identical dialogs at the same time.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20942
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Thu, 30 Oct 2008 19:32:50 +0000 (19:32 +0000)]
Use the k5buf module instead of strcpy/strcat in several places
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20941
dc483132-0cff-0310-8789-
dd5450dbe970
Justin Anderson [Wed, 29 Oct 2008 21:58:56 +0000 (21:58 +0000)]
KerberosAgent now saves successfully authenticated identities to favorites automatically when they were acquired through the Enter Identity dialog
ticket: 6222
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20940
dc483132-0cff-0310-8789-
dd5450dbe970
Justin Anderson [Wed, 29 Oct 2008 21:14:39 +0000 (21:14 +0000)]
Clear auth prompt on success or cancel. Leave change password fields filled in when password is incorrect, to save user from reentering new password
ticket: 6221
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20939
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Wed, 29 Oct 2008 21:07:40 +0000 (21:07 +0000)]
Removed unnecessary code that was resetting options whenever the
array changes in the background. The problem is that any external
change to the ticket list will cause this to happen, even when the
options dialog is open.
Also removed unused function resetOptions.
ticket: 6224
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20938
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Wed, 29 Oct 2008 20:39:47 +0000 (20:39 +0000)]
Only prompt automatically from GUI apps
Direct callers such as kinit need command line prompts.
Do not automatically prompt (via krb5 or gssapi calls)
unless the caller has loaded GUI libraries.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20937
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Wed, 29 Oct 2008 20:36:49 +0000 (20:36 +0000)]
Remove saved password if it fails to get tickets
Note that if preauth is turned on the password may be removed for
other reasons. This is because preauth failing can mean several
things. Better to always remove it than have the user sometimes
get stuck though.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20936
dc483132-0cff-0310-8789-
dd5450dbe970
Justin Anderson [Wed, 29 Oct 2008 19:36:06 +0000 (19:36 +0000)]
KerberosAgent hangs changing pw for passwordless identities
Trying to change the password for an identity which only uses non-password authentication methods left KerberosAgent with a spinning progress indicator. Problem was with auth sheet not being ended.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20935
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Tue, 28 Oct 2008 22:43:49 +0000 (22:43 +0000)]
Cleanup memory leaks in test program for dynamically allocated buffers..
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20934
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Tue, 28 Oct 2008 22:09:12 +0000 (22:09 +0000)]
Update the exports file for krb5int_buf_add_fmt
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20933
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Tue, 28 Oct 2008 22:03:35 +0000 (22:03 +0000)]
In the k5buf module, add a function to append formatted data to a
buffer.
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20932
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Tue, 28 Oct 2008 20:28:52 +0000 (20:28 +0000)]
The last change to plugins.c erroneously passes a size_t as a field
width to asprintf. Address the signed/unsigned warning cleanup using
a cast instead.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20931
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Tue, 28 Oct 2008 20:21:50 +0000 (20:21 +0000)]
Eliminate use of strcpy/strcat/sprintf in wconfig.c. Use memcpy since
we cannot rely on libkrb5support to give us the good stuff. Also fix
up (to some extent) an assumption that size_t == int.
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20930
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Tue, 28 Oct 2008 15:34:29 +0000 (15:34 +0000)]
Add the k5buf string module to libkrb5support
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20929
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Tue, 28 Oct 2008 13:37:47 +0000 (13:37 +0000)]
Unsigned/signed warnings cleanup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20928
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Mon, 27 Oct 2008 21:01:00 +0000 (21:01 +0000)]
localize format strings, not final error string
errors.c should localize the incoming format string, not
the string produced by vasprintf. The format string is
constant and thus can be added to a localization table,
whereas the output string is not.
Note that this change depends on error_message also
localizing error table strings (which it does for KfM
already).
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20927
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Mon, 27 Oct 2008 19:01:12 +0000 (19:01 +0000)]
Remove empty declaration of struct _krb5_kt_ops after the fully defined structure
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20926
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Mon, 27 Oct 2008 18:59:08 +0000 (18:59 +0000)]
signed vs. unsigned warnings cleanup
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20925
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Mon, 27 Oct 2008 18:58:07 +0000 (18:58 +0000)]
Remove kproplog on make clean
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20924
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 25 Oct 2008 07:03:11 +0000 (07:03 +0000)]
partial rewrite of the ASN.1 encoders
Instead of a pile of macros generating code, that have to be threaded
together in just the right way to get a valid ASN.1 encoding, we now
have a pile of macros for defining data structures describing the
objects and the ASN.1 types they should be encoded as, which
structures are interpreted by recursive invocations of an encoder
engine; there should be somewhat less rope for accidentally creating
invalid encodings. The new macros are commented in asn1_k_encode.c.
Putting most of the work into the encoder engine also reduces the code
size (in one configuration, including LDAP-KDB and PKINIT encoders,
code size went from 37K to <16K, though 10K of tables were added, and
the PKINIT encoders are still open-coded).
Some encoder interfaces have been revised to be more regular -- all
now take one pointer to const argument (no two-input encoders, no
pointer-to-non-const-pointer-to-const). A few encoders were
eliminated or disabled because they were neither used nor exported
from the library.
The LDAP-KDB encoder has been converted, but the PKINIT encoders have
not as there are no regression tests for them currently.
There is still plenty of room for improvement; some notes on specific
ideas have been added.
String encoding primitives have been combined to reduce code size. A
primitive for encoding bit strings has been added.
Some miscellaneous warnings in the decoders have been cleaned up.
A new dejagnu test case is added that ensures that KRB-SAFE messages
get exercised.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20923
dc483132-0cff-0310-8789-
dd5450dbe970
Ken Raeburn [Sat, 25 Oct 2008 05:58:13 +0000 (05:58 +0000)]
More regression tests for ASN.1 encoders
Export encode_krb5_sam_response_2 and encode_krb5_enc_sam_response_enc_2
via accessor. Add encode tests for encode_krb5_sam_key,
_enc_sam_response_enc, _predicted_sam_response, _sam_response_2,
_enc_sam_response_enc_2.
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20922
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 24 Oct 2008 20:54:12 +0000 (20:54 +0000)]
krb5_build_principal_ext walks off beginning of array
On error, krb5_build_principal_ext walks off the beginning of the
array by using i-- in a conditional when it should be using --i
(so that it actually compares the value of i that will be used
below).
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20920
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Fri, 24 Oct 2008 20:07:00 +0000 (20:07 +0000)]
Use strlcpy instead of strcpy in many places
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20919
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 24 Oct 2008 19:46:41 +0000 (19:46 +0000)]
krb5_build_principal_va does not allocate krb5_principal
krb5_build_principal_va does not allocate the outer krb5_principal,
making it useless for generating krb5_principals which can be freed
with krb5_free_principal. Added krb5_build_principal_alloc_va which
allocates the krb5_principal.
Added krb5int_build_principal_alloc_va which is used by KIM to avoid
code duplication. KIM's kim_identity_create_from_components takes
the first component as an argument because principals with no
components cannot be represented with the KIM UI. Modified KIM
to use this new API.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20918
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 24 Oct 2008 19:40:35 +0000 (19:40 +0000)]
Add test for kim_identity_create_from_components
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20917
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Fri, 24 Oct 2008 17:12:00 +0000 (17:12 +0000)]
Add build system support for strlcpy and strlcat on platforms which do
not provide it natively.
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20916
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Fri, 24 Oct 2008 16:22:59 +0000 (16:22 +0000)]
Amend svn:ignore properties to include more generated files
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20915
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 24 Oct 2008 00:58:38 +0000 (00:58 +0000)]
Use CFStringGetCStringPtr if possible to avoid wasting memory
since CFStringGetMaximumSizeForEncoding is wasteful for UTF8.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20914
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 24 Oct 2008 00:57:10 +0000 (00:57 +0000)]
Add support for localizing krb5 errors on Mac OS X.
Split into a separate function to make it easier to add
support for other localization techniques.
ticket: 6245
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20913
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Thu, 23 Oct 2008 19:59:05 +0000 (19:59 +0000)]
Use snprintf instead of strcpy/strcat in many places
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20912
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Thu, 23 Oct 2008 17:50:43 +0000 (17:50 +0000)]
Free global mach ports on library unload
ticket: 6248
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20911
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Wed, 22 Oct 2008 21:17:07 +0000 (21:17 +0000)]
Fix previous commit by adding "extern" to header declarations for
SPNEGO mechanism OID stuff. It was causing tentative definition
issues on the Mac. (where there are constraints about common-block
symbols)
ticket: 6015
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20910
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Wed, 22 Oct 2008 20:41:34 +0000 (20:41 +0000)]
KIM should only display error dialogs if it has displayed UI already
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20909
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Wed, 22 Oct 2008 20:40:29 +0000 (20:40 +0000)]
Remove ipc message sent on cc_context_release
IPC message was causing problems when called from thread fini function
(via krb5_stdcc_shutdown).
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20908
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Tue, 21 Oct 2008 21:56:10 +0000 (21:56 +0000)]
kim_identity_change_password_with_credential leaks krb5_creds
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20907
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Tue, 21 Oct 2008 21:54:51 +0000 (21:54 +0000)]
kim_os_library_get_caller_name leaks file path
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20906
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Tue, 21 Oct 2008 21:54:00 +0000 (21:54 +0000)]
kim_ccache_iterator_next leaks principal
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20905
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Tue, 21 Oct 2008 21:52:29 +0000 (21:52 +0000)]
kim_preferences should free old identity before overwriting
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20904
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Tue, 21 Oct 2008 21:50:52 +0000 (21:50 +0000)]
Free data in tests so leaks checking is easier
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20903
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Tue, 21 Oct 2008 21:05:27 +0000 (21:05 +0000)]
krb5_change_set_password should free chpw_rep contents
ticket: 6214
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20902
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Mon, 20 Oct 2008 21:14:47 +0000 (21:14 +0000)]
Use asprintf instead of malloc/strcpy/strcat in many places
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20901
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Mon, 20 Oct 2008 20:11:33 +0000 (20:11 +0000)]
Apply adapted patch from bugs.debian.org/480434 to recurse into
SPNEGO creds when attempting to retrieve a mechanism cred.
ticket: 5807
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20900
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Mon, 20 Oct 2008 19:39:52 +0000 (19:39 +0000)]
Apply (adapted) patch from Apple to check for SPNEGO mechanism in
export_lucid_sec_ctx.
ticket: 6015
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20899
dc483132-0cff-0310-8789-
dd5450dbe970
Greg Hudson [Mon, 20 Oct 2008 18:18:49 +0000 (18:18 +0000)]
Use strdup in place of malloc/strcpy in many places
ticket: 6200
status: open
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20898
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sun, 19 Oct 2008 13:10:52 +0000 (13:10 +0000)]
Include k5-platform.h for SIZE_MAX
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20897
dc483132-0cff-0310-8789-
dd5450dbe970
Ezra Peisach [Sun, 19 Oct 2008 10:44:56 +0000 (10:44 +0000)]
In krb5_def_store_mkey(), mktemp was being invoked with a string WRFILE:....
This returns an error - as it actually tries to open the file.
Move some of the logic that points to the actual filename earlier - so
mktemp works on the .... portion.
Note that the netbsd linker gives a warning on using mktemp as it may be
insecure - but there is no obvious way to avoid it.
ticket: new
subhect: netbsd mktemp actually tries to open file
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20896
dc483132-0cff-0310-8789-
dd5450dbe970
Justin Anderson [Fri, 17 Oct 2008 22:08:12 +0000 (22:08 +0000)]
Make all dialogs beyond the change password sheet also appear as sheets
ticket:6145
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20894
dc483132-0cff-0310-8789-
dd5450dbe970
Tom Yu [Fri, 17 Oct 2008 22:05:32 +0000 (22:05 +0000)]
Adapted patch from Apple to work around SAMBA mech OID quirks and to
disable sending request flags.
ticket: 6016
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20893
dc483132-0cff-0310-8789-
dd5450dbe970
Justin Anderson [Fri, 17 Oct 2008 19:41:02 +0000 (19:41 +0000)]
Stop using -[NSWindow setIgnoresMouseEvents:] and manage enabled state of each control individually with bindings
ticket:6198
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20890
dc483132-0cff-0310-8789-
dd5450dbe970
Justin Anderson [Fri, 17 Oct 2008 19:26:29 +0000 (19:26 +0000)]
KerberosAgent continues to ignore mouse events after error
Problem when trying to change password from behind a NAT. The ensuing error dialog ignores mouse events, making it impossible to dismiss.
ticket:new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20889
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 17 Oct 2008 18:50:19 +0000 (18:50 +0000)]
temporary testing code
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20888
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 17 Oct 2008 18:34:51 +0000 (18:34 +0000)]
KLCreatePrincipalFromTriplet should work with empty instance
Fixed bug where shim layer was calling kim_identity_create_from_components
incorrectly in this case.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20887
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 17 Oct 2008 18:15:25 +0000 (18:15 +0000)]
Added back KLL test programs
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20886
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 17 Oct 2008 18:13:47 +0000 (18:13 +0000)]
Double free and leak in kim_os_library_get_application_path
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20885
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 17 Oct 2008 17:07:03 +0000 (17:07 +0000)]
Remap some of the more confusing krb5 errors
Also enlarged last error buffer for UTF8 strings with multibyte chars.
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20884
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 17 Oct 2008 17:05:31 +0000 (17:05 +0000)]
Treat unreadable terminal as user cancelled so regression tests work
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20883
dc483132-0cff-0310-8789-
dd5450dbe970
Alexandra Ellwood [Fri, 17 Oct 2008 17:04:09 +0000 (17:04 +0000)]
Make sure identity is set when using kim prompter
ticket: new
git-svn-id: svn://anonsvn.mit.edu/krb5/trunk@20882
dc483132-0cff-0310-8789-
dd5450dbe970