krb5.git
16 years agoremoved unused header file inclusion CoreFoundation.h
Alexandra Ellwood [Thu, 13 Mar 2008 17:44:22 +0000 (17:44 +0000)]
removed unused header file inclusion CoreFoundation.h

Was there for a workaround to a bug in AppleConnect.

ticket: new

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

16 years agoAccidentally added file with wrong name. Removed
Alexandra Ellwood [Wed, 12 Mar 2008 16:52:23 +0000 (16:52 +0000)]
Accidentally added file with wrong name.  Removed

ticket: 5909

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

16 years agoAccidentally added empty file. Removed
Alexandra Ellwood [Wed, 12 Mar 2008 16:51:25 +0000 (16:51 +0000)]
Accidentally added empty file.  Removed

ticket: 5909

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

16 years agoAdded tests for iterators and NC info. Added test programs for
Alexandra Ellwood [Wed, 12 Mar 2008 16:50:21 +0000 (16:50 +0000)]
Added tests for iterators and NC info.  Added test programs for
each test.  Cleaned up portability issues introduced by Windows
testing.

ticket: 5909

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

16 years agoAdd CCAPI v2 tests
Alexandra Ellwood [Mon, 10 Mar 2008 21:55:44 +0000 (21:55 +0000)]
Add CCAPI v2 tests

First pass at CCAPI v2 tests.  More commits to this bug will follow.

ticket: new

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

16 years agoRemove C warnings
Alexandra Ellwood [Mon, 10 Mar 2008 21:51:49 +0000 (21:51 +0000)]
Remove C warnings

Some C++ conventions in the CCAPI tests were producing warnings
on C compilers.  Fixed code to stop producing warnings.

ticket: new

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

16 years agoRemoved script to run cc_context_get_version test
Alexandra Ellwood [Mon, 10 Mar 2008 21:48:08 +0000 (21:48 +0000)]
Removed script to run cc_context_get_version test

ticket: 5907

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

16 years agoRemoved tests for check_cc_context_get_version
Alexandra Ellwood [Mon, 10 Mar 2008 21:41:50 +0000 (21:41 +0000)]
Removed tests for check_cc_context_get_version

cc_context_get_version was part of the original CCAPI v3 documentation but was
never actually implemented.  Now that it has been removed from the documentation
we should remove the test.

ticket: new

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

16 years agoFixed error code remapping
Alexandra Ellwood [Mon, 10 Mar 2008 21:37:56 +0000 (21:37 +0000)]
Fixed error code remapping

Modified error code remapping to be consistent with CCAPI v2 documentation.

ticket: new

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

16 years agocc_remove_cred should only remove one cred
Alexandra Ellwood [Mon, 10 Mar 2008 21:27:30 +0000 (21:27 +0000)]
cc_remove_cred should only remove one cred

Fixed cc_remove_cred so it only removes the first matching
cred.  Also fixed the error handling so it returns errors
other than CC_NOTFOUND.

ticket: new

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

16 years agocc_set_principal should return error on bad cred version
Alexandra Ellwood [Mon, 10 Mar 2008 19:32:47 +0000 (19:32 +0000)]
cc_set_principal should return error on bad cred version

When the caller passes in a version different than the one the ccache
was opened with, cc_set_principal should return CC_ERR_CRED_VERSION
because "cred_vers is used as a double check".

ticket: new

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

16 years agoFix pointer cast in cc_seq_fetch_NCs_end
Alexandra Ellwood [Mon, 10 Mar 2008 19:24:18 +0000 (19:24 +0000)]
Fix pointer cast in cc_seq_fetch_NCs_end

cc_seq_fetch_NCs_end incorrectly casts from CCAPI v2 to CCAPI v3 types.
Cleaned up cc_seq_fetch_creds_end at the same time (it was correct but
difficult to read).

ticket: new

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

16 years agocci_cred_union_compare_to_credentials_union doesn't work for v5 creds
Alexandra Ellwood [Mon, 10 Mar 2008 19:15:39 +0000 (19:15 +0000)]
cci_cred_union_compare_to_credentials_union doesn't work for v5 creds

Fixed a bug in cci_cred_union_compare_to_credentials_union where it always
returned an error when comparing v5 creds.

ticket: new

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

16 years agoCCAPI v2 support crash when client or server strings are NULL
Alexandra Ellwood [Mon, 10 Mar 2008 19:13:07 +0000 (19:13 +0000)]
CCAPI v2 support crash when client or server strings are NULL

