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

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

metze

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

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

r24601: Fix if statements where we still expected cli_connect() to return BOOL.

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

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

r24595: Fix Coverity ID 393

In this error case we would have used "start" not having it initialized

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

metze

r24590: Reformatting to coding standards. Added my (C) in places it already should
have been :-).
Jeremy.

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.

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

Guenther

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

Guenther

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

Guenther

r24578: Fix build warning.

Guenther

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

Guenther

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

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

Patch from Ofir Azoulay <Ofir.Azoulay@expand.com> -- thanks

r24564: regenerate after pidl changes...

metze

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

metze

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

metze

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

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

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

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

r24547: Fix #4897, patch from David S. Collier-Brown <davecb@spamcop.net> -- Thanks!

r24543: Apply missing portion of correction for bug 4750

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

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

metze

r24537: regenerate after pidl fixes

metze

r24533: merge from SAMBA_4_0:
rename dcerpc_syntax_id -> ndr_syntax_id

metze

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

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

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).

r24519: Refactor printing of one parameter into a helper function of its own.

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

Michael

r24512: Fix RAW-NOTIFY

With the inbuf consistency checks we have to more correctly fake the
inbuf

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

Michael

r24501: Added bcc test for reply_tcon & removed Vl's comment :-).
Jeremy.

r24500: Add check that bcc is correct in an incoming packet.
Jeremy.

r24498: More extra code into a function, reply_openerror.
Jeremy.

r24497: Revert r24485 -- this needs more thought

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

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

r24485: Even in the failure case unix_convert must pass the already converted
part of the string to the caller.

r24481: copy .bzrignore => .gitignore

metze

r24476: Fix the mappings in reply_opeXXX calls. Now to test renames.
Jeremy.

r24475: Fix the error mapping for SMB_POSIX_PATH_OPEN.
Jeremy.

r24474: Map error value NT_STATUS_OBJECT_NAME_COLLISION
to ERRDOS, ERRfilexists for NTCreateX and NTTransCreate.
Jeremy.

r24470: Start fixing up the mapping of OBJECT_NAME_COLLISION
to DOS error ERRDOS, ERRfilexists on open calls.
Jeremy.

r24468: Don't forget the +4 for length :-).
Jeremy.

r24467: Do range checking on incoming smb request.
Jeremy.

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

r24464: Now Volker removed the readbmpx we don't need cached errors any more.
Jeremy.

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

r24461: Fix Bug 4852, thank to anto <sistemac@prosoft.ba> for reporting it.

r24458: regenerate pidl output after pidl fixes:

fixes winreg_EnumValues()

metze

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

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

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

r24445: Convert SMBwritebraw. No test suite unfortunately.... I need to write one for this.
Jeremy

r24444: Convert reply_find[n]close to the new API

r24443: Convert reply_search/fclose to the new API

r24442: Convert reply_copy to the new API

r24441: Convert reply_ioctl to the new API

r24440: Convert the reply_sendXX functions to the new API

r24439: Convert reply_get/setattrE to the new API

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

Guenther

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

Guenther

r24436: Convert reply_lockread/writeunlock to the new API

r24435: Fix typo.

Guenther

r24434: Simplify winbindd_raw_kerberos_login().

Guenther

r24433: Convert reply_ntrename to the new API

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

Guenther

r24431: Convert the reply_printXX calls to the new API

r24430: Convert reply_writeclose to the new API

r24429: reply_setdir is not used...

r24428: Convert reply_unlock to the new API

r24427: Convert reply_lock to the new API

r24426: Convert reply_read to the new API

r24425: Convert reply_write to the new API

r24424: Fix the build.

Guenther

r24423: Convert reply_lseek to the new API

r24422: Convert reply_ctemp to the new API

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

Guenther

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

Guenther

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

Guenther

r24418: Adding reg_deleteallvalues().

Guenther

r24417: Convert reply_setatr to the new API

r24416: Some more minor edits for libgpo, mostly reformatting and moving of functions.

Guenther

r24415: Convert reply_getatr to the new API

r24414: Convert reply_mv to the new API

r24413: Minor edits for libgpo.

Guenther

r24412: Convert reply_open to the new API

r24410: - I got tricked by function naming.  Contrary to what seemed obvious to me,
  prs_mem_free() is not the function to be called to free memory allocated by
  prs_alloc_mem().  I've added a comment so others may not get bitten too.

- Remove incorrect memory free calls added yesterday to replace SAFE_FREE.
  The memory is actually now on a talloc context, so gets freed by the caller
  when that context is freed.  We don't need to free it iternally.

Derrell

r24409: Check wct in reply_open

r24408: Remove reply_prep_legacy from reply_lockingX

r24406: Push reply_prep_legacy into reply_lockingX

r24405: Check wct in reply_lockingX

r24404: Remove get_OutBuffer usage from blocking.c