svn commit: samba r25744 - in branches/SAMBA_4_0/source/librpc/ndr: .

2007-10-27 Thread metze
Author: metze
Date: 2007-10-27 08:10:14 + (Sat, 27 Oct 2007)
New Revision: 25744

WebSVN: 
http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=revroot=sambarev=25744

Log:
add NDR_ERR_INVALID_POINTER

metze
Modified:
   branches/SAMBA_4_0/source/librpc/ndr/libndr.h
   branches/SAMBA_4_0/source/librpc/ndr/ndr.c


Changeset:
Modified: branches/SAMBA_4_0/source/librpc/ndr/libndr.h
===
--- branches/SAMBA_4_0/source/librpc/ndr/libndr.h   2007-10-26 23:28:36 UTC 
(rev 25743)
+++ branches/SAMBA_4_0/source/librpc/ndr/libndr.h   2007-10-27 08:10:14 UTC 
(rev 25744)
@@ -174,7 +174,8 @@
NDR_ERR_ALLOC,
NDR_ERR_RANGE,
NDR_ERR_TOKEN,
-   NDR_ERR_IPV4ADDRESS
+   NDR_ERR_IPV4ADDRESS,
+   NDR_ERR_INVALID_POINTER
 };
 
 enum ndr_compression_alg {

Modified: branches/SAMBA_4_0/source/librpc/ndr/ndr.c
===
--- branches/SAMBA_4_0/source/librpc/ndr/ndr.c  2007-10-26 23:28:36 UTC (rev 
25743)
+++ branches/SAMBA_4_0/source/librpc/ndr/ndr.c  2007-10-27 08:10:14 UTC (rev 
25744)
@@ -365,6 +365,8 @@
return NT_STATUS_NO_MEMORY;
case NDR_ERR_ARRAY_SIZE:
return NT_STATUS_ARRAY_BOUNDS_EXCEEDED;
+   case NDR_ERR_INVALID_POINTER:
+   return NT_STATUS_INVALID_PARAMETER_MIX;
default:
break;
}



[SCM] Samba Shared Repository - branch v3-2-test updated - initial-v3-2-unstable-127-g3625ed3

2007-10-27 Thread Stefan Metzmacher
The branch, v3-2-test has been updated
   via  3625ed395c23101095321de80e89fcbc8c76be4c (commit)
  from  25bbc9a6613bef0f3f73ecf634a38a9d56020f40 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-2-test


- Log -
commit 3625ed395c23101095321de80e89fcbc8c76be4c
Author: metze [EMAIL PROTECTED]
Date:   Sat Oct 27 08:10:14 2007 +

add NDR_ERR_INVALID_POINTER

metze

git-svn-id: svn+ssh://svn.samba.org/data/svn/samba/branches/[EMAIL 
PROTECTED] 0c0555d6-39d7-0310-84fc-f1cc0bd64818

---

Summary of changes:
 source/librpc/ndr/libndr.h |3 ++-
 source/librpc/ndr/ndr.c|2 ++
 2 files changed, 4 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/librpc/ndr/libndr.h b/source/librpc/ndr/libndr.h
index 4531293..ee27464 100644
--- a/source/librpc/ndr/libndr.h
+++ b/source/librpc/ndr/libndr.h
@@ -174,7 +174,8 @@ enum ndr_err_code {
NDR_ERR_ALLOC,
NDR_ERR_RANGE,
NDR_ERR_TOKEN,
-   NDR_ERR_IPV4ADDRESS
+   NDR_ERR_IPV4ADDRESS,
+   NDR_ERR_INVALID_POINTER
 };
 
 enum ndr_compression_alg {
diff --git a/source/librpc/ndr/ndr.c b/source/librpc/ndr/ndr.c
index 5525459..326ab12 100644
--- a/source/librpc/ndr/ndr.c
+++ b/source/librpc/ndr/ndr.c
@@ -328,6 +328,8 @@ static NTSTATUS ndr_map_error(enum ndr_err_code ndr_err)
return NT_STATUS_NO_MEMORY;
case NDR_ERR_ARRAY_SIZE:
return NT_STATUS_ARRAY_BOUNDS_EXCEEDED;
+   case NDR_ERR_INVALID_POINTER:
+   return NT_STATUS_INVALID_PARAMETER_MIX;
default:
break;
}