The CCAPI v2 support will crash if passed in a krb5 credential with
the client or server principal strings set to NULL.  Since CCAPI v3+
support checks for this we should check in CCAPI v2.

ticket: new

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

16 years agoccs_ccache_reset should check all arguments for NULL
Alexandra Ellwood [Mon, 10 Mar 2008 19:07:54 +0000 (19:07 +0000)]
ccs_ccache_reset should check all arguments for NULL

ticket: new

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

16 years agoInclude signal.h before ftp_var.h to silence my_sig_t redefinition problem
Ken Raeburn [Tue, 4 Mar 2008 23:29:30 +0000 (23:29 +0000)]
Include signal.h before ftp_var.h to silence my_sig_t redefinition problem

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

16 years agoFix memory leak by delaying instantiation of lid until it's needed
Justin Anderson [Tue, 4 Mar 2008 20:27:19 +0000 (20:27 +0000)]
Fix memory leak by delaying instantiation of lid until it's needed

ticket: 5897
tags: pullup

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

16 years agoAdd "-framework CoreFoundation" to crypto library link command in darwin-mode
Ken Raeburn [Tue, 4 Mar 2008 19:25:12 +0000 (19:25 +0000)]
Add "-framework CoreFoundation" to crypto library link command in darwin-mode
build on mac.

ticket: 5894
status: open

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

16 years agolib/win/Makefile.in: Added
Kevin Koch [Fri, 29 Feb 2008 19:08:06 +0000 (19:08 +0000)]
lib/win/Makefile.in:  Added

ccutils.c, ccapi_os_ipc.cxx, ccs_reply_proc.c, ccs_os_server.cpp, ccs_reply_proc.c, ccs_win_pipe.c:  comment out some debug messages.

test/Makefile.in: change which tests are built.
test/test_ccapi_context.c: API version 2 is now expected to pass.
test/test_ccapi_iterators.c:  Add progress indicator for long tests.

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594
Tags:  pullup

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

16 years agoMake a NUL-terminated copy of realm name before passing to a plugin
Tom Yu [Fri, 29 Feb 2008 05:23:56 +0000 (05:23 +0000)]
Make a NUL-terminated copy of realm name before passing to a plugin
interface that takes a C string rather than krb5_data.

ticket: 5893
tags: pullup
target_version: 1.6.4

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

16 years agokrb5int_arcfour_string_to_key does not support utf-8 strings
Alexandra Ellwood [Tue, 26 Feb 2008 22:18:15 +0000 (22:18 +0000)]
krb5int_arcfour_string_to_key does not support utf-8 strings

krb5int_arcfour_string_to_key converts C strings to UTF-16 before passing
them to the string to key function.  Currently the UTF-16 conversion assumes
the input string is ASCII only.

Added support to convert UTF-8 strings to UTF-16 on Mac OS X.  Leaving the
bug open until we discuss if we want to have implementations for Unix and
Windows platforms.

ticket: new
status: open
target_version: 1.7

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

16 years agoAdd some ignored files, for in-tree builds
Ken Raeburn [Sun, 24 Feb 2008 12:42:25 +0000 (12:42 +0000)]
Add some ignored files, for in-tree builds

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

16 years agoRemove stop-here target that stops the build
Kevin Koch [Tue, 19 Feb 2008 18:36:06 +0000 (18:36 +0000)]
Remove stop-here target that stops the build

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594
Tags:  pullup

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

16 years agoChanges to integrate the CCAPI build into the build structure, build the test suite...
Kevin Koch [Tue, 19 Feb 2008 15:22:13 +0000 (15:22 +0000)]
Changes to integrate the CCAPI build into the build structure, build the test suite and fixes to random problems discovered along the way

Since no platform other than windows builds CCAPI using the build system, some conditionalizing may be necessary when other platforms use the makefiles.

src/Makefile.in:  Add CPPFLAGS that seemed to be missing; run wconfig for ccapi/(lib, server, test).

config/win-pre.in:  DEBUGOPT /ZI doesn't seem to provide enough debugging information under VS2005; /Zi does.

windows/build/bkw.pl:  Fix -no<switch> so that -nonodebug will work.  Otherwise, can't do debug build.

Move Get/PutTspData out of dllmain; add tlsindex argument.

Comment out some debug messages.

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594
Tags:  pullup

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

