Commit Graph

6124 Commits

Author SHA1 Message Date
Matt Benjamin
e31f3f68b0 Make catalog file destdir when not transarc paths
The appropriate mkdir -p was missing for this case, the patch adds it.

Reviewed-on: http://gerrit.openafs.org/387
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-31 12:35:19 -07:00
Matt Benjamin
616ab5c9ec Remove AFS_KERBEROS_ENV in NetBSD 30, 40 param files
This creates dependency on ktc_newpag, which has been determined to be
obsolete.

Reviewed-on: http://gerrit.openafs.org/384
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-31 07:29:22 -07:00
Matt Benjamin
49d893eff3 Build shared afsauthent and afsrpc on *BSD
This seems to be a required step, going forward.  Omitting breaks pam,
for example.

Reviewed-on: http://gerrit.openafs.org/382
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-30 17:01:08 -07:00
Claudio Bisegni
dffd78ccb0 Fixed the tokens update and update .gitignore for MacOSX packaging file
Fixed the tokens update on preference pane if operation is made with NSStatusItem. 
Has been updated the src/packaging/MacOS/.gitingore for CellServDB file and src/libafs/.gitignore has been added afs.x86_darwin_100.plist.

Reviewed-on: http://gerrit.openafs.org/372
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
2009-08-28 13:05:50 -07:00
Michael Meffie
8540814140 doc: typo fix in asetkey man page example
Fixed a typo in the asetkey man page.

Reviewed-on: http://gerrit.openafs.org/373
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-28 11:30:49 -07:00
Derrick Brashear
e26235ed3e macos 10.5 doesn't support compiler kext flag
-kext was added in later xcodes; it's safer to just assume we don't
have it.

Reviewed-on: http://gerrit.openafs.org/365
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-27 22:03:10 -07:00
Jeffrey Altman
7bf32f6115 Windows: Version Update 1.5.6200.0
Version update to 1.5.6200.0

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/354
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-27 13:18:23 -07:00
Jeffrey Altman
f77cc39801 Windows: SMB Server does not support streams, so say that
The SMB server does not support streams.  When replying to
NT Create operations with extended responses report that
there are no streams.  In addition, add support for Query
Path/File Stream Information and return only the null stream.

Correct the 'totalData' response field in several places
within Trans2QPathInfo and Trans2QFileInfo.

Return CM_ERROR_BADOP for SMB NT Transact operations that
are not supported instead of CM_ERROR_INVAL.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/347
Tested-by: Asanka Herath <asanka@secure-endpoints.com>
Reviewed-by: Asanka Herath <asanka@secure-endpoints.com>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-27 13:15:23 -07:00
Jeffrey Altman
a767455f02 Correct description of afs krb.conf file format
The file format is a single line containing one or more realms
separated by white space.

LICENSE BSD

Reviewed-on: http://gerrit.openafs.org/352
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
2009-08-27 13:15:05 -07:00
Derrick Brashear
a747f5951b update decode-panic for 10.6
how panics get reported changed in 10.6. also, the 64 bit universal kext
means we don't know what the last boot was.

Reviewed-on: http://gerrit.openafs.org/353
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-27 12:54:57 -07:00
Derrick Brashear
6b555e5dc1 make openafs 1.5.62
update version strings for 1.5.62

Reviewed-on: http://gerrit.openafs.org/351
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-27 08:21:34 -07:00
Derrick Brashear
2b1336b59b macos afs prefspane support for 10.4
NSUInteger is newer than what's included in the 10.4 SDK. restore
functionality for 10.4

Reviewed-on: http://gerrit.openafs.org/349
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-27 07:39:03 -07:00
Derrick Brashear
abcb52c4fd correct GetuTime function name
GetuTime is really osi_GetuTime

Reviewed-on: http://gerrit.openafs.org/350
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-27 07:38:50 -07:00
Derrick Brashear
81099c479e macos 10.6 64 bit support
support 64 bit userland with 32 bit kernel, and
32 and 64 bit userland with 64 kit kernel