-- 
Samba Shared Repository


svn commit: samba r25745 - in branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4/NDR: .

2007-10-27 Thread metze
Author: metze
Date: 2007-10-27 09:57:09 + (Sat, 27 Oct 2007)
New Revision: 25745

WebSVN: 
http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=revroot=sambarev=25745

Log:
[pidl] ndr_pull/push_error(ndr, NDR_ERR_INVALID_POINTER,..)

instead NT_STATUS_INVALID_PARAMETER_MIX

metze

Modified:
   branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4/NDR/Parser.pm


Changeset:
Modified: branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4/NDR/Parser.pm
===
--- branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4/NDR/Parser.pm  
2007-10-27 08:10:14 UTC (rev 25744)
+++ branches/SAMBA_4_0/source/pidl/lib/Parse/Pidl/Samba4/NDR/Parser.pm  
2007-10-27 09:57:09 UTC (rev 25745)
@@ -423,8 +423,10 @@
} elsif ($l-{IS_ZERO_TERMINATED}) { # Noheader arrays
$length = $size = ndr_get_string_size($ndr, 
sizeof(*$var_name));
} else {
-   $length = $size = ParseExprExt($l-{SIZE_IS}, $env, 
$e-{ORIGINAL}, 
-   check_null_pointer($e, $env, sub { $self-pidl(shift); }, 
return NT_STATUS_INVALID_PARAMETER_MIX;), check_fully_dereferenced($e, $env));
+   $length = $size = ParseExprExt($l-{SIZE_IS}, $env, 
$e-{ORIGINAL},
+   check_null_pointer($e, $env, sub { $self-pidl(shift); 
},
+  return ndr_pull_error(ndr, 
NDR_ERR_INVALID_POINTER, \NULL Pointer for size_is()\);),
+   check_fully_dereferenced($e, $env));
}
 
