r24636: Raise one debug level, since the callers (that want to do so)
issue a level 0 debug message.

Michael
(This used to be commit c0022ff9278a06b92a550cf0d6a8e75af31a673c)

r24632: Fix build warnings.

Guenther
(This used to be commit e9178af01d68b7cbe91157d1d196f2148eeef7ca)

r24630: Store Samba configuratin options only under the default name, not as aliases.
This prevents creation of problematic configurations from registry editors
like regedit or "net rpc registry".

I will refactor the code to be somewhat more concise,
but I wanted to have this in the tree, now I got it working... :-)

Michael
(This used to be commit 4424a030324ab646a025b50fcb89e1b5cdfcbbe8)

r24629: Make read_sock return the total number of bytes read instead
of the number of bytes read in the last of possibly several
read calls.

This was noted by Metze.

Michael
(This used to be commit 0193a49223c6314e2834c89fff9920ae7edc4f8a)

r24627: merge from SAMBA_4_0:
as TALLOC_ABORT() is defined to abort() by default
wrap it into a function so that the function name
in the backtrace shows what happens.

metze
(This used to be commit ddbe971030070ab1b9fc37b8b0ac44a3d9303fe4)

r24625: add '-D' option to winbindd

TODO: don't allow '-i -D' and '-D -i' on all
      server binaries in the SAMBA_3_2 branch!
      The &server_mode patch makes this difficult to check...

metze
(This used to be commit 102bb0fc17d6dec102d628f1051675a1ae5a9c27)

r24621: - deferr calling build_options();exit(0);
- use poptPrintUsage() to give the user more info

metze
(This used to be commit a95d9d1ef99d6a2f77a289f8d2011cae482821b1)

r24618: move printing out the version string and exit()
into the popt _POST processing.

Now 'smbd -V --bla' complains about an unknown option

metze
(This used to be commit c115de4beac887442a73b4a85367d7b5fded68e6)

r24616: In set_boolean(), only pass the result back to the caller
if parsing of the boolean string was successful.
Also, initialize the local result variable
(although not strictly necessary anymore, now.)
(This used to be commit bf0daa74fadcd506b80c29d04ccf6ec7345ee0e1)

r24615: Say for which domain the getdcname request failed.

Guenther
(This used to be commit 1ce1958f7a5f41072ad1e2f1fb55f3cd4ebe4b14)

r24610: Add a debug message that the workaround has been activated...
(This used to be commit a12f3bf6e93f7a902a1f6274d67f15cc4eeb20bd)

r24609: Fix the fix of r23668 for win2k giving one zero byte
instead of a 2-byte zero character. I can't recall what
rode me when I put that "2" there. But now I think I
have got it right... :-)

Michael
(This used to be commit fa010bef11b78ac3bbf0091870ce8cd5a53334af)

r24605: remove some stuff samba3 doesn't need any more

metze
(This used to be commit 5514e8487fee453b730a966ebc6fcdcd56da047a)

r24603: In case of error we need to free prefix_cache otherwise on re-init the first
statement will return positively but prefix_cache_hashes would be NULL
(This used to be commit fdc20894a0deb3c68b834e5d9d466873ca634bed)

r24602: Add function lp_string_is_valid_boolean() to check if a string
contains a correct representation of a boolean value (in the
understanding of loadparm.c).

Make set_boolean() catch passing NULL for the boolean target.

Michael
(This used to be commit d13eaa60f504987445b7333ef6972491c9483e6a)

r24601: Fix if statements where we still expected cli_connect() to return BOOL.
(This used to be commit 12cb06d6ad73c37b9ee4bdf31f88dd3ce8cc3935)

r24599: patch from Karolin Seeger <ks@sernet.de>:

smbd, nmbd and winbindd can be started with invalid options currently.
The first patch attached would be a possible solution.
It contains an exit if an invalid option has been used. The main problem
is, that existing setups with wrong options or missing arguments in start
scripts will break (which is the right behaviour from my point of view).

metze
(This used to be commit 8532e3182ab44d4ac84823e9798293f156192aaf)

r24597: - talloc request structure for the samba3 server bindings
  and make that the primary context for the request
  which the implementations can also use.
- go via functions pointers in the ndr_interface_table
  instead of calling functions directly.

metze
(This used to be commit 5c4d998300d0c9836eb3cc6c3cd8ee4f262396b8)