16 years agoman page macro and hyphen fixes
Russ Allbery [Tue, 19 Feb 2008 04:49:11 +0000 (04:49 +0000)]
man page macro and hyphen fixes

Fix various unescaped hyphens, lines starting with . that shouldn't be
macros, undefined strings, and misspelled macros in the man pages.
Found via man --warnings on a current Debian unstable system.

ticket: new
component: krb5-doc
Version_Reported: 1.6.3
Target_Version: 1.6.4
Tags: pullup

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

16 years agomore tests for libdb btree page split on zero index
Tom Yu [Thu, 7 Feb 2008 07:07:06 +0000 (07:07 +0000)]
more tests for libdb btree page split on zero index

Enhance btree debugging output somewhat to limit key printout to the
key length if the key is not null-terminated.

Add additional test case for the zero-index page split bug; test case
can create a corrupted btree database with records unreachable by
random access but reachable by sequential access.  Requires
recompiling with CPPFLAGS='-DDEBUG -DDEBUG_IDX0SPLIT' to correctly
model mpool page reuse that would be present in production conditions.
(CPPFLAGS=-DDEBUG would otherwise explicitly overwrite the contents of
reused pages.)

ticket: new
target_version: 1.6.4
tags: pullup
component: krb5-kdc

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

16 years agoRemove AppleConnect workaround
Alexandra Ellwood [Wed, 6 Feb 2008 20:24:19 +0000 (20:24 +0000)]
Remove AppleConnect workaround

AppleConnect fixed their bug and this workaround can be removed.

ticket: new

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

16 years agoNeed CCAPI v2 support for Windows
Alexandra Ellwood [Wed, 6 Feb 2008 20:22:32 +0000 (20:22 +0000)]
Need CCAPI v2 support for Windows

Added initial checkin of CCAPI v2 shim layer.

ticket: new
status: open

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

16 years agotools for svn:eol-style fixups
Ken Raeburn [Tue, 5 Feb 2008 18:34:10 +0000 (18:34 +0000)]
tools for svn:eol-style fixups

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

16 years agomore eol-style
Ken Raeburn [Tue, 5 Feb 2008 18:33:20 +0000 (18:33 +0000)]
more eol-style

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

16 years agoset eol-style
Ken Raeburn [Tue, 5 Feb 2008 18:25:28 +0000 (18:25 +0000)]
set eol-style

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

16 years agoSet EOL style
Alexandra Ellwood [Tue, 5 Feb 2008 18:18:41 +0000 (18:18 +0000)]
Set EOL style

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

16 years agolibdb btree page split on zero index corrupts db
Tom Yu [Fri, 1 Feb 2008 01:03:11 +0000 (01:03 +0000)]
libdb btree page split on zero index corrupts db

Splitting a btree page on index 0 can corrupt the database if the key
length plus data length is exactly a certain value.  This certain size
causes the item to get the left page to itself, and causes the right
page to contain an erroneous additional index "hole" having an
uninitialized value.  This bug may be one of the remaining causes of
unexplained database corruption reported over the years.  Shawn Emery
provided useful data from actual instances of this corruption.

Add a test case for this bug.  (Raw libdb test rather than kdb; the
latter would be much harder.)

ticket: new
target_version: 1.6.4
tags: pullup
component: krb5-kdc

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

16 years agoActually pass the nmake arguments to nmake
Kevin Koch [Thu, 31 Jan 2008 15:51:18 +0000 (15:51 +0000)]
Actually pass the nmake arguments to nmake

TargetVersion: 1.7
Component: windows
Ticket: new
Subj: Build automation
Tags:  pullup

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

16 years agoCCAPI changes to the mac-specific code to use new kipc changes
Alexandra Ellwood [Tue, 29 Jan 2008 22:39:25 +0000 (22:39 +0000)]
CCAPI changes to the mac-specific code to use new kipc changes

ticket: 5878
status: open

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

16 years agoWindows: avoid use of cygwin mkdir and rmdir commands
Jeffrey Altman [Wed, 23 Jan 2008 22:10:56 +0000 (22:10 +0000)]
Windows: avoid use of cygwin mkdir and rmdir commands

Microsoft's nmake versions 8.x and 9.x prefer executables over
internal shell commands.  This is a change from previous versions.
Cygwin's mkdir and rmdir commands do not have the same semantics
as the cmd.exe shell versions.

Change the definitions of MKDIR and RMDIR to use 'md' and 'rd'
in order to avoid the use of the cygwin versions.