if ((!$l-{IS_SURROUNDING}) and $l-{IS_CONFORMANT}) {
@@ -447,7 +449,10 @@
if ($l-{IS_CONFORMANT} and not $l-{IS_ZERO_TERMINATED}) {
$self-defer(if ($var_name) {);
$self-defer_indent;
-   my $size = ParseExprExt($l-{SIZE_IS}, $env, $e-{ORIGINAL}, 
check_null_pointer($e, $env, sub { $self-defer(shift); }, return 
NT_STATUS_INVALID_PARAMETER_MIX;), check_fully_dereferenced($e, $env));
+   my $size = ParseExprExt($l-{SIZE_IS}, $env, $e-{ORIGINAL},
+   check_null_pointer($e, $env, sub { $self-defer(shift); 
},
+  return ndr_pull_error(ndr, 
NDR_ERR_INVALID_POINTER, \NULL Pointer for size_is()\);),
+   check_fully_dereferenced($e, $env));
$self-defer(NDR_CHECK(ndr_check_array_size(ndr, (void*) . 
get_pointer_to($var_name) . , $size)););
$self-defer_deindent;
$self-defer(});
@@ -457,7 +462,8 @@
$self-defer(if ($var_name) {);
$self-defer_indent;
my $length = ParseExprExt($l-{LENGTH_IS}, $env, 
$e-{ORIGINAL}, 
-   check_null_pointer($e, $env, sub { $self-defer(shift); 
}, return NT_STATUS_INVALID_PARAMETER_MIX;), 
+   check_null_pointer($e, $env, sub { $self-defer(shift); 
},
+  return ndr_pull_error(ndr, 
NDR_ERR_INVALID_POINTER, \NULL Pointer for length_is()\);),
check_fully_dereferenced($e, $env));
$self-defer(NDR_CHECK(ndr_check_array_length(ndr, (void*) . 
get_pointer_to($var_name) . , $length)););
$self-defer_deindent;
@@ -740,7 +746,11 @@
my ($self,$e,$l,$var_name) = @_;
 
if ($l-{POINTER_TYPE} eq ref) {
-   $self-pidl(if ($var_name == NULL) return 
NT_STATUS_INVALID_PARAMETER_MIX;);
+   $self-pidl(if ($var_name == NULL) {);
+   $self-indent;
+   $self-pidl(return ndr_push_error(ndr, 
NDR_ERR_INVALID_POINTER, \NULL [ref] pointer\););
+   $self-deindent;
+   $self-pidl(});
if ($l-{LEVEL} eq EMBEDDED) {
$self-pidl(NDR_CHECK(ndr_push_ref_ptr(ndr)););
}
@@ -875,7 +885,9 @@
 {
my($self,$e,$l,$ndr,$var_name,$env) = @_;
my $switch_var = ParseExprExt($l-{SWITCH_IS}, $env, $e-{ORIGINAL}, 
-   check_null_pointer($e, $env, sub { $self-pidl(shift); }, 
return NT_STATUS_INVALID_PARAMETER_MIX;), check_fully_dereferenced($e, $env));
+   check_null_pointer($e, $env, sub { $self-pidl(shift); },
+  return ndr_pull_error(ndr, 
NDR_ERR_INVALID_POINTER, \NULL Pointer for switch_is()\);),
+   check_fully_dereferenced($e, $env));
 
$var_name = get_pointer_to($var_name);
$self-pidl(NDR_CHECK(ndr_pull_set_switch_value($ndr, $var_name, 
$switch_var)););
@@ -887,7 +899,9 @@
 {
my($self,$e,$l,$ndr,$var_name,$env) = @_;
my $switch_var = ParseExprExt($l-{SWITCH_IS}, $env, $e-{ORIGINAL}, 
-   check_null_pointer($e, $env, sub { $self-pidl(shift); }, 
return NT_STATUS_INVALID_PARAMETER_MIX;), check_fully_dereferenced($e, $env));
+   check_null_pointer($e, $env, sub { $self-pidl(shift); },
+  return ndr_push_error(ndr, 
NDR_ERR_INVALID_POINTER, \NULL 

[SCM] Samba Shared Repository - branch v3-2-test updated - initial-v3-2-unstable-128-g83b3ecd

2007-10-27 Thread Stefan Metzmacher
The branch, v3-2-test has been updated
   via  83b3ecde1312092fd9875a2a8628652ffa6b6aca (commit)
  from  3625ed395c23101095321de80e89fcbc8c76be4c (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-2-test


- Log -
commit 83b3ecde1312092fd9875a2a8628652ffa6b6aca
Author: Stefan Metzmacher [EMAIL PROTECTED]
Date:   Sat Oct 27 11:57:48 2007 +0200

rerun make idl

metze

---

Summary of changes:
 source/librpc/gen_ndr/ndr_dfs.c  |   28 ++-
 source/librpc/gen_ndr/ndr_echo.c |   64 --
 source/librpc/gen_ndr/ndr_epmapper.c |   36 ++-
 source/librpc/gen_ndr/ndr_eventlog.c |   44 +++-
 source/librpc/gen_ndr/ndr_lsa.c  |  468 +-
 source/librpc/gen_ndr/ndr_netlogon.c |  280 +++-
 source/librpc/gen_ndr/ndr_srvsvc.c   |  212 
 source/librpc/gen_ndr/ndr_svcctl.c   |  316 +--
 source/librpc/gen_ndr/ndr_unixinfo.c |   24 ++-
 source/librpc/gen_ndr/ndr_winreg.c   |  240 +-
 source/librpc/gen_ndr/ndr_wkssvc.c   |   60 -
 11 files changed, 1325 insertions(+), 447 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/librpc/gen_ndr/ndr_dfs.c b/source/librpc/gen_ndr/ndr_dfs.c
index 959abdc..bc76339 100644
--- a/source/librpc/gen_ndr/ndr_dfs.c
+++ b/source/librpc/gen_ndr/ndr_dfs.c
@@ -3113,7 +3113,9 @@ _PUBLIC_ NTSTATUS ndr_push_dfs_GetManagerVersion(struct 
ndr_push *ndr, int flags
if (flags  NDR_IN) {
}
if (flags  NDR_OUT) {
-   if (r-out.version == NULL) return 
NT_STATUS_INVALID_PARAMETER_MIX;
+   if (r-out.version == NULL) {
+   return ndr_push_error(ndr, NDR_ERR_INVALID_POINTER, 
NULL [ref] pointer);
+   }
NDR_CHECK(ndr_push_dfs_ManagerVersion(ndr, NDR_SCALARS, 
*r-out.version));
}
return NT_STATUS_OK;
@@ -3167,12 +3169,16 @@ _PUBLIC_ void ndr_print_dfs_GetManagerVersion(struct 
ndr_print *ndr, const char
 static NTSTATUS ndr_push_dfs_Add(struct ndr_push *ndr, int flags, const struct 
dfs_Add *r)
 {
if (flags  NDR_IN) {
-   if (r-in.path == NULL) return NT_STATUS_INVALID_PARAMETER_MIX;
+   if (r-in.path == NULL) {
+   return ndr_push_error(ndr, NDR_ERR_INVALID_POINTER, 
NULL [ref] pointer);
+   }
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 
ndr_charset_length(r-in.path, CH_UTF16)));
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 0));
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 
ndr_charset_length(r-in.path, CH_UTF16)));
NDR_CHECK(ndr_push_charset(ndr, NDR_SCALARS, r-in.path, 
ndr_charset_length(r-in.path, CH_UTF16), sizeof(uint16_t), CH_UTF16));
-   if (r-in.server == NULL) return 
NT_STATUS_INVALID_PARAMETER_MIX;
+   if (r-in.server == NULL) {
+   return ndr_push_error(ndr, NDR_ERR_INVALID_POINTER, 
NULL [ref] pointer);
+   }
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 
ndr_charset_length(r-in.server, CH_UTF16)));
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 0));
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 
ndr_charset_length(r-in.server, CH_UTF16)));
@@ -3309,7 +3315,9 @@ _PUBLIC_ void ndr_print_dfs_Add(struct ndr_print *ndr, 
const char *name, int fla
 static NTSTATUS ndr_push_dfs_Remove(struct ndr_push *ndr, int flags, const 
struct dfs_Remove *r)
 {
if (flags  NDR_IN) {
-   if (r-in.dfs_entry_path == NULL) return 
NT_STATUS_INVALID_PARAMETER_MIX;
+   if (r-in.dfs_entry_path == NULL) {
+   return ndr_push_error(ndr, NDR_ERR_INVALID_POINTER, 
NULL [ref] pointer);
+   }
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 
ndr_charset_length(r-in.dfs_entry_path, CH_UTF16)));
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 0));
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 
ndr_charset_length(r-in.dfs_entry_path, CH_UTF16)));
@@ -3451,7 +3459,9 @@ static NTSTATUS ndr_push_dfs_SetInfo(struct ndr_push 
*ndr, int flags, const stru
NDR_CHECK(ndr_push_charset(ndr, NDR_SCALARS, 
r-in.sharename, ndr_charset_length(r-in.sharename, CH_UTF16), 
sizeof(uint16_t), CH_UTF16));
}
NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, r-in.level));
-   if (r-in.info == NULL) return NT_STATUS_INVALID_PARAMETER_MIX;
+   if (r-in.info == NULL) {
+   return ndr_push_error(ndr, NDR_ERR_INVALID_POINTER, 
NULL [ref] pointer);
+   }
NDR_CHECK(ndr_push_set_switch_value(ndr, r-in.info, 
r-in.level));
NDR_CHECK(ndr_push_dfs_Info(ndr, NDR_SCALARS|NDR_BUFFERS, 
r-in.info));
}
@@ -3592,7 