Reviewed-on: http://gerrit.openafs.org/343
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
2009-08-27 04:59:29 -07:00
Derrick Brashear
74d3b11062 redefine xdr functions consistently
if we redefine the names of all xdr_ functions, the same must be done
for xdr_alloc and xdr_free

Reviewed-on: http://gerrit.openafs.org/348
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-27 04:58:42 -07:00
Jeffrey Altman
af8dee05f0 Windows: Release Notes Updates for 1.5.62
Some formatting corrections

Section 3.26 updated to describe limitations caused by Apple
Bonjour on 32-bit Windows.

Section 3.40 updated to mentions Microsoft SMB Redirector
Extended Server Timeout support.

Section 3.41 updated to describe inability of pioctl operations
to succeed on service mapped drives.

Section 3.43 updated to include Windows 7 and Server 2008
in the known issues list.

New section 3.51 on Microsoft RPC Services

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/346
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-26 19:37:43 -07:00
Jeffrey Altman
d605bf109d Windows: Updates to Windows ChangeLog
1.5.6200

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/345
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-26 19:37:23 -07:00
Derrick Brashear
9473089bdf macos rc script server handling
only start bosserver if its configured. shut it down if we (likely) started it

Reviewed-on: http://gerrit.openafs.org/344
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
2009-08-26 15:42:28 -07:00
Russ Allbery
622065f98e Initialize xferStartTime for StoreData timings
In the reorganization of fetch and store procs, the initialization of
the transfer start time for a store was lost, leading to meaningless
statistics from the uninitialized variable.  Initialize the transfer
time at the start of transferring each chunk and move the variable into
the for loop.

Reviewed-on: http://gerrit.openafs.org/340
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
2009-08-26 15:29:51 -07:00
Russ Allbery
a59df36211 Add automatic sysname detection for ARM Linux
Add arm*-linux* to the case statement that attempts to automatically
determine the AFS sysname, similar to the other Linux sysname
determination cases.

Reviewed-on: http://gerrit.openafs.org/334
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-26 08:34:46 -07:00
Claudio Bisegni
b695261de4 10.6 update(Workaround for kerberos bug) and cleaning of deprecated function form code
For aklog has been added the workaround for kerberos function KLAcquireInitialTickets that in 10.6 return always error.
First is called KLCacheHasValidTickets for check user ticket presence, if there are no ticket  KLAcquireNewInitialTickets will be called for get user credential.
Has been removed also many deprecated api.

Reviewed-on: http://gerrit.openafs.org/341
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-26 08:34:21 -07:00
Felix Frank
a66dc4d0cb Some renames and reformattings in afs_fetchstore
Reviewed-on: http://gerrit.openafs.org/124
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-25 20:01:18 -07:00
Felix Frank
88b7deb5bc Sanity for the padd storeOp
Reviewed-on: http://gerrit.openafs.org/123
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-25 19:29:47 -07:00
Felix Frank
6220ede6a9 Add the padd storeOp and unify statistics keeping.
Filling with zeroes is done within a storeop of its own. This raises
consistency and readability.
Also, the formerly copy-pasted (to afs_dcache and afs_segments respectively)
statistics code is now in a seperate function of its own.

