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

r24403: Remove reply_prep_legacy from send_nttrans_replies

r24402: Fix a 64-bit warning

r24401: Push reply_prep_legacy into send_nt_replies

r24400: Fix a valgrind bug

r24399: Change change_notify_reply to use send_nt_replies_new

r24398: Convert call_nt_transact_notify_change to the new API

r24394: Convert call_nt_transact_rename to the new API

r24393: Convert call_nt_transact_set_user_quota to the new API

r24392: Minor simplifications

r24391: Convert call_nt_transact_get_user_quota to the new API

r24388: - ACL retrieval provided incomplete information because the buffer pointer was
 incremented too far in some circumstances.  In these cases, only the first
 of multiple concatenated strings would be seen.

- Working on bug 4649 pertaining to delete an ACL, this fixes the reported
 crash.  It appears to have been an incomplete switchover from malloc to
 talloc, as the memory was still being freed with SAFE_FREE.

 Deleting ACLs still doesn't work.  Although a valid request is sent to the
 server and a SUCCESS response is returned, the method that's used in
 libsmbclient for deleting ACLs seems to be incorrect.  In looking at the
 samba4 torture tests, it appears that we should be turning on the INHERIT
 flag if we want to delete the ACL.  (I could use some assistance on the
 proper flags to send, from anyone familiar with this stuff.)

- Apply patch from SATOH Fumiyasu to fix bug 4750.  smbc_telldir_ctx() was not
  returning a value useful to smbc_lseekdir_ctx().

Derrell

r24387: Convert readbraw to the new api. Volker, keep checking in
your patches please :-). I'll work on SMBreadBmpx tomorrow.
Jeremy.

r24386: Piss off Volker (not deliberately) by checking in a reply_mknew -> conversion.
Sorry vl, remove one of your 15 patches :-).
Jeremy.

r24385: Convert call_nt_transact_query_security_desc to the new API

r24384: Convert call_nt_transact_set_security_desc to the new API

r24383: Convert call_nt_transact_ioctl to the new API