svn commit: samba r25746 - in branches/SAMBA_4_0/source/selftest/output: .

2007-10-27 Thread metze
Author: metze
Date: 2007-10-27 10:00:44 + (Sat, 27 Oct 2007)
New Revision: 25746

WebSVN: 
http://websvn.samba.org/cgi-bin/viewcvs.cgi?view=revroot=sambarev=25746

Log:
[selftest] make plain output more readable

metze

Modified:
   branches/SAMBA_4_0/source/selftest/output/plain.pm


Changeset:
Modified: branches/SAMBA_4_0/source/selftest/output/plain.pm
===
--- branches/SAMBA_4_0/source/selftest/output/plain.pm  2007-10-27 09:57:09 UTC 
(rev 25745)
+++ branches/SAMBA_4_0/source/selftest/output/plain.pm  2007-10-27 10:00:44 UTC 
(rev 25746)
@@ -122,14 +122,6 @@
}
}
 
-   if ($self-{statistics}-{SUITES_FAIL} == 0) {
-   my $ok = $self-{statistics}-{TESTS_EXPECTED_OK} + 
-$self-{statistics}-{TESTS_EXPECTED_FAIL};
-   print ALL OK ($ok tests in $self-{statistics}-{SUITES_OK} 
testsuites)\n;
-   } else {
-   print FAILED ($self-{statistics}-{TESTS_UNEXPECTED_FAIL} 
failures and $self-{statistics}-{TESTS_ERROR} errors in 
$self-{statistics}-{SUITES_FAIL} testsuites)\n;
-   }
-
print SUMMARY = Skipped tests =\n;
foreach my $reason (keys %{$self-{skips}}) {
print SUMMARY $reason\n;
@@ -140,7 +132,16 @@
}
close(SUMMARY);
 