r24595: Fix Coverity ID 393

In this error case we would have used "start" not having it initialized
(This used to be commit 751834fff92e85a424152c1642b238daa3060dbd)

r24594: pass down the ndr_interface_table in the samba3 client bindings
instead of the pull and push functions

metze
(This used to be commit 5e3d4df9bca069708d72f548dc5ddfc7708ac122)

r24590: Reformatting to coding standards. Added my (C) in places it already should
have been :-).
Jeremy.
(This used to be commit 41611a22ed852bb74e2ef3f45766c0580ffd3a18)

r24589: Refactor our vuid code so that we keep the same
vuid that was allocated whilst the connection is
being constructed and after the connection has been set up.
This is what Windows does and at least one client
(and HP printer) depends on this behaviour. As it
depends on the req struct not yet ported to SAMBA_3_2_0
(Volker, hint hint.... :-) I am not yet adding this
to that branch, but will investigate that tomorrow.
Jeremy.
(This used to be commit a54f2805df92c67e74a6764568eedebe394fd500)

r24584: Merge all pam post-processing code (in particular all extra_data code) into
append_data().

Guenther
(This used to be commit 8c56ee2c58ef5aab64292d4420947ea0a5e2766f)

r24583: Make sure we don't accept invalid request options.
Thanks to Michael for his bit-magic.

Guenther
(This used to be commit 8a493cce98c5fcc2e498df9ef1ac0f1e3ee6a75e)

r24579: Merge error reporting to the end of winbindd_pam_auth().

Guenther
(This used to be commit d5bd8b1d618828fe390ca76120863316606dc250)

r24578: Fix build warning.

Guenther
(This used to be commit 27ecb0df792b54d167a829f23bd82c27e0731f0d)

r24573: Fix build without LDAP. Thanks Volker for pointing this out.

Guenther
(This used to be commit 1bae53e39f86a9b3b277578a71b906ca1cd0e4fa)

r24572: Allow for functions map_parameter_canonical() and is_synonym_of() to
be called with inverse == NULL.

Add a new function lp_parameter_is_canonical() to check whether a
parameter name is the canonical name (as apposed to an alias).

Michael
(This used to be commit 07dc0fecf5116b34e468bddae8b36920c492717a)

r24571: Only look at errno if the close call actually failed

Patch from Ofir Azoulay <Ofir.Azoulay@expand.com> -- thanks
(This used to be commit 888e657d758173c0eb4b68059d6fb5ae45b2b2ed)

r24564: regenerate after pidl changes...

metze
(This used to be commit ec8858c37482f0d2ac1291d9a9df00dace6944a8)

r24561: merge from SAMBA_4_0:
rename some DCERPC_ prefixes into NDR_

metze
(This used to be commit 8f07b8ab658ae3d63487ca5fb20065318cdd9d0e)

r24558: merge from SAMBA_4_0:
rename 'dcerpc_table_' -> 'ndr_table_'

metze
(This used to be commit 4e5908cd596f97d3bb73bd9c0bf3d360944f7810)

r24552: merge from SAMBA_4_0:
rename dcerpc_interface_table -> ndr_interface_table
rename dcerpc_interface_list  -> ndr_interface_list

and move them to libndr.h

metze
(This used to be commit f57d23d0f1b1c7a435f3a4ad801e58519cc92a77)

r24550: Attempt to fix the non-LDAP build

For example host "bigboy" doesn't have it.

Günther, please fix it properly and merge to 3_2_0

Thanks,

Volker
(This used to be commit 0d294267414b12c31afd316c39689984e5047865)

r24549: Fix unix_convert to return the already converted part

This API will change anyway when moving away from pstrings.

It took so long to fix, because that rename bug I just fixed gave make
test failures that had nothing to do with this one.

I have samba4 tests for both bugs, will check them in when the build
farm has caught up
(This used to be commit d4f442ed9b145990af4db26859663954ddd70926)

r24548: Fix the case-changing renames

This was broken when I changed reply_mv to wrap in a open_file_ntcreate
call, unix_convert on the destination was called twice
(This used to be commit fddc9db91175bdb0b7ac6a636f8bef918bd7c1b4)

r24547: Fix #4897, patch from David S. Collier-Brown <davecb@spamcop.net> -- Thanks!
(This used to be commit 4a90264d173ef5a870f2a44554c3bb9e738e98fb)

