ptserver-prototype-ubik-20031208

since ubik got prototypes, we have to make our shadow functions the same

import-fs-formatting-to-windows-20031207

TICKET 2651

Import string formating from venus/fs.c

set-service-error-condition-afsd-20031207

TICKET 2619

If the service crashes unexpectedly (eg, when the network adapter list
changes) we should return an error code so that system policy can be
used to determine if the service should be automatically restarted.
When there is an error GlobalStatus gets set to a non-zero value.
If GlobalStatus is set, we will return ERROR_EXCEPTION_IN_SERVICE
to the Service Manager.

remove-int64-conversion-warnings-20031207

Use casting to avoid repeated conversion warnings resulting in data loss.

jbuehler-add-missing-extern-c-patch-20031207

Add extern C modifiers to non-C++ functions in Windows files

more-must-returns-20031207

More must returns value warnings cleaned up

ntmakefile-fix-20031207

remove extraneous include of src/config/NTMakefile.version

add-missing-return-values-20031207

Add missing return values producing warnings on Windows

ticket-2618-corrections-20031207

Remove extraneous declarations in dir.c

ticket-2618-patches-20031207

TICKET 2618

flexelint patches for prototype handling from Joe Beuhler

>>>>>>>>>>>>>>>>>>>>>>>>>>>>

This patch adds a bunch of missing prototypes, makes various
corrections to global prototypes, and removes global declarations that
are not needed.

One set of added prototypes is commented out because it applies to
some printf-like functions that are implemented the way it used to be
done before varargs existed, and they need to be unprototyped or the
code will fail to compile.  (There are a number of functions in the
OpenAFS code that need to be converted to use varargs...)

There is one place of note in src/WINNT/afsd/fs.c where typedefs for
afs_int32 and afs_uint32 conflict with afs/stds.h.  I just added a
note to that effect.

Prototypes in src/WINNT/afsd/fs.c are incorrect for Windows NT.  I
don't know whether the return type involved changes from platform to
platform -- they come from .xg files.

Some prototypes are now commented out or #ifdef'd to match the code
that uses them.

Some global variables conflict between modules and have been made static.
The bufferBlock variable was of two completely different types...

In src/config/stds.h, I changed the declaration of afs_intmax_t to
reflect the fact that the Windows platform supports 64 bit integers.
This may be incorrect semantically, but I believe it is needed for the
%llx format to match afs_intmax_t arguments (for printing of inodes
mainly, I believe).

There were two different declarations for the rxkad_stats structure --
I synced them up.

- Joe

Removed the conflicting typedefs from src/WINNT/afsd/fs.c

Removed the changes to src/config/stds.h.  Added declaration of
AFS_64BIT_CLIENT to src/config/NTMakefile.i386_nt40 instead.

Do not remove unused variables which are defintions of data type
values.  Instead comment them out to avoid space utilization and
warnings.

afssvrcfg-updates-20031206

Replace the mechanism used to register drives with WNetAddConnection2.
This will hopefully be more reliable than then older code which was
unable to register drive mappings.

Corrected error string formatting.

add-multiple-interface-support-20031206

The ubik support on Windows only verified against the first
address on the system even if that address was assigned to
a disabled interface.

reorg-ntbuild-20031206

Reorganize ntbuild.bat to place all of the user edits at the front

fix-memory-alloc-srv-cfg-wizard-20031206

The Windows AFS Server Config Wizard would produce an invalid memory
deallocation error from the memory management routines due to the
failure to allocate PROGRESSDISPLAY objects with the same tools used
to deallocate them.

more-NSIS-fixes-20031206

from Rob Murawski

add-pdb-files-to-server-nsis-20031206

Add PDB files for debugging to NSIS Server install

more-vc-net-installer-fixes-20031206

Make sure we get the included libraries correct for install

another-correction-ntbuild-bat-20031206

I wish .BAT files had IF-THEN-ELSE instead of just IF

no-debug-for-free-builds-20031206

Reorganize makefile to not generate debug files for free builds.

correct-include-for-cl1300-20031206