Reviewed-on: http://gerrit.openafs.org/122
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-25 19:21:55 -07:00
Jeffrey Altman
4b10f1916c Windows: MSRPC_WriteMessage, call MSRPC_NewCall at right time
In MSRPC_WriteMessage, call MSRPC_NewCall after checking for
MAX_RPC_MSG_SIZE and return an error if MSRPC_NewCall fails.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/339
Reviewed-by: Asanka Herath <asanka@secure-endpoints.com>
Tested-by: Asanka Herath <asanka@secure-endpoints.com>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-25 13:25:18 -07:00
Felix Frank
1692c27300 Cleanups in rxfs_storeInit()
Reviewed-on: http://gerrit.openafs.org/121
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-24 20:37:21 -07:00
Felix Frank
0e1bd20462 More CacheStoreProc call context to afs_fetchstore from afs_segments
The loop over the dcaches is performed in afs_fetchstore now as well,
in a new routine afs_CacheStoreVCache that is called in afs_CacheStoreProc's
stead. The original afs_CacheStoreProc has largely moved to
afs_CacheStoreDCaches in afs_fetchstore.c.

Enhances readability of afs_StoreAllSegments. rxfs_storeInit() can be
performed earlier now, which is instrumental for the inclusion of
alternative protocols (which will call other storeInit() functions).

Reviewed-on: http://gerrit.openafs.org/120
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-24 20:04:53 -07:00
Felix Frank
334114ac58 Limited the scope of some local variables in afs_CacheStoreProc
Reviewed-on: http://gerrit.openafs.org/119
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-24 19:41:20 -07:00
Felix Frank
beb57af06f Open/close connections in afs_fetchstore rather than afs_segments
The respective snippets are moved from afs_StoreAllSegments to
rxfs_storeDestroy and rxfs_storeInit respectively. This makes sense
and enhances readability.

Reviewed-on: http://gerrit.openafs.org/118
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-21 17:49:49 -07:00
Simon Wilkinson
74d53f288c PAG numbers are unsigned (and other warnings)
PAG numbers are represented by afs_uint32. Fix assorted call sites to
reflect this.

While we're here, prototype ktc_newpag, and fix the casts used by
the ktc_curpag() call.

Reviewed-on: http://gerrit.openafs.org/167
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-21 16:34:49 -07:00
Simon Wilkinson
50f6529cf1 Make lockIDSet readable by mortals
The lockIDSet function was a sprawling mess of #ifdefs. This change
consolidates it so there is completely different function for each
platform, with some internal #ifdefs remaining for platform-specific
variations. The side effect of this is that the function is now readable.

Reviewed-on: http://gerrit.openafs.org/164
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-21 16:30:55 -07:00
Simon Wilkinson
0666fe12fa Fix printfs of time for Darwin
Darwin uses a fancy type for tv_sec, which needs to be cast to long
before it can be printed.

Reviewed-on: http://gerrit.openafs.org/165
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-21 14:16:26 -07:00
Felix Frank
01349d8a94 Move statistics and and call-closing from afs_segments to afs_fetchstore.
Rather than in afs_StoreAllSegments, do statistics in afs_CacheStoreProc,
further boosting the former's readability. Calls are closed in a new
storeOp now, rxfs_storeClose, and have been moved there from
afs_StoreAllSegments as well.

Reviewed-on: http://gerrit.openafs.org/117
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Russ Allbery <rra@stanford.edu>
Reviewed-by: Russ Allbery <rra@stanford.edu>
2009-08-21 00:04:10 -07:00
Jeffrey Altman
db717c69da Windows: Correct buf_Write data loss errors
buf_Write has two errors in it.  First, if offset at which the
write is supposed to begin does not align with the cm_data.blockSize,
then (offset %blockSize) octets at the end of the write will fail
to be written to the file server.   Second, if there is no knowledge
of the file server's ability to handle RXAFS_StoreData64 calls
and the file server does not support StoreData64 calls, then the
first chunkSize of data will be written to the file server before
the RXGEN_OPCODE error is detected.  In this case a retry using
RXAFS_StoreData is performed but the values of 'nbytes' and 'qdp'
were not reset before the retry.  As a result, the first chunkSize
of data would fail to be written to the file server.

The second problem can reliably occur when connectivity between the
cache manager and the file server is interrupted due to firewall
port mapping closures.  Upon establishing a new rx connection, a
SRXAFSCB_InitCallBackState3 RPC will be issued by the file server
erasing any prior knowledge of the file server's ability to process
64-bit data operations.  If the first 64-bit operation is a
StoreData64, then data loss will occur.