r24543: Apply missing portion of correction for bug 4750
(This used to be commit 5a83c306bb80b492a3c3d5e86b0767dc45e5c262)

r24541: merge from SAMBA_4_0:
rename struct dcerpc_endpoint_list/struct dcerpc_authservice_list
into ndr_interface_string_array and move it to libndr.h

metze
(This used to be commit 9fec0d6c2ceaf66752baa5c8a34821bef2c5b833)

r24538: merge from SAMBA_4_0:
rename struct dcerpc_interface_call -> struct ndr_interface_call
and move it to librpc/ndr/libndr.h

metze
(This used to be commit 24e096b3659c3070a1ce029174fba51ae59e89ad)

r24537: regenerate after pidl fixes

metze
(This used to be commit e827c7311ce9890358be145099391c6b3cee15a1)

r24533: merge from SAMBA_4_0:
rename dcerpc_syntax_id -> ndr_syntax_id

metze
(This used to be commit 7facd9bf3ffd0718bc9f9161274d18a0014f3744)

r24527: Add a function lp_canonicalize_parameter_with_value that turns
a parameter and value into the canonical paramter with the value
inverted if it was in invers boolean synonym.

Make net conf use this function when storing parameters.

Michael
(This used to be commit 3b762ab18392fd06427957b0263262e3b8e34b9d)

r24526: Refactor determiniation of the index of the canonical form of a
parameter given in string representation into a function of its
own (map_parameter_canonical).

Michael
(This used to be commit 0dfb5eee25e287c129f7d34e4d079c60d56dce59)

r24525: Add output of synonym information to show_parameter_list().
Also simplify lp_canonicalize_parameter by making use of
the new function "is_synonym_of".

Michael

Note: If anything depends on the exact former output format of
show_parameter list, I would trigger the output of synonym information
by a boolean verbose switch (that could be passed to testparm
via the "-v" command line switch).
(This used to be commit fd2dbae82594c6d6ab10a7318f127aea0a9a61ea)

r24519: Refactor printing of one parameter into a helper function of its own.
(This used to be commit 70dfe5d098b5d815e652b82fca57be0eb826f820)

r24513: Reformatting: eliminate trailing spaces, overly long lines
and adjust some formattings to standard.

Michael
(This used to be commit 00432bcd69bb1c30774af277af1dc7271380d75b)

r24512: Fix RAW-NOTIFY

With the inbuf consistency checks we have to more correctly fake the
inbuf
(This used to be commit 2b8ecda30fc57bb4027b7208d22d85d213074158)

r24510: Fix logic. Thanks to Volker for the hint!

Michael
(This used to be commit ce1d7ad1acb48a254add3ddfac5f13e05b8ec646)

r24501: Added bcc test for reply_tcon & removed Vl's comment :-).
Jeremy.
(This used to be commit 9d9ed41f2139051578c35f80112640cffb5f7608)

r24500: Add check that bcc is correct in an incoming packet.
Jeremy.
(This used to be commit 267a0ac48d358e1bd10d92238352ca3d867a58d4)

r24498: More extra code into a function, reply_openerror.
Jeremy.
(This used to be commit 43ddfb8c918bd27e2efd3b54077db815da80a53a)

r24497: Revert r24485 -- this needs more thought
(This used to be commit 01f701ba1ea9859b090aaf8f4ae9722c5502316c)

r24496: Add initial synonym handling to "net conf":
When storing parameters in the smbconf portion of the registry,
up to now, synonyms could be misused to store a parameter twice.
Now this is prevented by canonicalizing the paramter name first.

Also, the value for a boolean parameter checked for validity
before storing the bool in registry.

(The canonicalization should finally go into the registry smbconf
code to also prevent e.g. "regedit" or "net rpc registry" from
storing synonyms. - This is in the making.)

Michael
(This used to be commit 95447dde860f54d835b3fe90d95fe60231788d1b)

r24495: Add a function lp_canonicalize_parameter: It takes a name of a parameter
and produces the "canonical" (or main) name of the parameter (the one
synonym that does not have the flag FLAG_HIDE). The function also sets
a flag as to whether the synonym is a reverse boolean synonym.

Add some functions for the handling of string representations of boolean
values: return the canonical string representation of a bool, invert
a bool given as a string, canonicalize a bool given as a string.

