Implement S_ISDIR from S_IFMT or _S_IFMT if necessary (e.g., on Windows)

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

For Win32 environment, define CAN_COPY_VA_LIST, and define an inline
version of vsnprintf using the MS versions.  (Apparently no need for
plain snprintf just yet.)

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

If mkstemp is missing, define the mkstemp macro is k5-platform.h, not autoconf.h

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

* v4rcp.c (error): Use stdarg version always.  Call vsnprintf instead of vsprintf.
(strsave): Call strdup.

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

* v4rcp.c (kstream_write): Allocate buffer space if it hasn't been
allocated, in the encrypting case, even if outlen is zero.  While I
don't believe this can ever happen, it requires careful examination of
lots of code paths to figure it out.  This change doesn't fix a
serious bug, but makes the analysis simple.  Also, don't bother with
separate code paths for malloc vs realloc depending on the previous
values; we can just use realloc always.

Thanks to Domagoj Babic for pointing out the (false but
understandable) null-pointer problem.

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

Check malloc return for null in kstream_create_* instead of
unconditionally dereferencing.  Thanks to Domagoj Babic for finding
the bug.

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

fix MITKRB5-SA-2007-005 [CVE-2007-2798/VU#554257]

Truncate the principal names when logging a rename operation to avoid
a stack buffer overflow.

ticket: new
target_version: 1.6.2
tags: pullup

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

fix MITKRB5-SA-2007-004 [CVE-2007-2442/VU#356961, CVE-2007-2443/VU#365313]

CVE-2007-2442/VU#356961: The RPC library can free an uninitialized
pointer.  This may lead to execution of arbitrary code.

CVE-2007-2443/VU#365313: The RPC library can write past the end of a
stack buffer.  This may (but is unlikely to) lead to execution of
arbitrary code.

ticket: new
target_version: 1.6.2
tags: pullup

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

Update configuration schema in ui\uiconfig.csv to include additional
documentation for settings and and remove unused values.

ticket: 5584

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

The new credentials window now applies Windows XP theme settings to
dialog panels that are hosted within tab controls.

ticket: 5584

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

1. Add a '--minimized' option to start NIM in minimized mode

2. If the NIM window is requested to open in a minimized state, keep
   the window hidden until there's a request to show it.

ticket: 5584

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

1. Use the debug CRT versions of the memory allocation functions in
   util\perfstat.c.  These functions allow associating file and line
   numbers with each allocation.

2. Perform a heap verification periodically to check for heap
   corruption in the debug build when using util\perfstat.c to manage
   memory allocations.

3. Change wcscmp() to _wcscmp() for compatibility in util\perfstat.h

ticket: 5584

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

1. Fix a malformed line in ui\lang\en_us\khapp.rc in a TEXTINCLUDE
   section.

2. Update the dialog template for the "About" dialog to change the
   size of the dialog box and sort the modules list.

ticket: 5584

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

NetIDMgr:

Product version => 1.3.0.0
API version     => 9

ticket: 5584

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

Be more aggressive about forcing the new credentials dialog to the
top.

Many applications call GSSAPI functions from the main GUI thread.
Doing so blocks the window message pump for that application.  When
the GSSAPI call results in the new credentials dialog being displayed,
it will be unable to bring itself to the top and grab the user input
focus because the foreground window is not responding.  This patch
makes the new credentials window a top-most window at creation time
that results in it being created at the top of the Z-order.  It can
then flash the window to indicate that it requires user attention.

ticket: 5584

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

If the new credentials window is in the advanced mode and the user
tries to switch back to basic mode, we should set the active panel to
the identity selector before switching since the basic mode only
supports the identity selector panel.

ticket: 5584

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

Don't assume that WM_DESTROY is the last message to be received by a
window.  Since the child windows are still alive and kicking by the
time the parent receives WM_DESTROY, it's still possible to receive
other messages after WM_DESTROY.

If we free any window specific data when handling WM_DESTROY, we
should reset the window data field as well, and check if we have a
valid pointer when retrieving the window data field later.

ticket: 5584

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

The clean:: target of plugins\common\Makefile should reference the
target directories using the symbolic names rather than hard coded
paths.

ticket: 5584

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

1. When freeing a message type object, it should also be removed from
   the all_msg_types list.

2. When a type handler is being removed from a message type, we
   shouldn't re-create the message type object if it has already been
   destroyed.

ticket: 5584

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

. Fix formatting in kherr.c

2. The FormatMessage() call needs special handling for 64 bit
   platforms.  Add a compile time check for now until we figure out
   what to do.

3. Remove unused code from kherr.c.

4. Correct bad references to typedefs.  kherr_severity and
   kherr_suggestion were sometimes referenced as 'enum kherr_severity'
   and 'enum kherr_suggestion' even though they are typedefs.

ticket: 5584

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

Add new notification flags KHERR_CTX_NEWCHILD, KHERR_CTX_FOLDCHILD,
KHERR_CTX_PROGRESS for listeners of error context events to enable
them to listen for child context events and progress updates.

ticket: 5584

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

Error context objects should be verified by checking the magic number
in addition to checking if the pointer to the object is valid.

ticket: 5584

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

Each error context in NIM can maintain a progress counter to indicate
the progress of the operation.  The progress counter for a context
that contains child contexts should be computed by adding the progress
counters of its children.  This update corrects the behavior of
kherr_get_progress() and kherr_get_progress_i() to take child contexts
into account.

In addition, since the progress counter of a context now depend on the
progress counters of its children, kherr_set_progress() will send
progress change notifications for any parent context as well.

ticket: 5584

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

Update developer documentation

These updates reflect changes that were made to the new credentials
acquisition process and add information to incomplete API
documentation.

ticket: 5584

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

Remove unused file Makefile.w2k from the uilib directory

ticket: 5584

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

All exports in nidmgr32.dll are declared as __declspec(dllexport)
using the KHMEXP macro in the header files.  However, since the same
header files are used to declare imports when building applications
and plug-ins that use nidmgr32.dll, the KHMEXP macro should switch to
__declspec(import) so that the relevant import table entries are
created.

To make this switch, the source files that go into nidmgr32.dll are
compiled with the special macro _NIMLIB_ defined that indicates that
the KHMEXP should expand to __declspec(dllexport).  In the absence of
this macro, KHMEXP will expand to __declspec(dllimport).

ticket: 5584

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

Fix typo in yesterday's change (u_int_8_t -> u_int8_t)

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

Fix typo and add verbosity in function-capable shell test

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

Remove check-ac-syms rules

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

Configure libdb2 in top-level configure script

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

Add the libdb2 type-existence tests to the top-level configure script.
Replace the generated header files in db2 with static ones that
include autoconf.h and make decisions based on the macros defined
there.  Omit some tests that test for things we assume now, like
"const" working and "size_t" being defined.  Update dependencies.

The libdb2 configure script now generates no header files.

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

Use k5-platform.h instead of local configure to find mkstemp

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

Add 'make depend' capability, and dependencies

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

We have code elsewhere that assumes the existence of memmove and
strerror, and it builds everywhere we care about.  So punt the
substitute versions and everything else from libdb2/clib.

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

update dependencies

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

Move (BSD-licensed) mkstemp from libdb2 to libkrb5support, and rename
the function to krb5int_mkstemp.  Generate the symbol export list for
libkrb5support at build time.

Declare krb5int_mkstemp in k5-platform.h.
Change cc_file.c to use mkstemp unconditionally.

Make libdb2.so (built for testing only) link against the
libkrb5support, and use krb5int_mkstemp if mkstemp is not available.

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

initial draft contribution guidelines

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

set svn:eol-style to native for *.[ch]

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

provide asprintf functionality for internal use

I plan to use asprintf in some gssapi error-message management
routines, so let's make sure we have the functionality available,
implementing it locally if necessary.

This implementation assumes vsnprintf is available, an assumption that
the support library is already making at the moment.

Since this implementation requires calling vsnprintf potentially
multiple times with the same va_list, use va_copy if it's available,
or provide a hack version (which should work okay if va_list is a
scalar or array type that requires no other special handling, and if
va_end does nothing interesting, which is usually the case) if the
compiler doesn't provide it.

I also changed a couple bits of code to use asprintf, to make sure we
exercise our implementation in testing.

(C99 requires vsnprintf and va_copy; vasprintf is a GNU/BSD extension,
but an oh so useful one....)

* configure.in: Check for va_copy, or if va_list objects can be simply
assigned.  Define HAS_VA_COPY and CAN_COPY_VA_LIST as appropriate.
* include/k5-platform.h: Define a va_copy macro if the compiler
doesn't provide it.

* include/k5-platform.h: If vsnprintf isn't available from the OS,
abort compilation.  If vasprintf isn't available from the OS, provide
k5_{v,}asprintf based on vsnprintf and define {v,}asprintf macros.
* lib/krb5/keytab/t_keytab.c (do_test): Use asprintf.
* util/support/errors.c (krb5int_vset_error): Use asprintf
unconditionally.

ticket: new

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

set eol-style and mime-type properties

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

krb5_walk_realm_tree leaks in capaths case

Markus Moeller reports a leak in krb5_get_credentials() which was then
traced down to profile strings leaking from within
krb5_walk_realm_tree().  A pointer to a profile string was getting
overwritten without the string being freed when *cap_nodes[0] == '.'.
Fix is to free the string prior to overwriting the pointer if the
pointer is non-null.

ticket: new
target_version: 1.6.2
tags: pullup

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

    Corrected the order of the source and modified msi packages
        in the msitran command line used to produce the transform

ticket: 5577
tags: pullup

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

don't include time.h in CredentialsCache.h if it's not needed

ticket: new

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

Disable warnings about #pragma mark - in the right place this time

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

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

Disable warnings about #pragma mark

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

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

Add advisory locking to CCAPI

Added code to trash client crashes.  Reorganized server launch to make
it easier to implement platform specific code.

ticket: new
status: open

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

This patch adds src/include/krb5/krb5.h to the MSI installer SDK component.
(Thanks to Asanka Herath for the patch.)

ticket: 5573
tags: pullup

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

Removed cross-compilation uses of inline because they don't do
anything on our compilers and were interfering with the
Windows builds.  If we need to we can revisit this later with
an actual profiling tool.

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

Removed unused locking code

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

The #if _WIN32 block threw an error with Visual Studio 2005.  Bumped the _MSC_VER threshold by 100

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

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

Only use __attribute__ on GNUC compilers

ticket: new

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

Fixed bugs introduced while moving to krb5 repository

Fixed memory leak in ccs_ccache_t and ccs_cache_collection_t when returning
the reply data.  Changed to "will_block" semantics for flow control in all
server function so it doesn't get confusing.  Fixed bug in
ccs_server_send_reply() which was preventing ipc from working (was using
the wrong variable for the reply stream).

ticket:new

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

Configure libkdb_ldap directory at top level

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

Configure cksum_body and wpse in top-level configure script.  (Still don't build by default.)

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

Move CCAPI sources to krb5 repository

ticket: new
status: open

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

don't check for readability resolving SRVTAB: keytab

Greg Hudson has pointed out that the FILE: and SRVTAB: keytab types differ in
when they report errors for files not existing.  The implementations of ANY:
floating around cope better if they consistently accept names of files that
aren't present, and report the error later when fetching information.

ticket: new

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

Remove unused macro

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

Update dependencies

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

On Solaris 10, define NO_WEAK_PTHREADS for the build.  When that symbol is
defined, skip the weak and conditional references in k5-thread.h and always
use the real pthread functions.

ticket: 5560

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

Fix up declarations of some static functions.
Now the eDirectory support compiles on Mac OS X 10.4.9 (but doesn't link
because "ldap_explode_dn" is not found).

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

Change struct _krb5_context.os_context into a single element instead of array[1]

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

Simplify UNIX krb5int_zap_data a little.  Omit volatile cast, just call memset,
but for gcc, use a volatile asm afterwards to make the memory appear to be
referenced and deter optimizations that would remove the memset.

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

Remove unused files

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

Remove unused file

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

Fix various gcc warnings about pointer target signedness and qualifiers

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

Rewrite weak key check to use a small custom struct instead of wedging
data into krb5_data, and use the array size rather than a sentinel for
loop control.

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

We're requiring that 'inline' work nowadays, so don't conditionalize its use

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

Use new comparison inline fn

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

Define and use some inline helper functions for comparing data and authdata structures, instead
of open-coding checks of multiple fields everywhere.

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

Remove some unused macros

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

Remove unused file

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

Modify WIX installer to better support upgrading betas

The WIX installers did not upgrade previous installations with the same
version number as the current package being installed.  This would leave
multiple installations of KFW x.y.z registered as being installed on the
machine even though only the most recent install is being used.

This commit instructs the Windows Installer to uninstall previous installations
with the same version number (the Maximum Upgrade version) as the package
that is being installed while ensuring that the package being installed
will not be uninstalled if the installation is being modified or repaired.

ticket: new
component: windows
tags: pullup

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

  k5-int.h, gic_opt.c

  The krb5_get_init_creds_password() and krb5_get_init_creds_keytab()
  functions permit the gic_opts parameter to be NULL.   This is not
  taken into account when testing the value with the macros
  krb5_gic_opt_is_extended() and krb5_gic_opt_is_shadowed().
  Nor is it taken into account within krb5int_gic_opte_copy() which
  is called by krb5int_gic_opt_to_opte() when the input parameter is
  not a krb5_gic_opt_ext structure.

  This commit makes two changes:

  (1) it modifies the macros to ensure that the value is non-NULL
      before evaluation.

  (2) it modifies krb5int_gic_opte_copy() to avoid copying the
      original values with memcpy() when the input is NULL.

  In addition, the code was audited to ensure that the flag
  KRB5_GET_INIT_CREDS_OPT_SHADOWED is properly set and that when
  it is set, that the allocated krb5_gic_opt_ext structure is
  freed by krb5_get_init_creds_password() and
  krb5_get_init_creds_keytab().

ticket: 5552
tags: pullup

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

rd_req_decoded needs to deal with referral realms

  * Fix handling of null realm in krb5_rd_req_decoded; now we treat a
null realm as a default realm there, as we do in the keytab code.

ticket: new
Target_Version: 1.6.2
Tags: pullup

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

Move the code for queueing an outgoing response on a TCP stream into a separate
function.

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

Look for unix find command in multiple places

Not tagged yet in case it should not go in 3.2.

Look for the unix find command in a list of directories instead of in only one place.  This way Jeff can install cygwin in c:\tools\cygwin and Ken can have it in C:\cygwin.  The two places in the list are for those two possibilities.  c:\cygwin\bin and c:\tools\cygwin\bin.

Move some prints below where logging starts, so the messages will be in the log.

Ticket: new
Target_Version: 1.6.1

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

profile stores empty string values without double quotes

        prof_parse.c (need_double_quotes):
        The profile library will happily read in right hand values
        that represent the empty string by parsing "".  However,
        when storing the same empty string back to a file, the
        empty string is written without the double quotes.

        This means that

                [section] foo = ""

        becomes

                [section] foo =

        which is invalid input.  A subsequent attempt to parse the
        profile will result in an invalid input error.

        KFW and KFM's realm editors can inadvertently produce an
        invalid krb5 profile if one of the ignored sections of the
        input profile contains a right hand value that is "".

        This patch was produced by Asanka Herath and it was reviewed
        by jaltman and lxs.

ticket: new
tags: pullup

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

race condition in referrals fallback

* src/lib/krb5/krb/gc_frm_kdc.c (krb5_get_cred_from_kdc_opt):
During referrals fallback, set *tgts to NULL after freeing.  This
avoids returning a pointer to freed memory when the first call to
do_traversal() obtains some TGTs and the subsequent
krb5_cc_retrieve_cred() of the final-hop TGT succeeds (due to some
other thread or process storing that TGT into the ccache), causing
second do_traversal() call (which would re-initialize *tgts) to
not execute.  Race condition found during KfW-3.2 testing.

ticket: new
target_version: 1.6.2
tags: pullup

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

Optimize file/directory pruning

Not tagged yet so it can be evaluated first.

Consolidate find and rm operations in prunefiles.pl as suggested by Ken.

Remove pruning of SDK files.  This hasn't been needed since the installer builds moved from staging to temp directories.

Remove debug statement not cleaned up previously.

Add a troubleshooting tip to the doc.

Ticket: new
Target_Version: 1.6.1

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

remove debugging code accidentally left in ftp/cmds.c

Debugging code I added months ago in ftp/cmds.c and forgot to remove
has apparently escaped notice until now.  Markus Moeller reports
seeing a bunch of "cmds.c: at line (number)" lines printed when he
connects to a server.

This patch removes those lines.  A couple other debugging statements
that test for the debug flag being set on the command line are left
in.

ticket: new

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

Remove unused old .cvsignore files

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

Add /REPOSITORY EXPORT option

Adjust abbreviations: /svntag is now 't' so /src can be 's' so /repository can have the abbreviation 'r.'

Process /NOLOG.

Write out ignored command line options as ones being passed to NMAKE (via build.pl).

Generate GetOptions argument from the <Config> section of the config.xml file.

Ticket: new
Tags: Pullup
Target_Version: 1.6.1

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

Don't use sendmsg/recvmsg paths if CMSG_SPACE isn't defined

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

Modify Unix find test to only check the current directory for the non-existent a.tmp

When the pismere area is under the script area (as in Jeff's setup on afs), the find test slogs through 100MB of files and occasionally finds an a.tmp.

This correct the erroneous 'unix find not found' errors and improves performance noticeably when using afs.

Ticket: new

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

Rollback previous change.  This version is the same as R19472

Target_Version: 1.6.1
Ticket: 5521
Tags: pullup

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

More NIM Developer documentation updates

Revisions to the plug-in architecture documentation.

Adds a graphic showing the plug-in architecture

Defines "ALIASES" in the Makefile so that the current
NIM API Version number will be used within the docs
at build time.

ticket: new

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

Factor repository access out of bkw.pl into repository1.pl

Modify bkw.pl to use an initial config file to fetch the sources and then use the config file from those sources to do the build.  This way, the description of how to build the sources is in the config file that is part of the sources.  It is possible and probably reasonable for the initial config file to be the same as the tagged version.  Output all the options used.

Add bootstrap.xml - a sample minimal config file, sufficient to fetch the sources from a repository.

Target_Version: 1.6.1
Ticket: 5521
Tags: pullup

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

kfwlogon corrections for XP

This patch addresses a problem discovered on some XP systems.
After rundll32.exe starts, the CreateProcess can fail to
start kfwcpcc.exe if the current directory is not %WinDir%\System32.
CreateProcess() should be called with the lpApplicationName parameter
set to NULL in order to permit the use of the PATH.

Also, in ConfigureLogonScript ensure that the trailing NUL of the
constructed command line is processed when producing the wide
character version of the string.

ticket: new
component: windows
tags: pullup

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

updates for NIM developer documentation

Add the use of Tabs and Frames to the resulting output (doxyfile.cfg)

Stylesheet changes to improve readability (stylesheet.css)

Copyright updates (all files)

Language Editing (all files)

Version History (main_page.h)

There are still a number of missing sections that need to be filled
in as time permits:

 - Writing the dialog procedure for a cred type panel
 - Credential messages
 - configuration provider details
 - Actions
 - Menus

ticket: new
component: windows
tags: pullup

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

In testing it has been observed that remote desktop connections
will execute the NPLogonNotify function but if the logon is
re-connecting to an existing session, the LogonScript is ignored.
This leaves orphaned credential cache files.

This commit adds a function, KFW_cleanup_orphaned_caches, which
is called by NPLogonNotify to delete any orphaned cache files.
An orphaned cache file is one that is older than five minutes.

ticket: 5532
tags: pullup

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

KFW Network Provider Improvements

Change event log name from "KFW Logon" to "MIT Kerberos".  This
is being done to avoid confusion with the "KFW Logon" functionality
that was provided by older versions of OpenAFS. (kfwlogon.h)

Remove logging of the inability to access the "Debug" registry value.
On Vista, kfwlogon.dll is no longer loaded by winlogin.exe.  Instead
it is loaded by mpnotify.exe which is spawned once for each logon
request.  (kfwcommon.c)

Add a test to ensure that RegisterEventSource succeeded before calling
ReportEvent.  (kfwcommon.c)

Absolutely make sure that krb5_init_context() succeeded before calling
any other krb5 functions.  (kfwcommon.c)

Add a comment explaining why NPGetCaps() says we are a file system even
though we are not.  "It won't work otherwise."  (kfwlogon.c)

Change all comparisions for Windows Station and AuthentInfoType names
to case insensitive comparisons.  Vista does not use the same case as XP.
(kfwlogon.c)

Change the requested access to the temporary cache file from "All" to
"Read | Delete" when importing its contents into the API cache.
Otherwise, the access test will fail on Vista.  (kfwlogon.c)

ticket: new
component: windows
tags: pullup

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

Add the new kadm5srv function krb5_get_principal_keys to the export list

Build a separate copy of kadmin/cli/keytab.c for kadmin.local that exposes
the -norandkey flag in a way that doesn't require the compiler support -c
and -o at the same time.

Ticket: 914

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

Add support for extracting existing keys from the KDC with kadmin.local.
Adds a -norandkey option to the ktadd command only in kadmin.local, and
adds a new function to the libkadm5srv library that kadmin.local can
call.  There is no protocol or network access to this function.

Ticket: 914

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

The about dialog box did not respond to Alt-F4 because the
WM_CLOSE message was not being processed.

ticket: 5530

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

Leave built installers in their temp areas and change final copy step to copy them into <out> from their new location.  Delay cleaning up the temp areas until after that copy

Target_Version: 1.6.1
Ticket: 5521
Tags: pullup

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

Add resource data windows/version.rc for:
   kinit.exe, klist.exe, kdestroy.exe, kvno.exe
   kdeltkt.exe, kcpytkt.exe, kfwlogon.dll, kfwcpcc.exe
   gss-client.exe, gss-server.exe, kpasswd.exe

Remove resource files: kfwcpcc.rc kfwlogon.rc and use
windows/version.rc in their place

Add resource make rules to Makefile.in files included
in this commit

Add missing _VC_MANIFEST_EMBED_EXE rule to kpasswd/Makefile.in

Comment out addition Windows only make rules with ##WIN32##

ticket: 5529
tags: pullup

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

Implement environment variable settings from the config xml file.  See notes in bkwconfig.xml

Target_Version: 1.6.1
Ticket: 5521
Tags: pullup

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

Add vertical scrollbars to realm fields in dialogs

The obtain new credentials dialog and the change password
dialog provide a "Realm" combo-box.  These controls were
not configured to display a vertical scroll bar if there
were more than five realms in the list.

Version number remains 1.2.0.2

ticket: new
component: windows
tags: pullup

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

update copyrights and team members

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

Add BUILD_KFW=1 & DEBUG_SYMBOL=1 to build.pl invocation.
Align setting of RELEASE/DEBUG/BETA with release notes.

Target_Version: 1.6.1
Ticket: 5521
Tags: pullup

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