ticket: new
tags: pullup

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

16 years agoUpdate build instructions to indicate PATH ordering; add troubleshooting tip about...
Kevin Koch [Wed, 23 Jan 2008 21:26:49 +0000 (21:26 +0000)]
Update build instructions to indicate PATH ordering; add troubleshooting tip about nmake versions

Target_Version: 1.7
Ticket: 5859
Tags: pullup

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

16 years agoCorrect makefile so pingtest builds
Kevin Koch [Tue, 22 Jan 2008 20:41:48 +0000 (20:41 +0000)]
Correct makefile so pingtest builds

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594

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

16 years agoWindows CCAPI snapshot. Should build & pass ping test
Kevin Koch [Tue, 22 Jan 2008 19:14:04 +0000 (19:14 +0000)]
Windows CCAPI snapshot.  Should build & pass ping test

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594

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

16 years agoWindows debug message line endings were already correct; revert cci_debugging.c.
Kevin Koch [Tue, 22 Jan 2008 18:34:26 +0000 (18:34 +0000)]
Windows debug message line endings were already correct; revert cci_debugging.c.
Declare cci_thread_init__auxinit instead of defining a new function.

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594

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

16 years agoMake windows debug message line endings match the Mac endings.
Kevin Koch [Tue, 22 Jan 2008 16:34:31 +0000 (16:34 +0000)]
Make windows debug message line endings match the Mac endings.
Windows needs k5-platform helper function declared explicitly.
Change definition of ccs_pipe_t for windows.
#define strdup -- it's now deprecated on windows.

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594

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

16 years agoAdd ccs_pipe_compare
Alexandra Ellwood [Tue, 15 Jan 2008 22:13:27 +0000 (22:13 +0000)]
Add ccs_pipe_compare

Added a pipe comparison function.  Forgot it when I was adding
support for non-integer pipe types (such as used by Windows).

ticket: new

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

17 years agoimprove debugging of ticket verification in ksu
Russ Allbery [Sun, 30 Dec 2007 06:39:22 +0000 (06:39 +0000)]
improve debugging of ticket verification in ksu

When ksu is built with debugging support and -D is used, print out the
principals being compared before doing the verification rather than
afterwards so that the principals will be printed when the verification
fails.

ticket: new
Component: krb5-appl
Version_Reported: 1.6.2

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