LICENSE MIT

FIXES 125220
FIXES 108199

Reviewed-on: http://gerrit.openafs.org/332
Tested-by: Asanka Herath <asanka@secure-endpoints.com>
Reviewed-by: Asanka Herath <asanka@secure-endpoints.com>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-20 15:40:29 -07:00
Jeffrey Altman
687a40c7c6 Windows: cleanup smb polution of non-smb modules
There are a variety of functions that are declared in smb modules
that are used within non-smb modules.   In order to reduce some of
the polution, several time conversion functions and the thread
priority operations are migrated to cm_utils.  As part of this
migration the unused !USE_NUMERIC_TIME_CONV time implementations
are removed.

In addition, an unnecessary use of an SMB preprocessor symbols
is removed from cm_freelance.c and smb_ioctls.h is no longer
included within afsd.h.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/330
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-20 14:38:51 -07:00
Jeffrey Altman
ef197772e7 Windows: Init root.afs before smb_Init
It is not safe to initialize the root.afs cm_scache object
after the SMB package.  Break it out of afsd_InitDaemon()
into its own afsd_InitRoot() and call from afsd_InitCM().

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/331
Tested-by: Asanka Herath <asanka@secure-endpoints.com>
Reviewed-by: Asanka Herath <asanka@secure-endpoints.com>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-20 14:35:48 -07:00
Russ Allbery
da0673f0ea Describe 's' PTS group permissions more accurately
PTS groups with 's' access permissions can be examined by members of the
group and the owner of the group in addition to system:administrators.
State this in the manual page.

Reviewed-on: http://gerrit.openafs.org/327
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-20 12:11:13 -07:00
Jeffrey Altman
8b5846b080 Windows: Ensure consistency of error code mapping
For Windows, the error mapping table used throughout the system
is located in util/errmap_nt.h.   This table was incomplete and
WINNT/afsd/cm_utils.c added additional mapping locally.  Move
said error mapping to util/errmap_nt.h.

When including afs/unified_afs.h it must be done after including
afs/errmap_nt.h.   Otherwise, undefined errors are mapped to EIO
which should not be.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/328
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-20 06:47:48 -07:00
Jeffrey Altman
995c76e73e Windows: Add prototype for osi_SetLockOrderValidation
Add prototype for osi_SetLockOrderValidation() defined
in osibasel.c

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/329
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-20 06:47:20 -07:00
Felix Frank
5ff34cc364 Move context of afs_CacheStoreProc() call from afs_segments to afs_fetchstore
The innermost loop in afs_StoreAllSegments (looping over chunks) is now
inlined in afs_CachStoreProc. This is step one in a series of such
inlinings.

Reviewed-on: http://gerrit.openafs.org/116
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-19 19:47:19 -07:00
Felix Frank
ebd4249639 Move statistics keeping from afs_dcache to afs_fetchstore.
More context of the afs_CacheFetchProc() call in GetDCache is now inlined
in the afs_CacheFetchProc. Boosts readability of GetDCache and saves
stack, as counter pointers need not be passed anymore. The statistics
code is more readable due to lesser indentation.

Reviewed-on: http://gerrit.openafs.org/115
Reviewed-by: Russ Allbery <rra@stanford.edu>
Tested-by: Russ Allbery <rra@stanford.edu>
2009-08-19 18:37:35 -07:00
Jeffrey Altman
e2ce6a0f40 Windows: cm_conn_t must not be referenced after cm_Analyze is called
The 64-bit Large File Support added a reference to the cm_conn_t
object after cm_Analyze() is called and releases the hold.  This
permits the object to be freed if it is appropriate to do so
which can result in an invalid memory reference.