Michael
(This used to be commit 113ac07199aaf2271af6eba2611dad429ee4c416)

r24485: Even in the failure case unix_convert must pass the already converted
part of the string to the caller.
(This used to be commit 301ae3d68bc64c9526eb7d4458584a5ebb151137)

r24481: copy .bzrignore => .gitignore

metze
(This used to be commit 51507a0d7674301de01e3d5b107f13834a078dc0)

r24476: Fix the mappings in reply_opeXXX calls. Now to test renames.
Jeremy.
(This used to be commit 74d10b09a68f5c06d6b3ceffe0a40818dc84106e)

r24475: Fix the error mapping for SMB_POSIX_PATH_OPEN.
Jeremy.
(This used to be commit 6dac315ba1d9b9582c2b573e0785b2f6d627f846)

r24474: Map error value NT_STATUS_OBJECT_NAME_COLLISION
to ERRDOS, ERRfilexists for NTCreateX and NTTransCreate.
Jeremy.
(This used to be commit 5b2316d1734765ef6c31ff09beea974d9629d17f)

r24470: Start fixing up the mapping of OBJECT_NAME_COLLISION
to DOS error ERRDOS, ERRfilexists on open calls.
Jeremy.
(This used to be commit 4674486450bbe79ec9b22bcbc23c534365d6798a)

r24468: Don't forget the +4 for length :-).
Jeremy.
(This used to be commit 4d01e0b7443aa4e4d450d705df5762d801172809)

r24467: Do range checking on incoming smb request.
Jeremy.
(This used to be commit dbd58dd647279def1681d88701e41d8a738c680a)

r24466: - Sort ACEs according to support.microsoft.com/kb/269175 so that
  Windows Explorer doesn't complain about the order (and so that they get
  interpreted properly).

Derrell
(This used to be commit 8f371e2ea97a3b58d1c7c3aa1368a0904295f681)

r24464: Now Volker removed the readbmpx we don't need cached errors any more.
Jeremy.
(This used to be commit 9256ec0a20f532c7dd7ddc2d3534336a47e6c2d2)

r24462: - Removing all ACEs was causing removal of the DACL entirely.  Win2000 ignored
  the request, presumably due to the PROTECTED flag not being set.  Setting
  that flag (in make_sec_desc()) has much wider implications than just to
  libsmbclient, so instead of modifying that, we'll remove security
  descriptors by setting the number of ACEs to zero.  At some point, we might
  want to look into whether we should actually be setting the PROTECTED flag
  in the DACL.

  Reference http://www.microsoft.com/technet/prodtechnol/windows2000serv/reskit/distrib/dsce_ctl_qxju.mspx?mfr=true

Derrell
(This used to be commit 319df380e579fd860348a8f08a584e13161dde9d)

r24461: Fix Bug 4852, thank to anto <sistemac@prosoft.ba> for reporting it.
(This used to be commit 0fecd8a0c3aaa64e137d2efd3f9cc7705837ea2a)

r24458: regenerate pidl output after pidl fixes:

fixes winreg_EnumValues()

metze
(This used to be commit cedf7022c5c61ed5eb49bb1cb24b062858f7d2fd)

r24457: Convert reply_tcon to the new API

Jeremy, I really apologize for doing this, but I just wanted to enjoy
converting the last SMB call :-)

I've left one little task for you there, I'm not certain that checking
the inbuf length is correct here.

Volker
(This used to be commit 1e08fddafda11961f8855423b29c1f8a9a6b4457)

r24453: Remove the read and write bmpx calls

Talked to both Tridge and Jeremy about this, Tridge said that there is a
special error message persuading OS/2 to fall back to other methods.
The calls now checked in always return the error message we used to
return when "read bmpx = False" was set (the default): ERRSRV, ERRuseSTD.

If someone has a reproducable test case where this is really needed, we
can always dig it up from version control and convert it to the new API.
But that time without that silly parameter, and with a torture test case
for "make test" please :-)

Volker
(This used to be commit d941aae2dfd11609e807bf4ce712571a2e354627)

r24450: merge from SAMBA_4_0:
length_is() isn't supported without size_is().

I assume what we need is an array of strings,
so the code isn't used and broken anyway without
a testsuite...just get rid of the pidl warning