17 years agoWrap krb5_db_get_principal function with get_principal (which will
Ken Raeburn [Sat, 22 Dec 2007 03:12:50 +0000 (03:12 +0000)]
Wrap krb5_db_get_principal function with get_principal (which will
eventually release the global lock and reacquire it) and
get_principal_locked (which will retain the global lock), and change
callers to use the wrappers, so we can simplify some ugliness at the call
sites.

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

17 years agoImplement SWAP16 for Mac without compiler warning
Ken Raeburn [Sat, 22 Dec 2007 02:21:15 +0000 (02:21 +0000)]
Implement SWAP16 for Mac without compiler warning

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

17 years agoRename krb4_sendto to set_response since that's what it does
Ken Raeburn [Wed, 19 Dec 2007 22:33:46 +0000 (22:33 +0000)]
Rename krb4_sendto to set_response since that's what it does

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

17 years agoRemove unused args to krb4_sendto and unused global. Use byte-order macros defined...
Ken Raeburn [Wed, 19 Dec 2007 01:41:26 +0000 (01:41 +0000)]
Remove unused args to krb4_sendto and unused global.  Use byte-order macros defined elsewhere

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

17 years agoRemove excess parameters in formatting log calls
Ken Raeburn [Wed, 19 Dec 2007 01:20:55 +0000 (01:20 +0000)]
Remove excess parameters in formatting log calls

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

17 years agoMake some stuff static that we don't need accessible outside kerberos_v4.c
Ken Raeburn [Wed, 19 Dec 2007 00:51:14 +0000 (00:51 +0000)]
Make some stuff static that we don't need accessible outside kerberos_v4.c

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

17 years agoAdd the CCAPI design sketch to the new directory
Kevin Koch [Mon, 17 Dec 2007 15:21:58 +0000 (15:21 +0000)]
Add the CCAPI design sketch to the new directory

TargetVersion: 1.7
Component: krb5-libs
Ticket: new
Subj: Create doc directory

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

17 years agofix CVE-2007-5894: apparent uninit length in ftpd.c:reply()
Tom Yu [Fri, 14 Dec 2007 05:14:11 +0000 (05:14 +0000)]
fix CVE-2007-5894: apparent uninit length in ftpd.c:reply()

ticket: 5853
target_version: 1.6.4
tags: pullup

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

17 years agofix CVE-2007-5902: integer overflow in svcauth_gss_get_principal()
Tom Yu [Fri, 14 Dec 2007 05:01:23 +0000 (05:01 +0000)]
fix CVE-2007-5902: integer overflow in svcauth_gss_get_principal()

ticket: 5855
target_version: 1.6.4
tags: pullup

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

17 years agofix CVE-2007-5971: double-free in gss_krb5int_make_seal_token_v3()
Tom Yu [Fri, 14 Dec 2007 05:01:07 +0000 (05:01 +0000)]
fix CVE-2007-5971: double-free in gss_krb5int_make_seal_token_v3()

ticket: 5856
target_version: 1.6.4
tags: pullup

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

17 years agofix CVE-2007-5972: double fclose() in krb5_def_store_mkey()
Tom Yu [Fri, 14 Dec 2007 04:38:42 +0000 (04:38 +0000)]
fix CVE-2007-5972: double fclose() in krb5_def_store_mkey()

ticket: 5857
target_version: 1.6.4
tags: pullup

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

17 years agofix CVE-2007-5971: free of non-heap pointer in gss_indicate_mechs()
Tom Yu [Fri, 14 Dec 2007 04:38:28 +0000 (04:38 +0000)]
fix CVE-2007-5971: free of non-heap pointer in gss_indicate_mechs()

ticket: 5856
tags: pullup
target_version: 1.6.4

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

17 years agoRemove Mac-specific bit unused in KfM that causes test failures now
Ken Raeburn [Fri, 14 Dec 2007 02:48:52 +0000 (02:48 +0000)]
Remove Mac-specific bit unused in KfM that causes test failures now
under Darwin UNIXy builds.

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

17 years agoKFW: BUG: KRB5CRED: Set identity data before sending notification
Jeffrey Altman [Wed, 12 Dec 2007 22:32:19 +0000 (22:32 +0000)]
KFW: BUG: KRB5CRED: Set identity data before sending notification

Call tc_set_ident_data() before kcdb_credset_collect().  Make sure the
identity data is set before the credentials change notification is broadcast.

ticket: new
component: windows
tags: pullup

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

17 years agocopy correct key for lucid context acceptor_subkey
Kevin Coffman [Wed, 12 Dec 2007 14:10:33 +0000 (14:10 +0000)]
copy correct key for lucid context acceptor_subkey

Copy the correct key (acceptor_subkey) to lucid context's acceptor_subkey.

ticket: new
component: krb5-libs
Tags: pullup

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

17 years agoKFW BUG: WIX: 64-bit installer attempts to uninstall 32-bit NSIS
Jeffrey Altman [Wed, 12 Dec 2007 03:45:42 +0000 (03:45 +0000)]
KFW BUG: WIX: 64-bit installer attempts to uninstall 32-bit NSIS

The 64-bit MSI must examine Win64 registry keys and not the 32-bit registry
keys which is where the 32-bit NSIS installation will be detected.

ticket: new
component: windows
tags: pullup

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

17 years agoRemove no-longer-needed include of Kerberos.h
Kevin Koch [Mon, 3 Dec 2007 14:52:11 +0000 (14:52 +0000)]
Remove no-longer-needed include of Kerberos.h

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594

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

17 years agoConditionalize #pragma mark with TARGET_OS_MAC; remove any Windows #pragma disable
Kevin Koch [Wed, 21 Nov 2007 14:05:43 +0000 (14:05 +0000)]
Conditionalize #pragma mark with TARGET_OS_MAC; remove any Windows #pragma disable

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594

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

17 years agoDisable warnings about #pragma mark
Kevin Koch [Tue, 20 Nov 2007 16:46:22 +0000 (16:46 +0000)]
Disable warnings about #pragma mark

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594
Subj: Tweaks for 1.7 build on Windows.

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

17 years agoOn Windows, UINT64_MAX isn't defined; define it
Kevin Koch [Thu, 15 Nov 2007 14:14:15 +0000 (14:14 +0000)]
On Windows, UINT64_MAX isn't defined; define it

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594

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

17 years agoOn Windows, UINT64_MAX isn't defined; get it from k5-platform.h
Kevin Koch [Thu, 15 Nov 2007 14:11:31 +0000 (14:11 +0000)]
On Windows, UINT64_MAX isn't defined; get it from k5-platform.h

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594

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

17 years agoThe khm_show_main_window() function is no longer called
Jeffrey Altman [Wed, 14 Nov 2007 22:34:53 +0000 (22:34 +0000)]
The khm_show_main_window() function is no longer called
at startup with khm_nCmdShow == SW_SHOWMINIMIZED in order to
hide the main application by calling khm_hide_main_window().
Instead, the main application window is simply never shown.

As a result, khm_show_main_window() needs to respond to
khm_nCmdShow == SW_SHOWMINIMIZED not by hiding the window
but by changing the khm_nCmdShow state to SW_SHOW and then
calling ShowWindow().

This change will address the problem whereby "Show NIM Window"
had to be triggered twice by the user when the process
was started in a minimized state.

ticket: 5842

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

17 years agoInclude k5-platform.h
Ken Raeburn [Sun, 4 Nov 2007 04:37:16 +0000 (04:37 +0000)]
Include k5-platform.h

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

17 years agoUse 0 and 1 instead of relying on TRUE/FALSE being defined everywhere
Ken Raeburn [Thu, 1 Nov 2007 08:18:23 +0000 (08:18 +0000)]
Use 0 and 1 instead of relying on TRUE/FALSE being defined everywhere

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

17 years agoConditionalize "#pragma mark" on TARGET_OS_MAC.
Ken Raeburn [Thu, 1 Nov 2007 08:08:32 +0000 (08:08 +0000)]
Conditionalize "#pragma mark" on TARGET_OS_MAC.
Don't use "#pragma warning" when not on Windows.  (In fact, the just-added uses
shouldn't be needed if the above conditional is done right, so one of them was
deleted.)

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