This change removes the improper reference.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/326
Tested-by: Asanka Herath <asanka@secure-endpoints.com>
Reviewed-by: Asanka Herath <asanka@secure-endpoints.com>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-19 15:35:06 -07:00
Felix Frank
1d78be1694 Some code cleanups and variable renames.
Reviewed-on: http://gerrit.openafs.org/113
Tested-by: Russ Allbery <rra@stanford.edu>
Reviewed-by: Russ Allbery <rra@stanford.edu>
2009-08-19 15:14:53 -07:00
Jeffrey Altman
1c12747818 Windows: Permit build system to update symbol store
Microsoft Debugging Tools for Windows, Visual Studio Debugger,
and SysInternals tools can all make use of a Symbol Server.
  http://msdn.microsoft.com/en-us/magazine/cc163563.aspx
The commit adds functionality to the build system to automatically
add binaries and symbols to a symbol store during the build.
This functionality is only enabled if two environment variables
are defined:

  SYMSTORE_EXE  - specifies the location of symstore.exe
  SYMSTORE_ROOT - specifies the location of the symbol store

an optional environment variable permits an arbitrary comment
to be added to the symbol store history file.

  SYMSTORE_COMMENT - arbitrary text to be added to the history

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/324
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-19 10:19:43 -07:00
Jeffrey Altman
b48908cbba Windows: Add resource info to threaded volserver.exe
1.5.60 and 1.5.61 shipped without resource information
linked to the threaded version of the volserver.exe
executable.  Correct that oversight.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/325
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-19 10:16:39 -07:00
Simon Wilkinson
57e218ab61 Make MacOS installer pane permit dashes
The MacOS installer only permits characters in [0-9A-Za-z] in cellnames.
However, OpenAFS itself will also allow dashes, and they're in use by some
sites.

This trivially amends the installer pane to permit dashes.

FIXES 125211

Reviewed-on: http://gerrit.openafs.org/323
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-18 15:05:02 -07:00
Michael Meffie
f819561707 Fix stack corruption on 64 bit linux when md5inum is enabled
Fix a stack overflow on 64 bit linux which corrupts the stack
saved arguments and causes a kernel panic. This bug is seen
on platforms where ino_t is 64 bits and the md5inum sysctl
option is enabled.

Reviewed-on: http://gerrit.openafs.org/322
Tested-by: Derrick Brashear <shadow@dementia.org>
Reviewed-by: Derrick Brashear <shadow@dementia.org>
2009-08-18 12:13:21 -07:00
Jeffrey Altman
cbaf97a953 Windows: Be more conservative about checking error conditions
It has been reported that winlogon.exe is crashing on some
systems.  The reports indicate that the failure is somewhere
in GetLogonDomainOptions.  This commit ensures that we are
more conservative about the assumptions that are made regarding
which Lsa operations can fail.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/321
Reviewed-by: Asanka Herath <asanka@secure-endpoints.com>
Tested-by: Asanka Herath <asanka@secure-endpoints.com>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
Tested-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-18 10:02:57 -07:00
Jeffrey Altman
4eee2f24a4 Windows: src/WINNT/afsd cleanup
Update the "clean" rule to remove the files generated from
ms-srvsvc.idl and ms-wkssvc.idl.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/320
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-17 16:59:34 -07:00
Jeffrey Altman
fa07f46db9 Windows: Prevent smb_StartedLock use before initialization
The smb_StartedLock osi_mutex can be obtained before
initialization due to a race if the IP address configuration
is in a state of flux as afsd_service is starting up.
To avoid the problem call cm_InitDaemons() after smb_Init().

This race is the cause of the periodic assertions in some
environments when lock order validation is enabled.

LICENSE MIT

Reviewed-on: http://gerrit.openafs.org/318
Tested-by: Jeffrey Altman <jaltman@openafs.org>
Reviewed-by: Jeffrey Altman <jaltman@openafs.org>
2009-08-16 05:59:49 -07:00