metze
(This used to be commit a3fd68d48582865923e18e8412ac7dca7f725c64)

r24445: Convert SMBwritebraw. No test suite unfortunately.... I need to write one for this.
Jeremy
(This used to be commit edc17dfcbd21cccaffb76f4ae67fe4b06520f1a9)

r24444: Convert reply_find[n]close to the new API
(This used to be commit 6af85953465789594342f2d199ef5d29462cf34f)

r24443: Convert reply_search/fclose to the new API
(This used to be commit a8a33c377e38046b4103cf1d59032b97ab6bac9a)

r24442: Convert reply_copy to the new API
(This used to be commit 0cb00c54750837ab2d2dc12e4947fedb7d38e878)

r24441: Convert reply_ioctl to the new API
(This used to be commit a5af7ebb7f1d869659fbab187652e68ec4fafbb8)

r24440: Convert the reply_sendXX functions to the new API
(This used to be commit a64bc31098de8694b79eeafd3a226cf519700707)

r24439: Convert reply_get/setattrE to the new API
(This used to be commit 6b0ad071d85ddd8fbf24386db11688bde49baf81)

r24438: Use dump_data_pw() instead of dump_data() on sensitive data in winbindd.

Guenther
(This used to be commit adaa5e423dd022e3dc2cbe657146f3978fb59de3)

r24437: Remove "done" in winbindd_raw_kerberos_login() as well.

Guenther
(This used to be commit 44c10bbe9477fc9e731f319e62c1358a7497e316)

r24436: Convert reply_lockread/writeunlock to the new API
(This used to be commit 1b6add251ca1db565a03407db30884132dd93e7d)

r24435: Fix typo.

Guenther
(This used to be commit 8705f890b11e7158b5c77cbd3a6dfae8eb26776e)

r24434: Simplify winbindd_raw_kerberos_login().

Guenther
(This used to be commit 98ffae031ae641fb0ba7f2e972ab731dbe114a5f)

r24433: Convert reply_ntrename to the new API
(This used to be commit 42b07a52b9f4fe147426e9156958cb4d33691678)

r24432: Expand kerberos_return_pac() so that it can be used in winbindd.

Guenther
(This used to be commit e70bf0ecc3ec6d3ba8ba384024bbdf9a783072ea)

r24431: Convert the reply_printXX calls to the new API
(This used to be commit e528479f56bc936cc60eb95c9738a48de48dbd05)

r24430: Convert reply_writeclose to the new API
(This used to be commit 6def2ee03bb3510f000b1977c4d5293cad4ae364)

r24429: reply_setdir is not used...
(This used to be commit ca27a718b07949959efaeb49f310123de76af9a0)

r24428: Convert reply_unlock to the new API
(This used to be commit 01c7426fc0e2af6a955dcb37111ca439fc913d5f)

r24427: Convert reply_lock to the new API
(This used to be commit 212f43ddeaa6369e5437ed3915eb066a37bc9014)

r24426: Convert reply_read to the new API
(This used to be commit 30aada0ef8e16ce94035039b63ab140d158009d9)

r24425: Convert reply_write to the new API
(This used to be commit 244965f7b67becb85774311e6ce84318d554384d)

r24424: Fix the build.

Guenther
(This used to be commit 029bf26f8a571ae060f7be60fd3e8c61d86004f7)

r24423: Convert reply_lseek to the new API
(This used to be commit bd228853863ce5b4b9b974347c50c956d7f2e055)

r24422: Convert reply_ctemp to the new API
(This used to be commit 3cc22fd74f0ffc72f4340a963b4eca7178be2192)

r24421: Ignore long #defines (like for registry keys).

Guenther
(This used to be commit b310b1dfd69ba0470b66eb84007f28e5e69e92e7)

r24420: Pure pedantism: the talloc_init could have failed...

Guenther
(This used to be commit e6415c523234f60da7dcfa4de101e8634dc6927d)

r24419: Hm, I think this protoype is not required here.

Guenther
(This used to be commit 42abe8450b0fc3c9f8d768fadbd5e6b0a6c1bfc3)

r24418: Adding reg_deleteallvalues().

Guenther
(This used to be commit f7457c6173107f1730ae2e31c5377df3dbd2454e)

r24417: Convert reply_setatr to the new API
(This used to be commit 56bab5ea4cbcf8e46101053bd68f66691fd737c5)