17 years agoOn Windows, disable a pragma warning; add comment about what a pipe is
Kevin Koch [Wed, 31 Oct 2007 17:16:56 +0000 (17:16 +0000)]
On Windows, disable a pragma warning; add comment about what a pipe is

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594
Subj: Work on compiling the CCAPI test suite on Windows.

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

17 years agoSet GNU warning options in CFLAGS and CXXFLAGS separately, avoiding options
Ken Raeburn [Wed, 31 Oct 2007 11:37:57 +0000 (11:37 +0000)]
Set GNU warning options in CFLAGS and CXXFLAGS separately, avoiding options
in CXXFLAGS that G++ doesn't support.

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

17 years agoAdd some dependencies
Ken Raeburn [Wed, 31 Oct 2007 07:23:08 +0000 (07:23 +0000)]
Add some dependencies

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

17 years agoCorrect the filenames in comments
Ken Raeburn [Wed, 31 Oct 2007 07:05:40 +0000 (07:05 +0000)]
Correct the filenames in comments

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

17 years agoTest more installed headers for C++ compatibility
Ken Raeburn [Wed, 31 Oct 2007 06:59:11 +0000 (06:59 +0000)]
Test more installed headers for C++ compatibility

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

17 years agoSet CXX_LINK, CXX_LINK_STATIC, and CXX_LINK_SHARED, parallel to CC_ versions.
Ken Raeburn [Wed, 31 Oct 2007 06:07:44 +0000 (06:07 +0000)]
Set CXX_LINK, CXX_LINK_STATIC, and CXX_LINK_SHARED, parallel to CC_ versions.
Assumes for now that libpath, rpath, ldflags, pthread flags and such
apply to both C and C++ compilers.

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

17 years agoUse ac_cv_c_compiler_gnu instead of copying it to krb5_cv_prog_gcc
Ken Raeburn [Wed, 31 Oct 2007 05:40:23 +0000 (05:40 +0000)]
Use ac_cv_c_compiler_gnu instead of copying it to krb5_cv_prog_gcc

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

17 years agoInclude k5-platform.h
Ken Raeburn [Tue, 30 Oct 2007 23:33:46 +0000 (23:33 +0000)]
Include k5-platform.h

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

17 years agoDon't use GNU make features
Ken Raeburn [Tue, 30 Oct 2007 23:27:09 +0000 (23:27 +0000)]
Don't use GNU make features

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