CL 1300 uses the mfcatl paths just like CL 1310

nsis-installer-updates-20031205

Remove need for MakeCommon.bat

Produce nsi-includes.nsi from NTMakefile

Update OpenAFS.nsi to use MFC70.DLL and MFC71.DLL as needed.
(Need to confirm with Rob Murawski)

product-version-number-update-20031205

Modify the variables used to construct the product version numbers
to allow these same values to be used with the NSIS installer

typo-correction-ntmakefile-20031205

Fix a typo in msvcrtd.dll reference

updates-to-NSIS-to-support-debug-crt-20031205

Updates to NSIS installer scripts to conditionally build with
debug or non-debug CRT libraries; also, now supports conditional
library usage based upon the MS Compiler version used

i386_nt40-lex-macro-fix-20031205

Not that anyone has used it in forever and therefore would not notice
but the LEX macro was a version number "1.2.2 a" instead of a program
name.  Changed to "flex -l" but it really depends on which lex type
program is being used

NSIS-installer-debug-mode-20031205

NSIS installer debug mode activiation based upon AFSDEV_BUILDTYPE

pdb-files-for-checked-builds-20031205

Produce PDB files in the DEST/CHECKED tree for use by NSIS installer

fix-ntbuild-bat-20031205

Fix conditional contruction of AFSDEV_LIB variable

largefile-support-continued-20031205

FIXES 1493

configure switch and minor largefile cleanup. not done.

vos-full-listvol-on-by-default-20031204

at the suggestion of miles@cs.stanford.edu

salvager-remove-orphaned-header-20031204

FIXES 2522

this should be fixed for the global case, but that's harder to test

libafsrpc-fix-objdir-breakage-20031204

FIXES 2615

the not gmake workaround broke objdir builds. sigh.

solaris-kdump-build-rationalization-20031204

make kdump64 and kdump32 get built with sanity on solaris

salvager-fix-misparented-vnode-1-20031204

if vnode 1 gets a parent (which is incorrect) arrange for it to be cleaned
up properly

hpux-1123-update-20031204

update the hpux 11.23 on ia64 support
don't build libuafs. make kernel modules build. fix afsd.

fix-checked-build-libraries-20031203

checked builds were linking against non-debug versions of the MFC libraries
which produced warnings at link time.  Replaced with the debug versions.

typo-20031203

more-missing-crtdbg-includes-20031203

For when you are debugging ...

add-missing-include-file-afsd-windows-20031203

ctrdbg.h but only when _DEBUG is defined

i-hate-bat-files-20031203

Add missing % at end of variable names to be compatible with cmd.exe

do-not-make-commits-after-24hrs-awake-20031203

.BAT files do not use the same syntax as makefiles.
don't make commits after being up more than 24 hours straight.

ntbuild-support-for-visual-studio-net-2003-20031202

Add LIB and INCLUDE files for Visual Studio .NET 2003
(Thanks to Rob Murawski for details)

rpc-rename-avoid-loop-20031201

enumerated solution (n vnodes cannot yield n+1 loop passes) suggested by
jhutz@cmu.edu. avoid looping forever on a corrupt parent.

tvolser-update-20031121

add transaction mutex to handle global list of transactions

remove-duplicate-include-from-shell-ext-cpp-20031130

Remove atlconv.h include the second time it is listed

update-version-number-for-windows-20031129

Updated the version number to 1.3.51 for both nt40 and win95
However, it is unlike that win95 will build at this point in time.
Its version of the makefile had not been touched since 1.2.2b

jaltman-missing-variable-20031129

jbuehler left out a variable declaration

jbeuhler-flexelint-bugs-found-20031128

This patch covers bugs found by running flexelint on
the CVS code on Windows NT.