-   print A summary can be found in $self-{summaryfile}\n;
+   print \nA summary with detailed informations can be found in:\n  
$self-{summaryfile}\n;
+
+   if ($self-{statistics}-{SUITES_FAIL} == 0) {
+   my $ok = $self-{statistics}-{TESTS_EXPECTED_OK} + 
+$self-{statistics}-{TESTS_EXPECTED_FAIL};
+   print \nALL OK ($ok tests in $self-{statistics}-{SUITES_OK} 
testsuites)\n;
+   } else {
+   print \nFAILED ($self-{statistics}-{TESTS_UNEXPECTED_FAIL} 
failures and $self-{statistics}-{TESTS_ERROR} errors in 
$self-{statistics}-{SUITES_FAIL} testsuites)\n;
+   }
+
 }
 
 sub skip_testsuite($$)



Re: [SCM] Samba Shared Repository - branch v3-2-test updated - initial-v3-2-unstable-126-g25bbc9a

2007-10-27 Thread simo

On Fri, 2007-10-26 at 20:05 -0500, Jeremy Allison wrote:
 Ensure temporary memory is freed - pointed out by Li, Ying (ESG)
 [EMAIL PROTECTED].
 We aren't currently leaking memory, but are leaving it around for
 longer than we need to.
 Jeremy.

Thanks for pushing this Jeremy.
Simo.

-- 
Simo Sorce
Samba Team GPL Compliance Officer [EMAIL PROTECTED]
Senior Software Engineer at Red Hat Inc. [EMAIL PROTECTED]



[SCM] Samba Shared Repository - branch v3-2-test updated - initial-v3-2-unstable-130-g9f82a57