17 years agoDefine htonll, ntohll using k5_ versions if needed. Drop Mac-specific bits
Ken Raeburn [Mon, 29 Oct 2007 23:38:11 +0000 (23:38 +0000)]
Define htonll, ntohll using k5_ versions if needed.  Drop Mac-specific bits

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

17 years agoOn Mac, include TargetConditionals.h and architecture/byte_order.h,
Ken Raeburn [Mon, 29 Oct 2007 23:30:49 +0000 (23:30 +0000)]
On Mac, include TargetConditionals.h and architecture/byte_order.h,
and use OSSwapInt{16,32,64} for SWAP* macros.

Add k5_htonll and k5_ntohll functions.  (Implementation assumes the
swapping is its own reverse on all platforms we support.)

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

17 years agoTest that C++ code can use our installed headers
Ken Raeburn [Mon, 29 Oct 2007 22:00:53 +0000 (22:00 +0000)]
Test that C++ code can use our installed headers

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

17 years agoCredentialsCache.h needs uint32_t and other definitions which aren't in Windows....
Kevin Koch [Mon, 29 Oct 2007 15:44:46 +0000 (15:44 +0000)]
CredentialsCache.h needs uint32_t and other definitions which aren't in Windows.  Get the definitions from win-mac.h

TargetVersion: 1.7
Component: krb5-libs
Ticket: 5594
Subj: Work on compiling the CCAPI test suite on Windows.

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

17 years agoCheckpoint some incomplete UNIX makefiles
Ken Raeburn [Fri, 26 Oct 2007 23:24:04 +0000 (23:24 +0000)]
Checkpoint some incomplete UNIX makefiles

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

17 years agodelete unused configure.in
Tom Yu [Tue, 23 Oct 2007 21:38:49 +0000 (21:38 +0000)]
delete unused configure.in

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

17 years agoInclude time.h for time()
Alexandra Ellwood [Tue, 23 Oct 2007 19:42:05 +0000 (19:42 +0000)]
Include time.h for time()

ticket: new

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

17 years agoSet close-on-exec flag in most places where file descriptors are
Ken Raeburn [Mon, 22 Oct 2007 19:18:53 +0000 (19:18 +0000)]
Set close-on-exec flag in most places where file descriptors are
opened in our libraries (in case another application thread spawns a
new process) and in the KDC programs (in case a plugin library spawns
a new process).

Checked calls to: open fopen THREEPARAMOPEN mkstemp socket accept dup
dup2 pipe.  In: util lib plugins kdc kadmin/server krb524.

The various programs are less critical than the libraries, as any
well-written plugin that spawns a new process should close all file
descriptors it doesn't need to communicate with the new process.

This approach also isn't bulletproof, as the call to set the
close-on-exec flag is necessarily a separate call from creating the
file descriptor, and the fork call could happen in between them.  So
plugins should be careful regardless of this patch; it will only
reduce the window of potential lossage should a plugin be poorly
written.  (AFAIK there are currently no plugins that spawn processes
where this would be a problem.)

Update dependencies.

ticket: 5561

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

17 years agoCheck dependencies for fakeka.c
Ken Raeburn [Mon, 22 Oct 2007 18:26:14 +0000 (18:26 +0000)]
Check dependencies for fakeka.c

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

17 years agoDon't leak fd on fork error
Ken Raeburn [Sat, 20 Oct 2007 00:48:40 +0000 (00:48 +0000)]
Don't leak fd on fork error

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

17 years agoKFW: BUG: WIX: Beta value hard coded
Jeffrey Altman [Thu, 18 Oct 2007 15:22:43 +0000 (15:22 +0000)]
KFW: BUG: WIX: Beta value hard coded

The beta variable value was inadvertantly committed as part of
ticket 5820 (Revision 20117).  The build script needs to
export this value when appropriate.

ticket: new
component: windows
tags: pullup
target: 1.6.3

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

17 years agoReject socket fds > FD_SETSIZE
Ken Raeburn [Thu, 18 Oct 2007 00:14:01 +0000 (00:14 +0000)]
Reject socket fds > FD_SETSIZE

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

17 years agofixed mispelling in kadmin error message
Alexandra Ellwood [Tue, 16 Oct 2007 20:39:06 +0000 (20:39 +0000)]
fixed mispelling in kadmin error message

ticket: new

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