- incorrect enum used in al_creds.cpp and various other files
- use of local after it goes out of scope in al_wizard.cpp
- uninitialized class members in afsclass/*
- cm_config.c looks like it has code commented out accidentally
- useless statement in alert.cpp
- no default case handling in problems.cpp
- strange use of & on booleans in set_clone.cpp
- fgets() and fclose() on closed stream in cellconfig.c
- memory leak in cellconfig.c
- *scanf into variables of unknown length in cellconfig.c
- incorrect pointer passed to getAFSServer() in cellconfig.c
- possible buffer overflows in userok.c
- address of array in bos.c
- too many arguments for printf() in commands.c
- return code not saved in bucoord/main.c
- bad types for argv arrays in bucoord/main.c
- probably incorrect initializer list in ol_verify.c
- extra argument to TLog() in lwps.c
- memory leak in cmd/cmd.c
- lack of typecast on malloc() in a couple files
- lower-case L for long constant in touch.c (is it a 1 or an l...)
- parentheses nesting error in util_cr.c
- apparently invalid case fall-through in util_cr.c
- various memory leaks in util_cr.c
- macro missing parens in crypt.c
- macro missing parens in quad_cksum.c
- null pointer deref in kpasswd.c (CVS work in progress?)
- Print_bos_ProcessState_p() code does not look correct
- extra argument to printf() in kas.c
- typo in vsprocs.c
- macro missing parens in iomgr.c
- address of array in threadname.c
- storing result of getch() in char type in waitkey.c (may not match EOF constant)
- storing pointer to local car in global in ptserver.c (benign -- it is in main())
- address of array in rx_lwp.c
- macro missing parens in rx_packet.h
- impossible == (signed vs. unsigned?) in rxdebug.c
- bogus constants in xdr.c and xdr_array.c (possible security implications?)
- incorrect test of fd for open file in update/server.c
- dangerous unparenthesized macro in get_krbrlm.c
- buffer overrun in regex.c
- missing comma causes string concat and bad array init in vlclient.c
- comparison of string < 0 in ntops.c
- default case has no code in vos.c
--
Joe Buehler

>>>>

Comments: Several comments for FIXME left in the code - unaddressed

missing-assert-h-20031129

Missing assert.h for added assert()

jbeuhler-flexelint-fun-with-printf-20031128

This patch addresses a number of problems with printf-like functions.

Most of them are %x formats applied to pointers (which could be a
problem depending on the architecture I would assume).  There is a
%p format for pointers on many machines but I don't know if it
applies to all supported OpenAFS platforms so I didn't go that route.

There is a buffer overflow fix to ktime.c

There are a couple places where incorrect format characters were used.

One call had two arguments swapped.

Other fixes include the use of PrintInode() and %s formats for
printing inode numbers.  The code was inconsistent in this regard -- sometimes
PrintInode() was used, sometimes it was side-stepped.

There is one place where a local shadowed another variable, so I renamed
the local.

The last segment points out that an fdset need not be anything printable
with an integer format.
--
Joe Buehler

Comments - The fdset comment relates to a WinSock only file on which
the size is a pointer to int.  Modified the source to use a %p instead
of %x as it is supported on Windows.

Added an assert(p != NULL) to src/rx/rx_packet.c

jaltman-flexelint-aggregate-initialization-patches-20031129

Address FIXME concerns in jbuehler-flexelint-aggregate-initialization-patches-20031129

jbuelhler-flexelint-aggregate-initialization-patches-20031129

This patch mainly makes explicit some initializations that were implicit.

There are several places where it looks like the missing initialization
may be a bug, and I have inserted comments to that effect in the
relevant patches.  Someone needs to look at them and supply
whatever is missing (if anything is).

In make_keyperm.c, an array was sized too large.
--
Joe Buehler

hpux-rx-attach-file-struct-to-socket-20031124

FIXES 2450

attach a struct file to the socket; clean up when done

solaris-make-hates-dollar-lessthan-20031126

the $< macro support in solaris make is different from every other make
due to POSIX not being fully specific. a perl script cleans up the mess,
and gives us a different mess.

hpux-snprintf-support-20031126

FIXES 2587

need sys/socket.h on hpux

ia64-hpux1123-support-20031126

FIXES 2579

necessary changes to support hpux 11.23 on ia64.

ticket-2584-patches-20031126

hpux-ia64-dont-optimize-lwp-20031124

FIXES 2450

remove -O when compiling lwp

fix-joe-beuhler-patches-20031124

minor fixes to Joe Beuhler's NT patches for non-NT

joe-beuhler-patches-20031122

Joe Beuhler's many patches merged into the tree

threaded-volserver-progress-20031114

not ready for primetime. not even close.

tvolser-update-20031121

add transaction mutex to handle global list of transactions

jaltman-afsd-loopback-w2k-and-noipc-patches-20031121

 * Change handling of CM_ERROR_NOIPC to return an Access Denied
   error instead of a Bad Network Name error.  The reason is that
   Bad Network Name results in a subsequent query by many apps
   to find a good network name which cannot be accomplished.
   Access Denied is more appropriate and results in an immediate
   error return.

 * Disable the Loopback Adapter hack for operating systems prior
   to Windows XP because the Loopback adapter on previous versions
   does not perform loopback operations on Broadcast messages
   which are necessary for the proper registration of the
   NETBIOS name for the AFS Service.

 * Many minor corrections to uninitialized variables; spelling
   errors; formatting errors; etc.

rob-murawski-afscreds-scm-patch-20031121

Reduce the requested list of permissions to the minimum necessary to
access the service manager and service for the requested task.

fix-CH-lang-resources-for-afs_setup_utils-20031121

Rename afs_setup_utils/lang/zh_CH/afs_setup_utmls.rc to afs_setup_utils.rc

afscreds-drivemap-updates-20031121

#undef REALLOC before importing drivemap.h to prevent the use of the
wrong memory allocation routines.

Remove extraneous code which computed strings which are no longer used
and worse produced misleading debug information

Improved consistency of drive mapping when HIGHSECURITY is in use

thread-and-timer-corrections-to-afscreds-20031121

Changes to afscreds to place the obtain tokens dialog into a separate
thread to prevent blocking of the Windows Message queue.  Requires
utilization of mutex semaphores to protect credential data structures.

Previous versions of afscreds would set/unset the timer event every
time the program received an event indicating user activity including
dragging the mouse across the systemtray icon.  This resulted in
extremely unpredictable behavior.  Now the timer event is only turned
off when it must be turned off; and turned on when it must be turned on.
The result is a credential expiration check once a minute.

shadow-to-jaltman-bkbox-02-20031121

Remove another file which was included in the bkbox patch but is really
autogenerated.

Add a command to dolastcmp.bat to conditionally delete golast.bat if
it exists.  Otherwise, the initial CD command which is meant to be
written to the file is lost.

shadow-to-jaltman-bkbox-01-20031121

Incorrectly committed NTLang.bat when it is copied from src/config

patch-from-shadow-to-jaltman-bkbox-20031120

This massive patch contains changes in several significant areas for Windows:

- the ability to specify the mount point to be something other than /afs

- functionality to assist debugging of the NT Services

- support for languages other than English (NTLang.bat)

- revisions to the Build system to support separate trees for src, obj,
  dest and free or checked; allow any MS compiler to be used

- updates to NSIS installer build

- mutex locking added to critical locations

- updates to IS5 directory tree creation

- update to afswsNetscape_config.sh

compiler-warnings-20031119

fix various compiler warnings
some are from jbuehler@hekimian.com

error-typo-20031119

confusion of == and =

fixed-and-improved-afscp-20031118

Fix afscp to make it build and fix the -u option.
Add support for explicitly specifying a file by host/fid.

callbacks-later-locking-order-20031117

be consistent in the order we hold H_LOCK and FSYNC_LOCK;
in this case acquire FSYNC_LOCK as needed.

volser-protoization-20031114

more protoization in the volserver

threaded-volserver-progress-20031114

not ready for primetime. not even close.

volser-restore-create-timestamp-not-touched-on-incremental-20031114

FIXES 2521

        The issue: when you restore an incremental dump to an existing volume,
the creation timestamp of the target volume gets reset to the current timestamp
(literally the timestamp of when the 'vos restore' command is being executed).
That messes things up for any subsequent 'vos release' operation, because such
operation then needs to do a full release rather than an incremental one.  For
large volumes that can be quite an issue.

remove-redefined-warnings-20031114

FIXES 2512

don't redefine KABADARGUMENT

cvs-remove-wow3-20031114

FIXES 2505

remove empty file which came with the source

restorevol-needs-gethostbyaddr-20031114

FIXES 2514

so restorevol will link on solaris

callback-rxcon-protection-20031114

FIXES 2498

        CheckHost() sets the HOSTDELETED flag on a host, does its
        touch, calls H_UNLOCK at the end of its function;
        AddCallBCal1 in a different thread grabs the lock, does its
        thing.  Gets to GetSomeSpace_r(), which calls
        h_Enumerate_r; the host in question isn't held, so
        h_Enumerate_r calls h_Hold_r and h_Release_r on the host
        that CheckHost() set the HOSTDELETED flag on; h_Release_r
        sees the HOSTDELETED flag and calls h_TossStuff_r, poof, we
        have our broken host entry for ClearHostCallbacks_r to trip

tail-dont-use-dash-one-20031114

FIXES 2453

new tail axes -1 support

namei-ihandle-leak-20031114

FIXES 2518

namei_dec now logs if you have open ihandles when you unlink a file

====================
This delta was composed from multiple commits as part of the CVS->Git migration.
The checkin message with each commit was inconsistent.
The following are the additional commit messages.
====================
FIXES 2518

flush all dir buffers we have cached when rewriting a volume from a restore

avoids holding an ihandle which has an open fdhandle for an unlinked file

====================
FIXES 2518

remove logging code for now. it appears to trigger on some possibly legitimate operations.

more-posix-regex-20031112

remove more REG_BASIC

regex-no-reg-basic-20031112

apparently not all posix regex implementations have REG_BASIC.
get rid of it since it's the default.

rx-multi-incs-20031111

fix rx_multi includes so it will compile

rmtsys-uninit-20031110

fix uninitialized variable found by jbuehler@hekimian.com
consolidate duplicate cases so this won't happen again

cachemgr-rx-multi-gettime-20031109

use multi Rx to GetTime all servers we wish to talk to in parallel. This
should have the effect of compressing the amount of time to time out all
fileservers to (1 times full set of Rx transmit intervals) instead of (n times
same). VLservers get handled out of band and so aren't in scope for this
change.

linux-sysname-list-sys-links-resolve-dentry-20031109

without this, if you're in a directory that the second or later entry in a
sysname list resolved to, pwd returns ESTALE. force the true path to have its
dentry cached.

bos-fix-mode-complaints-20031024

FIXES 2277

complain correctly about mode

macos103-20031024

FIXES 2325

support for macos 10.3 for 1.2.10

freebsd-vnode-if-20031015

generate vnode_if.h dynamically for FreeBSD

openbsd-deadlock-fix-20031010

release global lock when calling getnewvnode
thanks to wollman@khavrinen.lcs.mit.edu for finding this

use-posix-regex-20031010

use posix regex if available instead of re_comp
eliminates need for -lcompat on the BSDs, so remove it from XLIBS

fix-lwp-20031010

abort needs _C_LABEL too
add dependency on process.i386.s to process.o

openbsd35-20031009

OpenBSD 3.5 config files
fix afs_osi_suser for 3.5 (from provos@citi.umich.edu)

openbsd-20031007

use poll instead of select for 3.5 and beyond
reported by provos

====================
This delta was composed from multiple commits as part of the CVS->Git migration.
The checkin message with each commit was inconsistent.
The following are the additional commit messages.
====================

works much better if you check the right vnode pointer

freebsd5-20031001

FreeBSD 5.2 config files.

ubik-dont-serve-data-while-syncing-20030927

while we're syncing a new database over, and if the sync fails, don't keep
serving out of the truncated and/or trashed database. set the db version to 0
before starting the sync.

openbsd-20030926

check for (v_type == VNON) from VFS_VGET in UFSOpen
remove dead prototypes from osi_vnodeops.c