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

more-sysnames-20030925

add openbsd 3.3-3.4, freebsd 5.1-5.2 (from wollman@khavrinen.lcs.mit.edu)

deal-with-new-gcc-multiline-behavior-20030924

FIXES 2154

"foo
bar" is right out

deal.

lwp-elf-bsd-and-assembly-cleanup-20030924

FIXES 2153

fsync-wait-typing-mistake-20030924

FIXES 2155

correct refs to fsync_wait

linux-dentry-revalidate-make-sysname-lists-work-20030924

pass the unmangled name into afs_lookup in dentry_revalidate if dnlc lookup
fails.

windows-doesnt-always-have-long-long-20030923

Still can't assume long long exists; can break Windows build.

callbacks-later-remove-lhs-cast-20030915

FIXES 2100

lhs cast illegal, some compilers barf, remove it.

====================
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 2100

lhs cast illegal, some compilers barf, remove it.
this time commit the correct patch

vos-fix-fprintf-args-20030915

FIXES 2096

kill extraneous argument to fprintf

res-search-test-shouldnt-use-res-close-20030915

FIXES 2099

res_close() not part of generally accepted resolver api

winnt-newcreds-dialog-missing-attribute-20030915

FIXES 2119

DS_CENTER attribute missing from dialog.

winnt-avoid-leak-and-memory-corruption-in-afscreds-20030915

FIXES 2120

WINNT/talocale/TaLocale.h conditionally defines REALLOC() to
TaLocaleReallocFunction().

WINNT/client_creds/afscred.h  conditionally defines REALLOC() to
AfsCredsReallocFunction().

Unfortunately, AfsCredsReallocFunction is never used because REALLOC is
always defined first by an include of Talocale.h within afscreds.h.
This is a problem because it results in memory being allocated within a
DLL and then freed by the EXE.  This results in both a memory leak in
the DLL and memory corruption in the EXE.

windows-doesnt-always-have-long-long-20030913

freebsd50-20030909

use VFS_VGET with correct args instead of igetinode
fix str warnings in dir.c

irix-volser-lwp-stack-20030904

FIXES 2031

may be needed on more platforms
based on suggestion from salvet@ics.muni.cz

sys-dont-rebuild-for-aix-files-20030904

FIXES 2040

hack, touch the files so we don't rebuild for them

winnt-use-current-platform-sdk-20030904

FIXES 2068

_WIN32_WINNT definition must match _WIN32_IE
to use current platform sdk

winnt-checked-do-debug-20030904

FIXES 2069

Disable optimization; generate full debug info in objs for checked build.

winnt-binaryify-icos-20030903

at suggestion of rsm4@cssd.pitt.edu

winnt-nsis-installer-update-20030903

more work on the nsis installer
should be mostly complete

aix52-support-20030903

AIX 5.2 support
does not include param.h
does not include deadlock fix for afs_vnop_remove.c

linux-fstrace-syscall-return-code-20030903

so fstrace doesn't get bogus return values

freebsd50-20030829

pass thread, not proc, to VOP_[UN]LOCK in afs_write
pass correct bio arg to biodone in afs_strategy
fix some warnings
reformat some #ifdefs for clarity

winnt-nsis-installer-server-support-more-languages-20030828

make server available to install. support more languages.

winnt-server-dirpaths-fix-20030828

make server stuff actually be able to run

ukernel-osi-credp-20030828

also declare and init afs_osi_credp for ukernel.
thanks to nathan for bug report.

winnt-64bit-env-20030827

define and support 64 bit env on windows

voldump-builds-with-volser-20030827

avoid circular dependancy issue.

winnt-nsis-installer-20030827

make the nsis installer actually work.

osi-credp-20030827

Use a pointer for afs_osi_cred, and crget() instead of just inventing one.
Required for FreeBSD, which puts a mutex in the cred.
Not strictly required for other BSDs, but it's more correct.