17 years agoThere appears to be a bug either in the WiX engine or the Windows Installer 3.1.
Jeffrey Altman [Fri, 12 Oct 2007 23:08:30 +0000 (23:08 +0000)]
There appears to be a bug either in the WiX engine or the Windows Installer 3.1.
The "File" type on the Registry Search property is supposed to provide the full
path name.  Instead, we are being given just the directory as if it were being
processed with the "Directory" type.

We can avoid this for a REG_SZ value by using the "Raw" type because we are
sure that the string is not going to begin with a '#' character.

Because the full path was not being obtained for the UPGRADENSIS property, the
Uninstall routine was unable to CreateProcess() the uninstall program.

This commit also includes addition debugging in the NSIS Uninstall custom
handler to report the path and the GetLastError() value when the uninstall
fails.  This will be logged in the msiexec log file and displayed in a
MessageBox.

ticket: 5820

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

17 years agoSet the Shortcut string to match the base product name
Jeffrey Altman [Fri, 12 Oct 2007 19:18:04 +0000 (19:18 +0000)]
Set the Shortcut string to match the base product name

ticket: 5820

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

17 years agoKFW: BUG: WIX: Improve Usability of multiple architecture MSI installations, remove...
Jeffrey Altman [Fri, 12 Oct 2007 19:01:38 +0000 (19:01 +0000)]
KFW: BUG: WIX: Improve Usability of multiple architecture MSI installations, remove non-unique GUID component identifiers, and include Beta ID in the package name

The WiX installation package suffered from several problems:

 * The Beta ID was not being included in the package name.
   Fixed this by swapping the priority of "Release" and "Beta".
   "Beta" is an official release that has a beta value.
   A non-release has a datestamp as part of the package name.

 * There were duplicate GUID values being used for registry components.
   This would prevent proper removal of the components on uninstall.

 * 64-bit Installers were being constructed with the 32-bit installer
   schema.  This prevented side-by-side installation of the 64-bit and
   32-bit versions.  This also permitted 64-bit installers to be
   installed on 32-bit systems.

 * The 64-bit and 32-bit installers had the same package name.
   64-bit and 32-bit are now identified in the package name.

 * 64-bit files were being installed to the WOW64 environment.

ticket: new
component: windows
tags: pullup
target_version: 1.6.3

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

17 years agoccs_lock_status_grant_lock granting wrong lock
Alexandra Ellwood [Wed, 10 Oct 2007 21:22:14 +0000 (21:22 +0000)]
ccs_lock_status_grant_lock granting wrong lock

ccs_lock_status_grant_lock sometimes tries to grant the wrong lock
because it is not keeping track of where it got moved to.  Reorganized
code so it is easier to read and doesn't look up the lock twice.

ticket: new

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

17 years agocci_array_move not returning correct new position
Alexandra Ellwood [Wed, 10 Oct 2007 21:20:00 +0000 (21:20 +0000)]
cci_array_move not returning correct new position

cci_array_move should return correct new position when moving
from and to the same spot.

ticket: new

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

17 years agocc_ccache_store_credentials should return ccErrBadCredentialsVersion
Alexandra Ellwood [Wed, 10 Oct 2007 21:17:05 +0000 (21:17 +0000)]
cc_ccache_store_credentials should return ccErrBadCredentialsVersion

cc_ccache_store_credentials should return ccErrBadCredentialsVersion when
the principal for that version is not set on the ccache.

ticket: new

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

17 years agoupdate expected error for cc_initialize with ccapi_version_2 to match new value,...
Justin Anderson [Wed, 10 Oct 2007 20:17:42 +0000 (20:17 +0000)]
update expected error for cc_initialize with ccapi_version_2 to match new value, CC_BAD_API_VERSION (9)

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

17 years agosimple multithreaded locking test for ccapi
Justin Anderson [Wed, 10 Oct 2007 19:02:54 +0000 (19:02 +0000)]
simple multithreaded locking test for ccapi

ticket: 5459

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

17 years agoAdd enhanced error messages to new error return cases, explaining the
Ken Raeburn [Wed, 10 Oct 2007 18:27:37 +0000 (18:27 +0000)]
Add enhanced error messages to new error return cases, explaining the
(fairly generic) errors codes.

ticket: 5777

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

17 years agoRun the script to verify all Windows-exported krb5 functions have the
Ken Raeburn [Wed, 10 Oct 2007 17:54:49 +0000 (17:54 +0000)]
Run the script to verify all Windows-exported krb5 functions have the
right calling convention markers in the header files, on Windows.
Always, for lack of a separate "maintainer mode" like we have on UNIX.

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