2007-10-27 Thread Jeremy Allison
The branch, v3-2-test has been updated
   via  9f82a5766968dc72db5a86ad0afc71e8cffd5be0 (commit)
   via  e17d3e10e860c96b6d5208e5fe51e43b8e58c174 (commit)
  from  83b3ecde1312092fd9875a2a8628652ffa6b6aca (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-2-test


- Log -
commit 9f82a5766968dc72db5a86ad0afc71e8cffd5be0
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Sat Oct 27 12:51:38 2007 -0700

Fix zero_addr(ip, INADDR_ANY); - zero_addr(ip, AF_INET)
from Volker's patch.
Jeremy.

commit e17d3e10e860c96b6d5208e5fe51e43b8e58c174
Author: Volker Lendecke [EMAIL PROTECTED]
Date:   Sat Oct 27 21:15:07 2007 +0200

Two patches

Hi!

Can you check and push them?

Thanks,

Volker

From b488af5905e2dee12a1a72a3b40801ae5c26f24f Mon Sep 17 00:00:00 2001
From: Volker Lendecke [EMAIL PROTECTED]
Date: Sat, 27 Oct 2007 14:20:09 +0200
Subject: [PATCH] Fix some warnings and errors

---

Summary of changes:
 source/Makefile.in   |5 +++--
 source/client/smbmount.c |   10 +-
 source/client/smbumount.c|2 +-
 source/include/interfaces.h  |   36 
 source/lib/interfaces.c  |4 ++--
 source/lib/replace/getpass.c |1 +
 source/smbd/server.c |2 +-
 source/winbindd/idmap_ad.c   |2 +-
 8 files changed, 50 insertions(+), 12 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/Makefile.in b/source/Makefile.in
index a559d63..98fd8f8 100644
--- a/source/Makefile.in
+++ b/source/Makefile.in
@@ -269,11 +269,12 @@ TALLOC_OBJ = lib/talloc/talloc.o
 
 LIB_WITHOUT_PROTO_OBJ = $(LIBREPLACE_OBJ) $(SOCKET_WRAPPER_OBJ) $(TALLOC_OBJ) \
lib/messages.o librpc/gen_ndr/ndr_messaging.o lib/messages_local.o \
-   lib/messages_ctdbd.o lib/packet.o lib/ctdbd_conn.o lib/talloc_stack.o
+   lib/messages_ctdbd.o lib/packet.o lib/ctdbd_conn.o lib/talloc_stack.o \
+   lib/interfaces.o
 
 LIB_WITH_PROTO_OBJ = $(VERSION_OBJ) lib/charcnv.o lib/debug.o lib/fault.o \
  lib/interface.o lib/md4.o \
- lib/interfaces.o lib/pidfile.o \
+ lib/pidfile.o \
  lib/signal.o lib/system.o lib/sendfile.o lib/time.o \
  lib/ufc.o lib/genrand.o lib/username.o \
  lib/util_pw.o lib/access.o lib/smbrun.o \
diff --git a/source/client/smbmount.c b/source/client/smbmount.c
index 95adc9a..0c57c40 100644
--- a/source/client/smbmount.c
+++ b/source/client/smbmount.c
@@ -37,7 +37,7 @@ static pstring mpoint;
 static pstring service;
 static pstring options;
 
-static struct in_addr dest_ip;
+static struct sockaddr_storage dest_ip;
 static bool have_ip;
 static int smb_port = 0;
 static bool got_user;
@@ -122,7 +122,7 @@ static struct cli_state *do_connection(char *the_service)
struct cli_state *c;
struct nmb_name called, calling;
char *server_n;
-   struct in_addr ip;
+   struct sockaddr_storage ip;
pstring server;
char *share;
 
@@ -146,7 +146,7 @@ static struct cli_state *do_connection(char *the_service)
make_nmb_name(called , server, 0x20);
 
  again:
-zero_ip_v4(ip);
+zero_addr(ip, AF_INET);
if (have_ip) ip = dest_ip;
 
/* have to open a new connection */
@@ -798,8 +798,8 @@ static void parse_mount_smb(int argc, char **argv)
} else if(!strcmp(opts, debug)) {
DEBUGLEVEL = val;
} else if(!strcmp(opts, ip)) {
-   dest_ip = *interpret_addr2(opteq+1);
-   if (is_zero_ip_v4(dest_ip)) {
+   if (!interpret_string_addr(dest_ip, opteq+1,
+  0)) {
fprintf(stderr,Can't resolve address 
%s\n, opteq+1);
exit(1);
}
diff --git a/source/client/smbumount.c b/source/client/smbumount.c
index 4be54b6..1664e4b 100644
--- a/source/client/smbumount.c
+++ b/source/client/smbumount.c
@@ -80,7 +80,7 @@ umount_ok(const char *mount_point)
 static char *
 canonicalize (char *path)
 {
-   char *canonical = malloc (PATH_MAX + 1);
+   char *canonical = (char*)malloc (PATH_MAX + 1);
 
if (!canonical) {
fprintf(stderr, Error! Not enough memory!\n);
diff --git a/source/include/interfaces.h b/source/include/interfaces.h
index 66ea151..b9d9000 100644
--- a/source/include/interfaces.h
+++ b/source/include/interfaces.h
@@ -1,8 +1,31 @@
 /*
+   Unix SMB/CIFS implementation.
+   Machine customisation and include handling
+   Copyright (C) Jeremy Allison [EMAIL PROTECTED] 2007
+
+   This program is free software; you can redistribute it and/or modify
+   it under the terms of the GNU General 

Build status as of Sun Oct 28 00:00:02 2007

2007-10-27 Thread build
URL: http://build.samba.org/

--- /home/build/master/cache/broken_results.txt.old 2007-10-27 
00:01:00.0 +
+++ /home/build/master/cache/broken_results.txt 2007-10-28 00:01:09.0 
+
@@ -1,25 +1,25 @@
-Build status as of Sat Oct 27 00:00:02 2007
+Build status as of Sun Oct 28 00:00:02 2007
 
 Build counts:
 Tree Total  Broken Panic 
 SOC  0  0  0 
 build_farm   0  0  0 
-ccache   29 11 0 
+ccache   30 11 0 
 ctdb 0  0  0 
 distcc   1  0  0 
-ldb  29 6  0 
-libreplace   28 11 0 
-lorikeet-heimdal 23 14 0 
-pidl 16 3  0 
-ppp  9  7  0 
+ldb  30 6  0 
+libreplace   29 11 0 
+lorikeet-heimdal 24 14 0 
+pidl 17 3  0 
+ppp  10 8  0 
 python   0  0  0 
-rsync29 13 0 
+rsync30 13 0 
 samba-docs   0  0  0 
-samba-gtk2  2  0 
-samba4   26 19 5 
+samba-gtk3  3  0 
+samba4   27 18 5 
 samba_3_20  0  0 
 samba_3_2_test 29 21 0 
-smb-build28 28 0 
-talloc   29 2  0 
-tdb  29 3  0 
+smb-build29 29 0 
+talloc   30 2  0 
+tdb  30 3  0 
 


[SCM] Samba Shared Repository - branch v3-2-test updated - initial-v3-2-unstable-131-g2e92418

2007-10-27 Thread Jeremy Allison
The branch, v3-2-test has been updated
   via  2e92418a138bf2738b77b7e0fcb2fa37ad84fc0c (commit)
  from  9f82a5766968dc72db5a86ad0afc71e8cffd5be0 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=v3-2-test


- Log -
commit 2e92418a138bf2738b77b7e0fcb2fa37ad84fc0c
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Sat Oct 27 20:29:36 2007 -0700

Change all occurrences of zero_addr(ss,AF_INET) to
zero_addr(ss). All current uses were always of the
AF_INET form, so simplify the call. If in the future
we need to zero an addr to AF_INET6 this can be
done separately.
Jeremy.

---

Summary of changes:
 source/client/client.c   |2 +-
 source/client/smbmount.c |2 +-
 source/lib/util_sock.c   |8 
 source/libsmb/cliconnect.c   |2 +-
 source/libsmb/clidfs.c   |4 ++--
 source/libsmb/libsmbclient.c |6 +++---
 source/libsmb/namequery.c|8 
 source/libsmb/namequery_dc.c |4 ++--
 source/rpcclient/rpcclient.c |2 +-
 source/torture/locktest.c|4 ++--
 source/torture/masktest.c|4 ++--
 source/torture/torture.c |2 +-
 source/utils/net.c   |2 +-
 source/utils/smbcacls.c  |2 +-
 source/utils/smbcquotas.c|2 +-
 15 files changed, 27 insertions(+), 27 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/client/client.c b/source/client/client.c
index 7561ecb..f18505a 100644
--- a/source/client/client.c
+++ b/source/client/client.c
@@ -3853,7 +3853,7 @@ static int do_message_op(void)
snprintf(name_type_hex, sizeof(name_type_hex), #%X, name_type);
fstrcat(server_name, name_type_hex);
 
-zero_addr(ss,AF_INET);
+zero_addr(ss);
if (have_ip)
ss = dest_ss;
 
diff --git a/source/client/smbmount.c b/source/client/smbmount.c
index 0c57c40..9c671e3 100644
--- a/source/client/smbmount.c
+++ b/source/client/smbmount.c
@@ -146,7 +146,7 @@ static struct cli_state *do_connection(char *the_service)
make_nmb_name(called , server, 0x20);
 
  again:
-zero_addr(ip, AF_INET);
+zero_addr(ip);
if (have_ip) ip = dest_ip;
 
/* have to open a new connection */
diff --git a/source/lib/util_sock.c b/source/lib/util_sock.c
index 91da407..e66bd5f 100644
--- a/source/lib/util_sock.c
+++ b/source/lib/util_sock.c
@@ -238,7 +238,7 @@ bool interpret_string_addr(struct sockaddr_storage *pss,
}
 #endif
 
-   zero_addr(pss, AF_INET);
+   zero_addr(pss);
 
if (!interpret_string_addr_internal(res, str, flags|AI_ADDRCONFIG)) {
return false;
@@ -343,14 +343,14 @@ void zero_ip_v4(struct in_addr *ip)
 }
 
 /***
- Set an address to INADDR_ANY, or IN6ADDR_ANY.
+ Set an address to INADDR_ANY.
 **/
 
-void zero_addr(struct sockaddr_storage *pss, int family)
+void zero_addr(struct sockaddr_storage *pss)
 {
memset(pss, '\0', sizeof(*pss));
/* Ensure we're at least a valid sockaddr-storage. */
-   pss-ss_family = family;
+   pss-ss_family = AF_INET;
 }
 
 /***
diff --git a/source/libsmb/cliconnect.c b/source/libsmb/cliconnect.c
index 826315a..448bfd7 100644
--- a/source/libsmb/cliconnect.c
+++ b/source/libsmb/cliconnect.c
@@ -1519,7 +1519,7 @@ NTSTATUS cli_start_connection(struct cli_state 
**output_cli,
if (dest_ss) {
ss = *dest_ss;
} else {
-   zero_addr(ss, AF_INET);
+   zero_addr(ss);
}
 
 again:
diff --git a/source/libsmb/clidfs.c b/source/libsmb/clidfs.c
index e1ca924..6393f65 100644
--- a/source/libsmb/clidfs.c
+++ b/source/libsmb/clidfs.c
@@ -83,13 +83,13 @@ static struct cli_state *do_connect( const char *server, 
const char *share,
 
server_n = server;

-   zero_addr(ss, AF_INET);
+   zero_addr(ss);
 
make_nmb_name(calling, global_myname(), 0x0);
make_nmb_name(called , server, name_type);
 
  again:
-   zero_addr(ss, AF_INET);
+   zero_addr(ss);
if (have_ip)
ss = dest_ss;
 
diff --git a/source/libsmb/libsmbclient.c b/source/libsmb/libsmbclient.c
index d5bf182..0b45cad 100644
--- a/source/libsmb/libsmbclient.c
+++ b/source/libsmb/libsmbclient.c
@@ -654,7 +654,7 @@ smbc_server(SMBCCTX *context,
 const char *username_used;
NTSTATUS status;
 
-   zero_addr(ss, AF_INET);
+   zero_addr(ss);
ZERO_STRUCT(c);
 
if (server[0] == 0) {
@@ -742,7 +742,7 @@ smbc_server(SMBCCTX *context,
  again:
slprintf(ipenv,sizeof(ipenv)-1,HOST_%s, server_n);
 
-   zero_addr(ss, AF_INET);
+   zero_addr(ss);
 
/* have to open a new connection */
if