[Samba] Fwd: samba : Strong(er) authentication required

2008-10-01 Thread francis thaon
-- Forwarded message --
From: francis thaon [EMAIL PROTECTED]
Date: 2008/9/30
Subject: samba : Strong(er) authentication required
To: samba@lists.samba.org


 Good morning
I am running a windows server 2003 AD domain where the option server
signing cannot be changed

I have installed the latest version of samba and i always get this error msg
 Failed to join domain : Strong(er) authentication required

Kinit is working
net ads join -U xx -d4 does give above error message

Is there a solution for this ?
It seems lots of person are getting the problem and the only solution i have
seeen is to change DC configuration for server signing that could not be
allowed in the entreprise.


thanks
best regards
Francis
-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


Re: [Samba] smbldap and password expiration

2008-10-01 Thread manu Baylac

Onatawahtaw a écrit :

 Greetings,

 I just recently set up a new server with samba and openldap
 authentication using smbldap. The passwords seem to be expiring after
 about 30 days. How do I set them so that they don't expire?

 Thanks,

 Onatawahtaw



Hi,

using smbldap-tools ?
see -A and -B options in smbldap-user[add|mod]

Manu


--
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


Re: [Samba] Fwd: samba : Strong(er) authentication required

2008-10-01 Thread Martin Zielinski

hi!

You need Samba 3.2.x for this.

And add the smb.conf parameter:
client ldap sasl wrapping = sign

The only other way is to use NTLM authentication (security = domain), 
but this might cause even more troubles.


Bye,

Martin


francis thaon schrieb:

-- Forwarded message --
From: francis thaon [EMAIL PROTECTED]
Date: 2008/9/30
Subject: samba : Strong(er) authentication required
To: samba@lists.samba.org


 Good morning
I am running a windows server 2003 AD domain where the option server
signing cannot be changed

I have installed the latest version of samba and i always get this error msg
 Failed to join domain : Strong(er) authentication required

Kinit is working
net ads join -U xx -d4 does give above error message

Is there a solution for this ?
It seems lots of person are getting the problem and the only solution i have
seeen is to change DC configuration for server signing that could not be
allowed in the entreprise.


thanks
best regards
Francis



--
Martin Zielinski[EMAIL PROTECTED]   
Softwareentwicklung T +49 (0)521 94226 76   

SEH Computertechnik GmbHwww.seh.de

--
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


Re: [Samba] can root still connect clients to a domain?

2008-10-01 Thread Volker Lendecke
On Tue, Sep 30, 2008 at 01:37:00PM +0200, Helmut Hullen wrote:
 when I connect a Windows client to a Samba domain I am asked for the  
 user's login name and password on the Linux server who has the rights to  
 do this job.
 
 Under Samba 3.0.22 and older I could use root with a password defined  
 via smbpasswd (must not be the Linux password). Under Samba 3.0.28 and  
 newer this way seems no more possible; I have to use another user from  
 the admin users = xxx, yyy, zzz line.
 
 Is that a change which is somewhere documented?

Probably this has to do with group mapping changes. The
login as root should definitely still work, however root is
no longer automatically in the domain admins group (probably
required for accessing the client's c$ share). You have to
create an explicit group mapping entry for the sid 

S-1-5-21-DOMAIN-SID-512

and make root member of the corresponding unix group.

Volker


pgpUUHproXddD.pgp
Description: PGP signature
-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba

[Samba] RE: Browse QNX directories thro windows PC thro SAMBA

2008-10-01 Thread Jayakumar Gopalakrishnan-TLS,Chennai
DISCLAIMER:
---

The contents of this e-mail and any attachment(s) are confidential and intended 
for the named recipient(s) only.
It shall not attach any liability on the originator or HCL or its affiliates. 
Any views or opinions presented in 
this email are solely those of the author and may not necessarily reflect the 
opinions of HCL or its affiliates.
Any form of reproduction, dissemination, copying, disclosure, modification, 
distribution and / or publication of 
this message without the prior written consent of the author of this e-mail is 
strictly prohibited. If you have
received this email in error please delete it and notify the sender 
immediately. Before opening any mail and 
attachments please check them for viruses and defect.

- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba

Re: [Samba] Samba over bridged ethernet VPN

2008-10-01 Thread Wes Deviers
On Tuesday 30 September 2008 18:07:25 Daniel Bye wrote:
 Thanks for your reply, Wes.

...

 As for the routing between sites, if I understand correctly what you're
 asking, then it's simply a small LAN in the office attached via a commodity
 ADSL modem, with Samba and OpenVPN running on the same host. OpenVPN is
 running in bridged Ethernet mode, and assigns IP addresses to connecting
 clients. Therefore, effectively there is no routing between sites, as far
 as our CIFS/SMB clients are concerned. However, the physical routing is
 essentially as you'd expect - the office is on a standard domestic grade
 ADSL link, as are two of the remote users. The other remote users and I
 are connected over cable, and all are subject to our upstream providers'
 routing policies.

 I'm going to try fiddling with the MTU/fragment/mssfix settings in my
 OpenVPN configs, and see how we get on.

If you're using Linux routers, a good diag tool is iperf, which has a 
maximum MTU discovery mode.  The problem (apparently) comes in because 
fragmenting OpenVPN packets is Bad.  So if you set no-fragment then large 
packets just get dropped, which is also Bad.  It can also come from an 
interface or router in the middle that's broken somehow.  In my case, I had a 
PCI T1 interface that wasn't reporting the correct MTU for path discovery.  It 
was reporting 1500 as standard but it actually cut off somewhere around 1480; I 
think the driver implementation was broken.  Normally it wouldn't be an issue, 
but since OVPN can't be fragmented it became a problem.

Regardless, I hard-set the MTU on the ethernet devices (both ends) to 1400 and 
that fixed the transport issue.  In theory, you can do the same in the OVPN 
config (such that it pre-fragments, basically) but it didn't work as well for 
me.

Good luck!

Wes

-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


[Samba] Samba authentication using ADS

2008-10-01 Thread Prashanth Adiyodi
Greetings

 

I need help in setting up my linux box with ADS authentication on Samba.
I know that it can be done using winbind and Kerberos. I tried some of
the online methods but I am not able to get a result.

 

Request you to please help me with this.

 

These are the steps I followed to setup winbind

 

*   Using Authconfig command I put in the relavant details like Use
Winbind and Use Winbind Authentication and left Cache Information,
Use MD5 Passwords and Use Shadow Passwords selected
*   Then I put details about the domain with authentication.

 

*   I placed entries in /etc/nssswitch as 

passwd: files winbind

shadow: files winbind

group:  files winbind

 

 

This is the output I get 

 

[2008/10/01 18:27:56, 0] libads/kerberos.c:ads_kinit_password(146)

  kerberos_kinit_password [EMAIL PROTECTED] failed: Cannot find
KDC for requested realm

[2008/10/01 18:27:56, 0] utils/net_ads.c:ads_startup(186)

  ads_connect: Cannot find KDC for requested realm

[2008/10/01 18:27:56, 0] rpc_client/cli_pipe.c:cli_nt_session_open(1451)

  cli_nt_session_open: cli_nt_create failed on pipe \lsarpc to machine
ads.example.com.  Error was NT_STATUS_ACCESS_DENIED

could not initialise lsa pipe

could not obtain sid for domain

 

Shutting down Winbind services:[FAILED]

Starting Winbind services: [  OK  ]

 

Please help me as to what is going wrong. Appreciate if any members
could help me out in configuring using Kerberos. Here also I edited the
krb5.conf, krb.conf and krb.realm with the correct parameters but stll
am not able to get a solution.

 

Thanking you 

 

Prashanth Adiyodi
System Administrator


Roamware (I) Pvt. Ltd.
7th Floor, Sigma, Hiranandani Gardens
Technology Street, Powai, 
Mumbai-400 076
Tel: 40406000 Ext: 6124
GSM: 91-9833377712

www.roamware.com http://www.roamware.com 
The information contained herein may include confidential or privileged
information and is intended solely for the recipient(s) noted above. If
you receive this e-mail in error, please respond to the sender and
delete the e-mail. Any dissemination of this e-mail or the information
contained in this e-mail or attachments to unintended parties is
prohibited.

 

 

--
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


RE: [Samba] Samba authentication using ADS

2008-10-01 Thread Andrew Masterson
Try this:

http://wiki.samba.org/index.php/Samba__Active_Directory


 -Original Message-
 From: [EMAIL PROTECTED]

[mailto:[EMAIL PROTECTED]
]
 On Behalf Of Prashanth Adiyodi
 Sent: Wednesday, October 01, 2008 7:42 AM
 To: samba@lists.samba.org
 Subject: [Samba] Samba authentication using ADS
 
 Greetings
 
 
 
 I need help in setting up my linux box with ADS authentication on
Samba.
 I know that it can be done using winbind and Kerberos. I tried some of
 the online methods but I am not able to get a result.
 
 
 
 Request you to please help me with this.
 
 
 
 These are the steps I followed to setup winbind
 
 
 
 * Using Authconfig command I put in the relavant details like Use
 Winbind and Use Winbind Authentication and left Cache
Information,
 Use MD5 Passwords and Use Shadow Passwords selected
 * Then I put details about the domain with authentication.
 
 
 
 * I placed entries in /etc/nssswitch as
 
 passwd: files winbind
 
 shadow: files winbind
 
 group:  files winbind
 
 
 
 
 
 This is the output I get
 
 
 
 [2008/10/01 18:27:56, 0] libads/kerberos.c:ads_kinit_password(146)
 
   kerberos_kinit_password [EMAIL PROTECTED] failed: Cannot
find
 KDC for requested realm
 
 [2008/10/01 18:27:56, 0] utils/net_ads.c:ads_startup(186)
 
   ads_connect: Cannot find KDC for requested realm
 
 [2008/10/01 18:27:56, 0]
rpc_client/cli_pipe.c:cli_nt_session_open(1451)
 
   cli_nt_session_open: cli_nt_create failed on pipe \lsarpc to machine
 ads.example.com.  Error was NT_STATUS_ACCESS_DENIED
 
 could not initialise lsa pipe
 
 could not obtain sid for domain
 
 
 
 Shutting down Winbind services:[FAILED]
 
 Starting Winbind services: [  OK  ]
 
 
 
 Please help me as to what is going wrong. Appreciate if any members
 could help me out in configuring using Kerberos. Here also I edited
the
 krb5.conf, krb.conf and krb.realm with the correct parameters but stll
 am not able to get a solution.
 
 
 
 Thanking you
 
 
 
 Prashanth Adiyodi
 System Administrator
 
 
 Roamware (I) Pvt. Ltd.
 7th Floor, Sigma, Hiranandani Gardens
 Technology Street, Powai,
 Mumbai-400 076
 Tel: 40406000 Ext: 6124
 GSM: 91-9833377712
 
 www.roamware.com http://www.roamware.com
 The information contained herein may include confidential or
privileged
 information and is intended solely for the recipient(s) noted above.
If
 you receive this e-mail in error, please respond to the sender and
 delete the e-mail. Any dissemination of this e-mail or the information
 contained in this e-mail or attachments to unintended parties is
 prohibited.
 
 
 
 
 
 --
 To unsubscribe from this list go to the following URL and read the
 instructions:  https://lists.samba.org/mailman/listinfo/samba
--
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


Re: [Samba] nmbd fails to start - permission denied

2008-10-01 Thread Volker Lendecke
On Tue, Sep 30, 2008 at 10:36:49PM -0400, Onotsky, Steve x55328 wrote:
 Once all of that was in place, I started up Samba via the Red Hat
 startup script, but nmbd failed to load.  The nmbd.log file indicates
 that it can't open the smb.conf file, with the error Permission
 denied.  I've verified that unprivileged users can change into the
 directory where the config file resides, and I've given full (777)
 permissions to the file.  It also fails at boot time.

Is it possible that you have something like SELinux
installed? If yes, please deactivate that and try again.

Volker


pgpHGine6ONrD.pgp
Description: PGP signature
-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba

RE: [Samba] nmbd fails to start - permission denied

2008-10-01 Thread Onotsky, Steve x55328
Danke, Volker.  SELinux bit me in the ass again; I set it to permissive,
rebooted, and voila - nmbd starting.

I wonder why it fails during boot/via the script, but runs manually...

Cheers
Steve

-Original Message-
From: [EMAIL PROTECTED]
[mailto:[EMAIL PROTECTED] On
Behalf Of Volker Lendecke
Sent: October 1, 2008 11:10
To: Onotsky, Steve x55328
Cc: samba@lists.samba.org
Subject: Re: [Samba] nmbd fails to start - permission denied

On Tue, Sep 30, 2008 at 10:36:49PM -0400, Onotsky, Steve x55328 wrote:
 Once all of that was in place, I started up Samba via the Red Hat
 startup script, but nmbd failed to load.  The nmbd.log file indicates
 that it can't open the smb.conf file, with the error Permission
 denied.  I've verified that unprivileged users can change into the
 directory where the config file resides, and I've given full (777)
 permissions to the file.  It also fails at boot time.

Is it possible that you have something like SELinux
installed? If yes, please deactivate that and try again.

Volker
This message and any attachments are intended only for the use of the addressee 
and
may contain information that is privileged and confidential. If the reader of 
the 
message is not the intended recipient or an authorized representative of the
intended recipient, you are hereby notified that any dissemination of this
communication is strictly prohibited. If you have received this communication in
error, please notify us immediately by e-mail and delete the message and any
attachments from your system.

--
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


Re: [Samba] Revisiting Samba's interaction with LDAP's ppolicy overlay

2008-10-01 Thread Volker Lendecke
On Mon, Sep 29, 2008 at 10:14:01AM -0400, Adam Tauno Williams wrote:
 This is, AFAIK, the only solution currently.  We do the
 same thing.  It stinks.

As I said in the former mail thread: Patches are welcome. If
you really want it done quickly, some companies on
http://samba.org/samba/support also offer development
services.

Volker


pgpBYc6QWOEWF.pgp
Description: PGP signature
-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba

[Samba] samba-3.2.4 Solaris 9: configure gets error: Active Directory support requires ldap_initialize

2008-10-01 Thread Wayne Rasmussen
Unable to compile samba-3.2.4 on Solaris 9 for sparc.

 

#building openldap
CC=gcc
#echo $CC
#exit
CPPFLAGS=-I/usr/local/include -I/usr/local/ssl/include
-I/usr/local/BerkeleyDB.4.2/include -I/usr/local/include/sasl
LDFLAGS=-L/usr/local/lib -L/usr/local/ssl/lib
-L/usr/local/BerkeleyDB.4.2/lib
export CC CPPFLAGS LDFLAGS
#
CFLAGS='-D_AVL_H'
export CFLAGS
#


./configure --enable-bdb

make depend
make install
#end building openldap

#build samba
make clean
./configure --with-ldap --with-acl-support --with-ads --with-pam
--with-winbind --with-krb5=/usr/local

#make   
#make install
#end build samba

The samba script, configure, seems to be failing with the following:




checking for LDAP support... yes
checking ldap.h usability... yes
checking ldap.h presence... yes
checking for ldap.h... yes
checking lber.h usability... yes
checking lber.h presence... yes
checking for lber.h... yes
checking for ber_tag_t... yes
checking for ber_scanf in -llber... yes
checking for ber_sockbuf_add_io... yes
checking for LDAP_OPT_SOCKBUF... yes
checking for LBER_OPT_LOG_PRINT_FN... yes
checking for ldap_init in -lldap... yes
checking for ldap_set_rebind_proc... yes
checking whether ldap_set_rebind_proc takes 3 arguments... 3
checking for ldap_initialize... no
checking whether LDAP support is used... yes
checking for Active Directory and krb5 support... yes
checking for ldap_initialize... (cached) no
configure: error: Active Directory support requires ldap_initialize
 

 

--
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


[Samba] Samba PDC - first Domain Admin

2008-10-01 Thread James
Hi guys,

i'm trying to setup a samba PDC and was hoping to delegate Admin control
to the Domain Admins Group.
The backend is run off of ldap and there is no root user account in the
ldap directory

i set my user james with the rid ending in -500

i used:
 net rpc rights grant TESTING/Domain Admins SeMachineAccountPrivilege
SePrintOperatorPrivilege SeAddUsersPrivilege
SeRemoteShutdownPrivilege SeDiskOperatorPrivilege -U james

i get:
Failed to grant privileges for Domain Admins (NT_STATUS_ACCESS_DENIED)

does the first Admin user HAVE to have uid=0 and be in the ldap directory?
if so can i just shove him in and remove him later?
and does the net rpc rights grant command have to be run on every
domain controller or does it right something to ldap so it'll know?

Thanks,
James

-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


Re: [Samba] nmbd fails to start - permission denied

2008-10-01 Thread Ryan Novosielski
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

Hopefully this is not taken as rude: Disabling/setting permissive
SELinux (particularly for an app that ships with the distribution) is
the WRONG way to go, and is not required. It's a good way to figure if
it is SELinux that is the problem, but the real solution is to fix the
problems with the setup. Frequently it's nothing more than using
restorecon or chcon. None of this stuff is rocket science, and is there
to help keep admins from getting called in to clean up after intruders.

Onotsky, Steve x55328 wrote:
 Danke, Volker.  SELinux bit me in the ass again; I set it to permissive,
 rebooted, and voila - nmbd starting.
 
 I wonder why it fails during boot/via the script, but runs manually...
 
 Cheers
 Steve
 
 -Original Message-
 From: [EMAIL PROTECTED]
 [mailto:[EMAIL PROTECTED] On
 Behalf Of Volker Lendecke
 Sent: October 1, 2008 11:10
 To: Onotsky, Steve x55328
 Cc: samba@lists.samba.org
 Subject: Re: [Samba] nmbd fails to start - permission denied
 
 On Tue, Sep 30, 2008 at 10:36:49PM -0400, Onotsky, Steve x55328 wrote:
 Once all of that was in place, I started up Samba via the Red Hat
 startup script, but nmbd failed to load.  The nmbd.log file indicates
 that it can't open the smb.conf file, with the error Permission
 denied.  I've verified that unprivileged users can change into the
 directory where the config file resides, and I've given full (777)
 permissions to the file.  It also fails at boot time.
 
 Is it possible that you have something like SELinux
 installed? If yes, please deactivate that and try again.
 
 Volker
 This message and any attachments are intended only for the use of the 
 addressee and
 may contain information that is privileged and confidential. If the reader of 
 the 
 message is not the intended recipient or an authorized representative of the
 intended recipient, you are hereby notified that any dissemination of this
 communication is strictly prohibited. If you have received this communication 
 in
 error, please notify us immediately by e-mail and delete the message and any
 attachments from your system.
 


- --
  _  _ _  _ ___  _  _  _
 |Y#| |  | |\/| |  \ |\ |  | |Ryan Novosielski - Systems Programmer II
 |$| |__| |  | |__/ | \| _| |[EMAIL PROTECTED] - 973/972.0922 (2-0922)
 \__/ Univ. of Med. and Dent.|IST/AST - NJMS Medical Science Bldg - C630
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.6 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iD8DBQFI48ubmb+gadEcsb4RApB6AJ9v+w++NOyJmsTNKWsNtf8QjCsIoQCeN0Jj
x0jrKpAiiP/bl8MQWDKV6ek=
=kSP5
-END PGP SIGNATURE-
-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba

[Samba] file share issues

2008-10-01 Thread Tipton, Josh
I have /oracle/fin shared out but when a user tries to hit it they get
the following error:

 

[2008/10/01 15:55:24, 0] smbd/service.c:make_connection_snum(1003)

  '/oracle/fin' does not exist or permission denied when connecting to
[fin] Error was Permission denied

 

This is AD based authentication which is working fine.  If I use setfacl
and give the user permissions to the parent folder /oracle then they are
able to access the fin share in windows xp.  Below is a snippet of the
share information.

 

 

[fin]

path = /oracle/fin

browseable = yes

read only = no

create mask = 700

directory mask = 700

writeable = yes

 

getfacl /oracle/fin

# file: oracle/fin

# owner: oracle

# group: dba

user::rwx

user:tiptonj:rwx

user:lawrencej:rwx

group::rwx

mask::rwx

other::---

default:user::rwx

default:group::rwx

default:mask::rwx

default:other::---

 

 

Josh


**
This email contains confidential and proprietary information and is not to be 
used 
or disclosed to anyone other than the named recipient of this email, 
and is to be used only for the intended purpose of this communication.
**
--
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


Re: [Samba] samba-3.2.4 Solaris 9: configure gets error: Active Directory support requires ldap_initialize

2008-10-01 Thread John Center

Hi Wayne,

I had the same problem.  I fixed one line in configure:

if test x$debug = xyes ; then
CFLAGS=${CFLAGS} -g
else
CFLAGS=-O
fi

The second if statement was clearing my CFLAGS setting, replacing it 
with just -O.  I changed the line from CFLAGS=-O to 
CFLAGS=${CFLAGS} -O.  This kept my settings.  I don't know if this is 
the correct fix, but it worked for me.


HTH

-John


Wayne Rasmussen wrote:

Unable to compile samba-3.2.4 on Solaris 9 for sparc.



#building openldap
CC=gcc
#echo $CC
#exit
CPPFLAGS=-I/usr/local/include -I/usr/local/ssl/include
-I/usr/local/BerkeleyDB.4.2/include -I/usr/local/include/sasl
LDFLAGS=-L/usr/local/lib -L/usr/local/ssl/lib
-L/usr/local/BerkeleyDB.4.2/lib
export CC CPPFLAGS LDFLAGS
#
CFLAGS='-D_AVL_H'
export CFLAGS
#


./configure --enable-bdb

make depend
make install
#end building openldap

#build samba
make clean
./configure --with-ldap --with-acl-support --with-ads --with-pam
--with-winbind --with-krb5=/usr/local

#make
#make install
#end build samba

The samba script, configure, seems to be failing with the following:




checking for LDAP support... yes
checking ldap.h usability... yes
checking ldap.h presence... yes
checking for ldap.h... yes
checking lber.h usability... yes
checking lber.h presence... yes
checking for lber.h... yes
checking for ber_tag_t... yes
checking for ber_scanf in -llber... yes
checking for ber_sockbuf_add_io... yes
checking for LDAP_OPT_SOCKBUF... yes
checking for LBER_OPT_LOG_PRINT_FN... yes
checking for ldap_init in -lldap... yes
checking for ldap_set_rebind_proc... yes
checking whether ldap_set_rebind_proc takes 3 arguments... 3
checking for ldap_initialize... no
checking whether LDAP support is used... yes
checking for Active Directory and krb5 support... yes
checking for ldap_initialize... (cached) no
configure: error: Active Directory support requires ldap_initialize




--
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba

--
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


Re: [Samba] samba-3.2.4 Solaris 9: configure gets error: Active Directory support requires ldap_initialize

2008-10-01 Thread Volker Lendecke
On Wed, Oct 01, 2008 at 06:28:42PM -0400, John Center wrote:
 I had the same problem.  I fixed one line in configure:
 
 if test x$debug = xyes ; then
 CFLAGS=${CFLAGS} -g
 else
 CFLAGS=-O
 fi
 
 The second if statement was clearing my CFLAGS setting, replacing it 
 with just -O.  I changed the line from CFLAGS=-O to 
 CFLAGS=${CFLAGS} -O.  This kept my settings.  I don't know if this is 
 the correct fix, but it worked for me.

Pushed the attached patch which should also solve it,
configure is made from configure.in.

Volker
From 73f9e17e6457be8dfb6c1132cc6ca43ef65c677c Mon Sep 17 00:00:00 2001
From: Volker Lendecke [EMAIL PROTECTED]
Date: Thu, 2 Oct 2008 02:17:04 +0200
Subject: [PATCH] Fix bug 5798: CFLAGS info lost in configure

Michael, please check and merge to the other branches if it's right.

Thanks,

Volker
---
 source3/configure.in |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/source3/configure.in b/source3/configure.in
index f04ddbe..545a565 100644
--- a/source3/configure.in
+++ b/source3/configure.in
@@ -128,7 +128,7 @@ fi
 if test x$debug = xyes ; then
 	CFLAGS=${CFLAGS} -g
 else
-	CFLAGS=-O
+	CFLAGS=${CFLAGS} -O
 fi
 
 m4_include(../lib/socket_wrapper/config.m4)
-- 
1.5.5



pgpuCL7vatIeS.pgp
Description: PGP signature
-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba

[Samba] files disappearing from Samba server

2008-10-01 Thread Robert Wickberg
I'm the tech coordinator for a high school.  Last year, we had a file
server kids could save work to that was a generic Celeron 800 PC with an
IDE hard drive.  It ran Debian Sarge, with whatever version of Samba
ships with that.  It was down to a couple of gig of free disk space by
the end of the year, so this year I took an old Compaq Proliant server
(ML360 or something like that) with a three drive RAID 5 SCSI array and
installed Debian Etch on that with whatever version of Samba is shipped
with that.  School's been in session a month now, and I've had dozens of
kids come up to me and claim that they've written files to their mapped
shares (P: maps to their home directory when they log onto a Windows
machine), and the files have disappeared.  Teachers have confirmed that
they've looked over the kids shoulders while they're saving files, and
checked when they were done by accessing the drive via My Computer, and
the files are there, but the next day they're gone.  Sometimes they say
a kid will write a file to the server, and not get any error message,
but when they go to My Computer, the file's no where to be found.  

Needless to say, my credibility is taking quite a beating.  Has anyone
else experienced this, and have any ideas how to put a stop to it?

On a slightly different subject, does anyone know how to get a linux
file system to do what we used to be able to do with Netware, ie just
launch some utility and see every file you've ever deleted that the file
system hasn't yet overwritten, and recover them at will?  

-- 
To unsubscribe from this list go to the following URL and read the
instructions:  https://lists.samba.org/mailman/listinfo/samba


[SCM] Samba Shared Repository - branch v3-3-stable updated - release-3-3-0pre1-377-g562dce5

2008-10-01 Thread Karolin Seeger
The branch, v3-3-stable has been updated
   via  562dce5505f47c1f505c538b88107e3d9f9eae54 (commit)
   via  1020caf615e5c413363ead70f3930895889edc0b (commit)
   via  ba0cb8c74dd2fdeab014a8b90869725b85b6d557 (commit)
   via  394325b79c9f381e8b4bdaf1b8aec698e2e83cf7 (commit)
   via  f23599f59415f367c7a11b89e5ceac1d6032b87f (commit)
   via  4cfb63a07d7748d58dd869cdbb48f813534a8dea (commit)
   via  cf1786f82c819d3aeaf26e89dd4c8a627c291fa4 (commit)
   via  7fb8734762812808905de737495fbe5e550dbe00 (commit)
   via  bea0a1ee8d931bb95d29acbb74593849167a38fd (commit)
   via  4d2eda0d28380ec04234a75177423d31190a2ef6 (commit)
   via  3d21d7fc44e528bb27350bb373755241cb46e57c (commit)
   via  b381e23c5498f37208772ac1357b272ad931c92a (commit)
   via  64bf96ec26d6b479e6e41ec15e0f5036df4f0cd5 (commit)
   via  3c7f173dff889ba67cf200fe5a6f3b65c276b865 (commit)
   via  79978a4098fda2bcbcdaffc82c035548917781c8 (commit)
  from  a251d1bf1499f2e9f717028ccd3a46ecef8928e3 (commit)

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


- Log -
commit 562dce5505f47c1f505c538b88107e3d9f9eae54
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Tue Sep 30 16:20:43 2008 -0700

Convert to allocated strings. Use write_data(), not send as this doesn't 
correctly
deal with EINTR. Jim and Holger please check this still works.
Jeremy.
(cherry picked from commit bec55237c9c5687d947562524124b7aa76209b8f)

commit 1020caf615e5c413363ead70f3930895889edc0b
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Tue Sep 30 15:22:24 2008 -0700

Remove current_user_info - not needed.
Jeremy.
(cherry picked from commit 093fc8c6bf1420c0f1e171598c4894b4df24862c)

commit ba0cb8c74dd2fdeab014a8b90869725b85b6d557
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Tue Sep 30 15:11:26 2008 -0700

Restructure the module so it connects to the remote data sink
on connect, and closes the socket on client disconnect. This should
make it much more efficient. Store the remote fd in a private data
pointer off the handle. Finally we need to remove the fstrings and
convert to allocated buffer storage.
Jeremy.
(cherry picked from commit e3e89077a5559eda39ea27f0ce6f9f0ebeba8f1e)

commit 394325b79c9f381e8b4bdaf1b8aec698e2e83cf7
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Tue Sep 30 13:22:44 2008 -0700

Fix the make test problem Karolin reported. Now rename_open_files actually 
works correctly we must emit the change notify before we change the name, not 
before.
Jeremy
(cherry picked from commit 48c0dff0c5e36313619fb39d2bfb57d49ed2141e)

commit f23599f59415f367c7a11b89e5ceac1d6032b87f
Author: Andrew Tridgell [EMAIL PROTECTED]
Date:   Mon Sep 29 13:34:58 2008 -0700

re-added winbind:ignore domains patch

This option really is essential, as we discover again and again at
customer sites. Due to bugs in winbind some domains are toxic. When
you are installing at a site and a particular domain in a complex
setup causes winbind to segfault or hang then you need a way to
disable that domain and continue.

In an ideal world winbind could handle arbitrarily complex ADS
domains, but we are nowhere near that yet. If we ever get to that
stage then we won't need this option.
(cherry picked from commit f0f2fbd4800fb4a1df26f03f37b9972101657fdd)

commit 4cfb63a07d7748d58dd869cdbb48f813534a8dea
Author: Andrew Tridgell [EMAIL PROTECTED]
Date:   Mon Sep 29 12:04:57 2008 -0700

fixed an (unlikely) memory leak
(cherry picked from commit 8b908862ec7de000865cd8dfa691f5643cb3d099)

commit cf1786f82c819d3aeaf26e89dd4c8a627c291fa4
Author: Andrew Tridgell [EMAIL PROTECTED]
Date:   Mon Sep 29 11:27:31 2008 -0700

fixed a segfault on the ctdb destructor code
(cherry picked from commit f495968c88f07b74a943da9e034084b72f5a1c8b)

commit 7fb8734762812808905de737495fbe5e550dbe00
Author: Andrew Tridgell [EMAIL PROTECTED]
Date:   Mon Sep 29 11:24:48 2008 -0700

removed unused variables
(cherry picked from commit c48fd3cee0a91617123d79bccff7ec1f6008fb26)

commit bea0a1ee8d931bb95d29acbb74593849167a38fd
Author: Andrew Tridgell [EMAIL PROTECTED]
Date:   Mon Sep 29 11:18:21 2008 -0700

fixed segv on startup with trusted domains

With some setups, idmap_tdb2_allocate_id can be called before the
allocate backend is initialised, leading to a segv. This change
ensures that the db is opened in all paths that use it
(cherry picked from commit 8ec11590f471d2852e5ffd3d1d65da54bd2182ca)

commit 4d2eda0d28380ec04234a75177423d31190a2ef6
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Mon Sep 29 10:56:54 2008 -0700

Ensure we null out fsp after a close in all paths.
Jeremy.
(cherry picked from commit 35a074c24b9b320bb23e82d3f44e17c5fadbcf62)

commit 3d21d7fc44e528bb27350bb373755241cb46e57c
Author: Jelmer Vernooij [EMAIL 

[SCM] Samba Shared Repository - branch v3-3-stable updated - release-3-3-0pre1-378-gb311ded

2008-10-01 Thread Karolin Seeger
The branch, v3-3-stable has been updated
   via  b311ded4ae326ed7d140d4e8a8a1c81994311e9f (commit)
  from  562dce5505f47c1f505c538b88107e3d9f9eae54 (commit)

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


- Log -
commit b311ded4ae326ed7d140d4e8a8a1c81994311e9f
Author: Karolin Seeger [EMAIL PROTECTED]
Date:   Wed Oct 1 14:22:10 2008 +0200

WHATSNEW: Update changes since 3.3.0pre1 and release date.

Karolin
(cherry picked from commit eef33fe6383afb5870aef0eef132cd0ba88bac83)

---

Summary of changes:
 WHATSNEW.txt |   16 +++-
 1 files changed, 15 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/WHATSNEW.txt b/WHATSNEW.txt
index 8a11744..e6b2855 100644
--- a/WHATSNEW.txt
+++ b/WHATSNEW.txt
@@ -1,6 +1,6 @@
=
Release Notes for Samba 3.3.0pre2
-  September 30, 2008
+   October 2, 2008
=
 
 This is the second preview release of Samba 3.3.0.  This is *not*
@@ -186,6 +186,7 @@ o   Jeremy Allison [EMAIL PROTECTED]
 * BUG 5783: Fix FindFirst where search pattern == mangled filename.
 * BUG 5790: Fix returning of STATUS_OBJECT_NAME_NOT_FOUND on set file
   disposition.
+* BUG 5797: Fix moving of readonly files.
 * Fix crashes when looking up a non-existant uid.
 * Fix getting/setting of NT ACLs on a file.
 * Add st_birthtime and friends for accurate create times on *BSD
@@ -196,6 +197,8 @@ o   Jeremy Allison [EMAIL PROTECTED]
   mappings.
 * Write times code update.
 * Add experimental version of VFS module acl_xattr.
+* Fix rename_open_files.
+* Make SMB traffic analyzer VFS module more efficient.
 
 
 o   Gerald W. Carter [EMAIL PROTECTED]
@@ -205,6 +208,7 @@ o   Gerald W. Carter [EMAIL PROTECTED]
 * Allow an admin to define the uid attribute for a RFC2307
   user object in AD to be the username alias.
 * Add new idmap backend adex to support RFC2307 enabled AD forests.
+* Add new idmap backend hash.
 
 
 o   Steven Danneman [EMAIL PROTECTED]
@@ -217,6 +221,8 @@ o   Günther Deschner [EMAIL PROTECTED]
 * BUG 5784: Fix pam_winbind build issue on Solaris.
 * Fix invalid sid copy (hit when enumerating sibling domains) in Winbind.
 * Fix double installation of cifs.upcall.
+* Add change-user-password command to wbinfo.
+* Fix segfault in _srvsvc_NetShareAdd.
 
 
 o   James Ding [EMAIL PROTECTED]
@@ -271,6 +277,14 @@ o   Andrew Tridgell [EMAIL PROTECTED]
 * Fix an errno handling bug that could lead to an infinite loop.
 * Fix logic of tsmsm_sendfile().
 * Fix handling of arbitrary new PAC types.
+* Fix segfault on startup with trusted domains.
+* Fix segfault on the CTDB destructor code.
+* Fix memory leak.
+* Re-add winbind:ignore domains.
+
+
+o   Jelmer Vernooij [EMAIL PROTECTED]
+* Fix segfault (Debian bug #431696).
 
 
 o   Qiao Yang [EMAIL PROTECTED]


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch v3-3-test updated - release-3-2-0pre2-4202-geef33fe

2008-10-01 Thread Karolin Seeger
The branch, v3-3-test has been updated
   via  eef33fe6383afb5870aef0eef132cd0ba88bac83 (commit)
  from  bec55237c9c5687d947562524124b7aa76209b8f (commit)

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


- Log -
commit eef33fe6383afb5870aef0eef132cd0ba88bac83
Author: Karolin Seeger [EMAIL PROTECTED]
Date:   Wed Oct 1 14:22:10 2008 +0200

WHATSNEW: Update changes since 3.3.0pre1 and release date.

Karolin

---

Summary of changes:
 WHATSNEW.txt |   16 +++-
 1 files changed, 15 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/WHATSNEW.txt b/WHATSNEW.txt
index 8a11744..e6b2855 100644
--- a/WHATSNEW.txt
+++ b/WHATSNEW.txt
@@ -1,6 +1,6 @@
=
Release Notes for Samba 3.3.0pre2
-  September 30, 2008
+   October 2, 2008
=
 
 This is the second preview release of Samba 3.3.0.  This is *not*
@@ -186,6 +186,7 @@ o   Jeremy Allison [EMAIL PROTECTED]
 * BUG 5783: Fix FindFirst where search pattern == mangled filename.
 * BUG 5790: Fix returning of STATUS_OBJECT_NAME_NOT_FOUND on set file
   disposition.
+* BUG 5797: Fix moving of readonly files.
 * Fix crashes when looking up a non-existant uid.
 * Fix getting/setting of NT ACLs on a file.
 * Add st_birthtime and friends for accurate create times on *BSD
@@ -196,6 +197,8 @@ o   Jeremy Allison [EMAIL PROTECTED]
   mappings.
 * Write times code update.
 * Add experimental version of VFS module acl_xattr.
+* Fix rename_open_files.
+* Make SMB traffic analyzer VFS module more efficient.
 
 
 o   Gerald W. Carter [EMAIL PROTECTED]
@@ -205,6 +208,7 @@ o   Gerald W. Carter [EMAIL PROTECTED]
 * Allow an admin to define the uid attribute for a RFC2307
   user object in AD to be the username alias.
 * Add new idmap backend adex to support RFC2307 enabled AD forests.
+* Add new idmap backend hash.
 
 
 o   Steven Danneman [EMAIL PROTECTED]
@@ -217,6 +221,8 @@ o   Günther Deschner [EMAIL PROTECTED]
 * BUG 5784: Fix pam_winbind build issue on Solaris.
 * Fix invalid sid copy (hit when enumerating sibling domains) in Winbind.
 * Fix double installation of cifs.upcall.
+* Add change-user-password command to wbinfo.
+* Fix segfault in _srvsvc_NetShareAdd.
 
 
 o   James Ding [EMAIL PROTECTED]
@@ -271,6 +277,14 @@ o   Andrew Tridgell [EMAIL PROTECTED]
 * Fix an errno handling bug that could lead to an infinite loop.
 * Fix logic of tsmsm_sendfile().
 * Fix handling of arbitrary new PAC types.
+* Fix segfault on startup with trusted domains.
+* Fix segfault on the CTDB destructor code.
+* Fix memory leak.
+* Re-add winbind:ignore domains.
+
+
+o   Jelmer Vernooij [EMAIL PROTECTED]
+* Fix segfault (Debian bug #431696).
 
 
 o   Qiao Yang [EMAIL PROTECTED]


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch master updated - bdd80734c977336b09f64cd55d76815de49a5fee

2008-10-01 Thread Jelmer Vernooij
The branch, master has been updated
   via  bdd80734c977336b09f64cd55d76815de49a5fee (commit)
   via  90d1481cf6b100988f9a6157674d03ed677f280f (commit)
   via  d56a81f60a820e6a4a3f2617c4fdd4f88e7853cd (commit)
  from  c164ff2be5f9af7cc83e43d8c54b54186444fac0 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit bdd80734c977336b09f64cd55d76815de49a5fee
Author: Jelmer Vernooij [EMAIL PROTECTED]
Date:   Wed Oct 1 17:29:16 2008 +0200

Fix header.

commit 90d1481cf6b100988f9a6157674d03ed677f280f
Author: Jelmer Vernooij [EMAIL PROTECTED]
Date:   Wed Oct 1 17:28:12 2008 +0200

Fix formatting to be compatible with ReST.

commit d56a81f60a820e6a4a3f2617c4fdd4f88e7853cd
Author: Jelmer Vernooij [EMAIL PROTECTED]
Date:   Wed Oct 1 17:27:42 2008 +0200

Update to refer to 3 and 4, not just Samba 3.

---

Summary of changes:
 README.Coding   |   65 +++
 prog_guide4.txt |   31 ++---
 2 files changed, 49 insertions(+), 47 deletions(-)


Changeset truncated at 500 lines:

diff --git a/README.Coding b/README.Coding
index 8063ae8..c1b6641 100644
--- a/README.Coding
+++ b/README.Coding
@@ -1,6 +1,7 @@
-##
-## Coding conventions in the Samba 3 tree
-##
+Coding conventions in the Samba tree
+
+
+.. contents::
 
 ===
 Quick Start
@@ -14,15 +15,13 @@ style should never outweigh coding itself and so the the 
guidelines
 described here are hopefully easy enough to follow as they are very
 common and supported by tools and editors.
 
-The basic style, also mentioned in the SAMBA_4_0/prog_guide.txt is the
-Linux kernel coding style (See Documentation/CodingStyle in the kernel
-source tree).  The closely matches what most Samba developers use already
-anyways.
+The basic style, also mentioned in prog_guide4.txt, is the Linux kernel coding 
+style (See Documentation/CodingStyle in the kernel source tree). This closely 
+matches what most Samba developers use already anyways.
 
 But to save you the trouble of reading the Linux kernel style guide, here
 are the highlights.
 
-
 * Maximum Line Width is 80 Characters
   The reason is not for people with low-res screens but rather sticking
   to 80 columns prevents you from easily nesting more than one level of
@@ -59,14 +58,14 @@ Vi
 --
 (Thanks to SATOH Fumiyasu [EMAIL PROTECTED] for these hints):
 
-For the basic vi editor including with all variants of *nix, add the 
+For the basic vi editor including with all variants of \*nix, add the 
 following to $HOME/.exrc:
 
   set tabstop=8
   set shiftwidth=8
 
 For Vim, the following settings in $HOME/.vimrc will also deal with 
-displaying trailing whitespace:
+displaying trailing whitespace::
 
   if has(syntax)  (t_Co  2 || has(gui_running))
syntax on
@@ -91,7 +90,7 @@ FAQ  Statement Reference
 Comments
 
 
-Comments should always use the standard C syntax.  I.e. /* ... */.  C++ 
+Comments should always use the standard C syntax.  C++ 
 style comments are not currently allowed.
 
 
@@ -145,7 +144,7 @@ The exception to the ending rule is when the closing brace 
is followed by
 another language keyword such as else or the closing while in a do..while 
 loop.
 
-Good examples:
+Good examples::
 
if (x == 1) {
printf(good\n);
@@ -162,7 +161,7 @@ Good examples:
printf(also good\n);
} while (1);
 
-Bad examples:
+Bad examples::
 
while (1)
{
@@ -177,29 +176,29 @@ evil, they can greatly enhance readability and reduce 
memory leaks when used
 as the single exit point from a function.  But in no Samba world what so ever 
 is a goto outside of a function or block of code a good idea.
 
-Good Examples:
-
-int function foo(int y)
-{
-   int *z = NULL;
-   int ret = 0;
+Good Examples::
 
-   if ( y  10 ) {
-   z = malloc(sizeof(int)*y);
-   if (!z) {
-   ret = 1;
-   goto done;
+   int function foo(int y)
+   {
+   int *z = NULL;
+   int ret = 0;
+
+   if ( y  10 ) {
+   z = malloc(sizeof(int)*y);
+   if (!z) {
+   ret = 1;
+   goto done;
+   }
}
-   }
 
-   print(Allocated %d elements.\n, y);
+   print(Allocated %d elements.\n, y);
 
- done: 
-   if (z)
-   free(z);
+done: 
+   if (z)
+   free(z);
 
-   return ret;
-}
+   return ret;
+   }
 
 
 Checking Pointer Values
@@ -207,13 +206,13 @@ Checking Pointer Values
 
 When invoking functions that return pointer values, either of the following 
 are acceptable.  Use you best judgement 

[SCM] Samba Shared Repository - branch master updated - ca44340891d60de7e48bb263c7debd45cc3f3ed0

2008-10-01 Thread Jelmer Vernooij
The branch, master has been updated
   via  ca44340891d60de7e48bb263c7debd45cc3f3ed0 (commit)
  from  bdd80734c977336b09f64cd55d76815de49a5fee (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit ca44340891d60de7e48bb263c7debd45cc3f3ed0
Author: Jelmer Vernooij [EMAIL PROTECTED]
Date:   Wed Oct 1 18:06:55 2008 +0200

Fix some syntax errors for use with ReST.

---

Summary of changes:
 lib/talloc/talloc_guide.txt |   43 ++-
 1 files changed, 26 insertions(+), 17 deletions(-)


Changeset truncated at 500 lines:

diff --git a/lib/talloc/talloc_guide.txt b/lib/talloc/talloc_guide.txt
index 18663b3..3201fe6 100644
--- a/lib/talloc/talloc_guide.txt
+++ b/lib/talloc/talloc_guide.txt
@@ -1,5 +1,7 @@
 Using talloc in Samba4
---
+==
+
+.. contents::
 
 Andrew Tridgell
 September 2004
@@ -18,7 +20,7 @@ get used to it.
 Perhaps the biggest change from Samba3 is that there is no distinction
 between a talloc context and a talloc pointer. Any pointer
 returned from talloc() is itself a valid talloc context. This means
-you can do this:
+you can do this::
 
   struct foo *X = talloc(mem_ctx, struct foo);
   X-name = talloc_strdup(X, foo);
@@ -271,7 +273,7 @@ equivalent to:
 =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
 void *talloc_named_const(const void *context, size_t size, const char *name);
 
-This is equivalent to:
+This is equivalent to::
 
ptr = talloc_size(context, size);
talloc_set_name_const(ptr, name);
@@ -288,7 +290,7 @@ talloc_set_name() for details.
 void *talloc_init(const char *fmt, ...);
 
 This function creates a zero length named talloc context as a top
-level context. It is equivalent to:
+level context. It is equivalent to::
 
   talloc_named(NULL, 0, fmt, ...);
 
@@ -309,7 +311,7 @@ The talloc_realloc() macro changes the size of a talloc
 pointer. The count argument is the number of elements of type type
 that you want the resulting pointer to hold. 
 
-talloc_realloc() has the following equivalences:
+talloc_realloc() has the following equivalences::
 
   talloc_realloc(context, NULL, type, 1) == talloc(context, type);
   talloc_realloc(context, NULL, type, N) == talloc_array(context, type, N);
@@ -490,7 +492,7 @@ This disables tracking of the NULL memory context.
 =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
 (type *)talloc_zero(const void *ctx, type);
 
-The talloc_zero() macro is equivalent to:
+The talloc_zero() macro is equivalent to::
 
   ptr = talloc(ctx, type);
   if (ptr) memset(ptr, 0, sizeof(type));
@@ -505,7 +507,7 @@ The talloc_zero_size() function is useful when you don't 
have a known type
 =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
 void *talloc_memdup(const void *ctx, const void *p, size_t size);
 
-The talloc_memdup() function is equivalent to:
+The talloc_memdup() function is equivalent to::
 
   ptr = talloc_size(ctx, size);
   if (ptr) memcpy(ptr, p, size);
@@ -514,13 +516,14 @@ The talloc_memdup() function is equivalent to:
 =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
 char *talloc_strdup(const void *ctx, const char *p);
 
-The talloc_strdup() function is equivalent to:
+The talloc_strdup() function is equivalent to::
 
   ptr = talloc_size(ctx, strlen(p)+1);
   if (ptr) memcpy(ptr, p, strlen(p)+1);
 
 This functions sets the name of the new pointer to the passed
-string. This is equivalent to:
+string. This is equivalent to::
+
talloc_set_name_const(ptr, ptr)
 
 =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
@@ -540,7 +543,8 @@ The talloc_append_string() function appends the given 
formatted
 string to the given string.
 
 This function sets the name of the new pointer to the new
-string. This is equivalent to:
+string. This is equivalent to::
+
talloc_set_name_const(ptr, ptr)
 
 =-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-=-
@@ -550,7 +554,8 @@ The talloc_vasprintf() function is the talloc equivalent of 
the C
 library function vasprintf()
 
 This functions sets the name of the new pointer to the new
-string. This is equivalent to:
+string. This is equivalent to::
+
talloc_set_name_const(ptr, ptr)
 
 
@@ -561,7 +566,8 @@ The talloc_asprintf() function is the talloc equivalent of 
the C
 library function asprintf()
 
 This functions sets the name of the new pointer to the new
-string. This is equivalent to:
+string. This is equivalent to::
+
talloc_set_name_const(ptr, ptr)
 
 
@@ -574,7 +580,8 @@ Use this varient when the string in the current talloc 
buffer may
 have been truncated in length.
 
 This functions sets the name of the new pointer to the new
-string. This is equivalent to:
+string. This is equivalent to::
+

[SCM] Samba Shared Repository - branch v3-3-test updated - release-3-2-0pre2-4203-g9a90cea

2008-10-01 Thread Jeremy Allison
The branch, v3-3-test has been updated
   via  9a90ceac52f8623dde2bf50e630b10aebbc0e28d (commit)
  from  eef33fe6383afb5870aef0eef132cd0ba88bac83 (commit)

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


- Log -
commit 9a90ceac52f8623dde2bf50e630b10aebbc0e28d
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 12:28:45 2008 -0700

Turn the socket connections into a refcounted list - in the common case 
there'll now only be one socket per smbd.
Changed the format of the wire data to (a) include a version number (V1) as 
the first element. (b) removed the
;) at the end an replaced it with a \n. Receiver can change back if 
needed, and now receiver can just log
as-is to a text file (making testing easier). Added my (C). Sorry Holger, 
but I've changed quite a bit now.
Jeremy.

---

Summary of changes:
 source/modules/vfs_smb_traffic_analyzer.c |  139 -
 1 files changed, 95 insertions(+), 44 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/modules/vfs_smb_traffic_analyzer.c 
b/source/modules/vfs_smb_traffic_analyzer.c
index 3925424..ff61768 100644
--- a/source/modules/vfs_smb_traffic_analyzer.c
+++ b/source/modules/vfs_smb_traffic_analyzer.c
@@ -3,6 +3,7 @@
  * on the net.
  *
  * Copyright (C) Holger Hetterich, 2008
+ * Copyright (C) Jeremy Allison, 2008
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -21,12 +22,14 @@
 #include includes.h
 
 /* abstraction for the send_over_network function */
-#define UNIX_DOMAIN_SOCKET 1
-#define INTERNET_SOCKET 0
+
+enum sock_type {INTERNET_SOCKET = 0, UNIX_DOMAIN_SOCKET};
+
+#define LOCAL_PATHNAME /var/tmp/stadsocket
 
 static int vfs_smb_traffic_analyzer_debug_level = DBGC_VFS;
 
-static int smb_traffic_analyzer_connMode(vfs_handle_struct *handle)
+static enum sock_type smb_traffic_analyzer_connMode(vfs_handle_struct *handle)
 {
connection_struct *conn = handle-conn;
 const char *Mode;
@@ -41,28 +44,22 @@ static int smb_traffic_analyzer_connMode(vfs_handle_struct 
*handle)
 
 /* Connect to an internet socket */
 
-static int smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle)
+static int smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle,
+   const char *name, uint16_t port)
 {
/* Create a streaming Socket */
-   const char *Hostname;
int sockfd = -1;
-uint16_t port;
struct addrinfo hints;
struct addrinfo *ailist = NULL;
struct addrinfo *res = NULL;
-   connection_struct *conn = handle-conn;
int ret;
 
-   /* get port number, target system from the config parameters */
-   Hostname=lp_parm_const_string(SNUM(conn), smb_traffic_analyzer,
-   host, localhost);
-
ZERO_STRUCT(hints);
/* By default make sure it supports TCP. */
hints.ai_socktype = SOCK_STREAM;
hints.ai_flags = AI_ADDRCONFIG;
 
-   ret = getaddrinfo(Hostname,
+   ret = getaddrinfo(name,
NULL,
hints,
ailist);
@@ -70,16 +67,13 @@ static int 
smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle)
 if (ret) {
DEBUG(3,(smb_traffic_analyzer_connect_inet_socket: 
getaddrinfo failed for name %s [%s]\n,
-Hostname,
+name,
 gai_strerror(ret) ));
return -1;
 }
 
-   port = atoi( lp_parm_const_string(SNUM(conn),
-   smb_traffic_analyzer, port, 9430));
-
DEBUG(3,(smb_traffic_analyzer: Internet socket mode. Hostname: %s,
-   Port: %i\n, Hostname, port));
+   Port: %i\n, name, port));
 
for (res = ailist; res; res = res-ai_next) {
struct sockaddr_storage ss;
@@ -113,15 +107,16 @@ static int 
smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle)
 
 /* Connect to a unix domain socket */
 
-static int smb_traffic_analyzer_connect_unix_socket(vfs_handle_struct *handle)
+static int smb_traffic_analyzer_connect_unix_socket(vfs_handle_struct *handle,
+   const char *name)
 {
/* Create the socket to stad */
int len, sock;
struct sockaddr_un remote;
 
DEBUG(7, (smb_traffic_analyzer_connect_unix_socket: 
-   Unix domain socket mode. Using 
-   /var/tmp/stadsocket\n));
+   Unix domain socket mode. Using %s\n,
+   name ));
 
if ((sock = socket(AF_UNIX, SOCK_STREAM, 0)) == -1) {
DEBUG(1, 

[SCM] Samba Shared Repository - branch v3-2-test updated - release-3-2-0pre2-3061-g7ea95c1

2008-10-01 Thread Jeremy Allison
The branch, v3-2-test has been updated
   via  7ea95c19305a2fb6efc95acc4e96f083e31f6c6f (commit)
  from  dc1ff1a9c07edf7e3e649b256667704ad59aafbd (commit)

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


- Log -
commit 7ea95c19305a2fb6efc95acc4e96f083e31f6c6f
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 12:30:12 2008 -0700

Turn the socket connections into a refcounted list - in the common case 
there'll now only be one socket per smbd.
Changed the format of the wire data to (a) include a version number (V1) as 
the first element. (b) removed the
;) at the end an replaced it with a \n. Receiver can change back if 
needed, and now receiver can just log
as-is to a text file (making testing easier). Added my (C). Sorry Holger, 
but I've changed quite a bit now.
Jeremy.

---

Summary of changes:
 source/modules/vfs_smb_traffic_analyzer.c |  138 -
 1 files changed, 94 insertions(+), 44 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/modules/vfs_smb_traffic_analyzer.c 
b/source/modules/vfs_smb_traffic_analyzer.c
index 6be58cd..714df02 100644
--- a/source/modules/vfs_smb_traffic_analyzer.c
+++ b/source/modules/vfs_smb_traffic_analyzer.c
@@ -3,6 +3,7 @@
  * on the net.
  *
  * Copyright (C) Holger Hetterich, 2008
+ * Copyright (C) Jeremy Allison, 2008
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -21,14 +22,15 @@
 #include includes.h
 
 /* abstraction for the send_over_network function */
-#define UNIX_DOMAIN_SOCKET 1
-#define INTERNET_SOCKET 0
+enum sock_type {INTERNET_SOCKET = 0, UNIX_DOMAIN_SOCKET};
+
+#define LOCAL_PATHNAME /var/tmp/stadsocket
 
 extern userdom_struct current_user_info;
 
 static int vfs_smb_traffic_analyzer_debug_level = DBGC_VFS;
 
-static int smb_traffic_analyzer_connMode(vfs_handle_struct *handle)
+static enum sock_type smb_traffic_analyzer_connMode(vfs_handle_struct *handle)
 {
connection_struct *conn = handle-conn;
 const char *Mode;
@@ -43,28 +45,22 @@ static int smb_traffic_analyzer_connMode(vfs_handle_struct 
*handle)
 
 /* Connect to an internet socket */
 
-static int smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle)
+static int smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle,
+   const char *name, uint16_t port)
 {
/* Create a streaming Socket */
-   const char *Hostname;
int sockfd = -1;
-uint16_t port;
struct addrinfo hints;
struct addrinfo *ailist = NULL;
struct addrinfo *res = NULL;
-   connection_struct *conn = handle-conn;
int ret;
 
-   /* get port number, target system from the config parameters */
-   Hostname=lp_parm_const_string(SNUM(conn), smb_traffic_analyzer,
-   host, localhost);
-
ZERO_STRUCT(hints);
/* By default make sure it supports TCP. */
hints.ai_socktype = SOCK_STREAM;
hints.ai_flags = AI_ADDRCONFIG;
 
-   ret = getaddrinfo(Hostname,
+   ret = getaddrinfo(name,
NULL,
hints,
ailist);
@@ -72,16 +68,13 @@ static int 
smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle)
 if (ret) {
DEBUG(3,(smb_traffic_analyzer_connect_inet_socket: 
getaddrinfo failed for name %s [%s]\n,
-Hostname,
+name,
 gai_strerror(ret) ));
return -1;
 }
 
-   port = atoi( lp_parm_const_string(SNUM(conn),
-   smb_traffic_analyzer, port, 9430));
-
DEBUG(3,(smb_traffic_analyzer: Internet socket mode. Hostname: %s,
-   Port: %i\n, Hostname, port));
+   Port: %i\n, name, port));
 
for (res = ailist; res; res = res-ai_next) {
struct sockaddr_storage ss;
@@ -115,15 +108,16 @@ static int 
smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle)
 
 /* Connect to a unix domain socket */
 
-static int smb_traffic_analyzer_connect_unix_socket(vfs_handle_struct *handle)
+static int smb_traffic_analyzer_connect_unix_socket(vfs_handle_struct *handle,
+   const char *name)
 {
/* Create the socket to stad */
int len, sock;
struct sockaddr_un remote;
 
DEBUG(7, (smb_traffic_analyzer_connect_unix_socket: 
-   Unix domain socket mode. Using 
-   /var/tmp/stadsocket\n));
+   Unix domain socket mode. Using %s\n,
+   name ));
 
if ((sock = socket(AF_UNIX, 

[SCM] Samba Shared Repository - branch master updated - 370cbe0060cb2670c7f65100954dac6c63030ca0

2008-10-01 Thread Jeremy Allison
The branch, master has been updated
   via  370cbe0060cb2670c7f65100954dac6c63030ca0 (commit)
  from  ca44340891d60de7e48bb263c7debd45cc3f3ed0 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit 370cbe0060cb2670c7f65100954dac6c63030ca0
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 12:50:29 2008 -0700

Turn the socket connections into a refcounted list - in the common case 
there'll now only be one socket per smbd.
Changed the format of the wire data to (a) include a version number (V1) as 
the first element. (b) removed the
;) at the end an replaced it with a \n. Receiver can change back if 
needed, and now receiver can just log
as-is to a text file (making testing easier). Added my (C). Sorry Holger, 
but I've changed quite a bit now.
Jeremy.

---

Summary of changes:
 source3/modules/vfs_smb_traffic_analyzer.c |  139 +++-
 1 files changed, 95 insertions(+), 44 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/modules/vfs_smb_traffic_analyzer.c 
b/source3/modules/vfs_smb_traffic_analyzer.c
index 3925424..ff61768 100644
--- a/source3/modules/vfs_smb_traffic_analyzer.c
+++ b/source3/modules/vfs_smb_traffic_analyzer.c
@@ -3,6 +3,7 @@
  * on the net.
  *
  * Copyright (C) Holger Hetterich, 2008
+ * Copyright (C) Jeremy Allison, 2008
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -21,12 +22,14 @@
 #include includes.h
 
 /* abstraction for the send_over_network function */
-#define UNIX_DOMAIN_SOCKET 1
-#define INTERNET_SOCKET 0
+
+enum sock_type {INTERNET_SOCKET = 0, UNIX_DOMAIN_SOCKET};
+
+#define LOCAL_PATHNAME /var/tmp/stadsocket
 
 static int vfs_smb_traffic_analyzer_debug_level = DBGC_VFS;
 
-static int smb_traffic_analyzer_connMode(vfs_handle_struct *handle)
+static enum sock_type smb_traffic_analyzer_connMode(vfs_handle_struct *handle)
 {
connection_struct *conn = handle-conn;
 const char *Mode;
@@ -41,28 +44,22 @@ static int smb_traffic_analyzer_connMode(vfs_handle_struct 
*handle)
 
 /* Connect to an internet socket */
 
-static int smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle)
+static int smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle,
+   const char *name, uint16_t port)
 {
/* Create a streaming Socket */
-   const char *Hostname;
int sockfd = -1;
-uint16_t port;
struct addrinfo hints;
struct addrinfo *ailist = NULL;
struct addrinfo *res = NULL;
-   connection_struct *conn = handle-conn;
int ret;
 
-   /* get port number, target system from the config parameters */
-   Hostname=lp_parm_const_string(SNUM(conn), smb_traffic_analyzer,
-   host, localhost);
-
ZERO_STRUCT(hints);
/* By default make sure it supports TCP. */
hints.ai_socktype = SOCK_STREAM;
hints.ai_flags = AI_ADDRCONFIG;
 
-   ret = getaddrinfo(Hostname,
+   ret = getaddrinfo(name,
NULL,
hints,
ailist);
@@ -70,16 +67,13 @@ static int 
smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle)
 if (ret) {
DEBUG(3,(smb_traffic_analyzer_connect_inet_socket: 
getaddrinfo failed for name %s [%s]\n,
-Hostname,
+name,
 gai_strerror(ret) ));
return -1;
 }
 
-   port = atoi( lp_parm_const_string(SNUM(conn),
-   smb_traffic_analyzer, port, 9430));
-
DEBUG(3,(smb_traffic_analyzer: Internet socket mode. Hostname: %s,
-   Port: %i\n, Hostname, port));
+   Port: %i\n, name, port));
 
for (res = ailist; res; res = res-ai_next) {
struct sockaddr_storage ss;
@@ -113,15 +107,16 @@ static int 
smb_traffic_analyzer_connect_inet_socket(vfs_handle_struct *handle)
 
 /* Connect to a unix domain socket */
 
-static int smb_traffic_analyzer_connect_unix_socket(vfs_handle_struct *handle)
+static int smb_traffic_analyzer_connect_unix_socket(vfs_handle_struct *handle,
+   const char *name)
 {
/* Create the socket to stad */
int len, sock;
struct sockaddr_un remote;
 
DEBUG(7, (smb_traffic_analyzer_connect_unix_socket: 
-   Unix domain socket mode. Using 
-   /var/tmp/stadsocket\n));
+   Unix domain socket mode. Using %s\n,
+   name ));
 
if ((sock = socket(AF_UNIX, SOCK_STREAM, 0)) == -1) {
DEBUG(1, 

[SCM] Samba Shared Repository - branch master updated - e4f5bfb34b7a515e2cf107eb94489260594b8733

2008-10-01 Thread Jeremy Allison
The branch, master has been updated
   via  e4f5bfb34b7a515e2cf107eb94489260594b8733 (commit)
  from  370cbe0060cb2670c7f65100954dac6c63030ca0 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit e4f5bfb34b7a515e2cf107eb94489260594b8733
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 13:15:54 2008 -0700

Fix use of DLIST_REMOVE as spotted by Constantine Vetoshev [EMAIL 
PROTECTED].
This API is unusual in that if used to remove a non-list head it nulls out
the next and prev pointers. This is what you want for debugging (don't want
an entry removed from the list to be still virtually linked into it) but
means there is no consistent idiom for use as the next and prev pointers
get trashed on removal from the list, meaning you must save them yourself.
You can use it one way when deleting everything via the head pointer, as
this preserves the next pointer, but you *must* use it another way when not
deleting everything via the head pointer. Fix all known uses of this (the 
main
one is in conn_free_internal() and would not free all the private data 
entries
for vfs modules. The other changes in web/statuspage.c and winbindd_util.c
are not strictly neccessary, as the head pointer is being used, but I've 
done
them for consistency. Long term we must revisit this as this API is too hard
to use correctly.
Jeremy.

---

Summary of changes:
 source3/smbd/conn.c  |2 +-
 source3/web/statuspage.c |5 +++--
 source3/winbindd/winbindd_util.c |3 +--
 3 files changed, 5 insertions(+), 5 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/smbd/conn.c b/source3/smbd/conn.c
index b9433bb..7f34d2b 100644
--- a/source3/smbd/conn.c
+++ b/source3/smbd/conn.c
@@ -252,8 +252,8 @@ void conn_free_internal(connection_struct *conn)
/* Free vfs_connection_struct */
handle = conn-vfs_handles;
while(handle) {
-   DLIST_REMOVE(conn-vfs_handles, handle);
thandle = handle-next;
+   DLIST_REMOVE(conn-vfs_handles, handle);
if (handle-free_data)
handle-free_data(handle-data);
handle = thandle;
diff --git a/source3/web/statuspage.c b/source3/web/statuspage.c
index ce24c7c..e684a07 100644
--- a/source3/web/statuspage.c
+++ b/source3/web/statuspage.c
@@ -43,9 +43,10 @@ static void initPid2Machine (void)
 {
/* show machine name rather PID on table Open Files? */
if (PID_or_Machine) {
-   PIDMAP *p;
+   PIDMAP *p, *next;
 
-   for (p = pidmap; p != NULL; ) {
+   for (p = pidmap; p != NULL; p = next) {
+   next = p-next;
DLIST_REMOVE(pidmap, p);
SAFE_FREE(p-machine);
SAFE_FREE(p);
diff --git a/source3/winbindd/winbindd_util.c b/source3/winbindd/winbindd_util.c
index b8cb27c..fdfc8ed 100644
--- a/source3/winbindd/winbindd_util.c
+++ b/source3/winbindd/winbindd_util.c
@@ -1075,13 +1075,12 @@ void free_getent_state(struct getent_state *state)
temp = state;
 
while(temp != NULL) {
-   struct getent_state *next;
+   struct getent_state *next = temp-next;
 
/* Free sam entries then list entry */
 
SAFE_FREE(state-sam_entries);
DLIST_REMOVE(state, state);
-   next = temp-next;
 
SAFE_FREE(temp);
temp = next;


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch v3-2-test updated - release-3-2-0pre2-3062-g347fc4e

2008-10-01 Thread Jeremy Allison
The branch, v3-2-test has been updated
   via  347fc4ed725083f2c520afc39f7c81db0dd60324 (commit)
  from  7ea95c19305a2fb6efc95acc4e96f083e31f6c6f (commit)

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


- Log -
commit 347fc4ed725083f2c520afc39f7c81db0dd60324
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 13:22:02 2008 -0700

Fix use of DLIST_REMOVE as spotted by Constantine Vetoshev [EMAIL 
PROTECTED].
This API is unusual in that if used to remove a non-list head it nulls out
the next and prev pointers. This is what you want for debugging (don't want
an entry removed from the list to be still virtually linked into it) but
means there is no consistent idiom for use as the next and prev pointers
get trashed on removal from the list, meaning you must save them yourself.
You can use it one way when deleting everything via the head pointer, as
this preserves the next pointer, but you *must* use it another way when not
deleting everything via the head pointer. Fix all known uses of this (the 
main
one is in conn_free_internal() and would not free all the private data 
entries
for vfs modules. The other changes in web/statuspage.c and winbindd_util.c
are not strictly neccessary, as the head pointer is being used, but I've 
done
them for consistency. Long term we must revisit this as this API is too hard
to use correctly.
Jeremy.

---

Summary of changes:
 source/smbd/conn.c  |2 +-
 source/web/statuspage.c |5 +++--
 source/winbindd/winbindd_util.c |3 +--
 3 files changed, 5 insertions(+), 5 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/smbd/conn.c b/source/smbd/conn.c
index c5a2df4..cfcec51 100644
--- a/source/smbd/conn.c
+++ b/source/smbd/conn.c
@@ -267,8 +267,8 @@ void conn_free_internal(connection_struct *conn)
/* Free vfs_connection_struct */
handle = conn-vfs_handles;
while(handle) {
-   DLIST_REMOVE(conn-vfs_handles, handle);
thandle = handle-next;
+   DLIST_REMOVE(conn-vfs_handles, handle);
if (handle-free_data)
handle-free_data(handle-data);
handle = thandle;
diff --git a/source/web/statuspage.c b/source/web/statuspage.c
index eda53b6..5d9a239 100644
--- a/source/web/statuspage.c
+++ b/source/web/statuspage.c
@@ -43,9 +43,10 @@ static void initPid2Machine (void)
 {
/* show machine name rather PID on table Open Files? */
if (PID_or_Machine) {
-   PIDMAP *p;
+   PIDMAP *p, *next;
 
-   for (p = pidmap; p != NULL; ) {
+   for (p = pidmap; p != NULL; p = next) {
+   next = p-next;
DLIST_REMOVE(pidmap, p);
SAFE_FREE(p-machine);
SAFE_FREE(p);
diff --git a/source/winbindd/winbindd_util.c b/source/winbindd/winbindd_util.c
index 3425b4c..cf7663d 100644
--- a/source/winbindd/winbindd_util.c
+++ b/source/winbindd/winbindd_util.c
@@ -1042,13 +1042,12 @@ void free_getent_state(struct getent_state *state)
temp = state;
 
while(temp != NULL) {
-   struct getent_state *next;
+   struct getent_state *next = temp-next;
 
/* Free sam entries then list entry */
 
SAFE_FREE(state-sam_entries);
DLIST_REMOVE(state, state);
-   next = temp-next;
 
SAFE_FREE(temp);
temp = next;


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch v3-3-test updated - release-3-2-0pre2-4204-g7378251

2008-10-01 Thread Jeremy Allison
The branch, v3-3-test has been updated
   via  737825183115732de1f1d6d2bd89ce6402a65b20 (commit)
  from  9a90ceac52f8623dde2bf50e630b10aebbc0e28d (commit)

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


- Log -
commit 737825183115732de1f1d6d2bd89ce6402a65b20
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 13:23:09 2008 -0700

Fix use of DLIST_REMOVE as spotted by Constantine Vetoshev [EMAIL 
PROTECTED].
This API is unusual in that if used to remove a non-list head it nulls out
the next and prev pointers. This is what you want for debugging (don't want
an entry removed from the list to be still virtually linked into it) but
means there is no consistent idiom for use as the next and prev pointers
get trashed on removal from the list, meaning you must save them yourself.
You can use it one way when deleting everything via the head pointer, as
this preserves the next pointer, but you *must* use it another way when not
deleting everything via the head pointer. Fix all known uses of this (the 
main
one is in conn_free_internal() and would not free all the private data 
entries
for vfs modules. The other changes in web/statuspage.c and winbindd_util.c
are not strictly neccessary, as the head pointer is being used, but I've 
done
them for consistency. Long term we must revisit this as this API is too hard
to use correctly.
Jeremy.

---

Summary of changes:
 source/smbd/conn.c  |2 +-
 source/web/statuspage.c |5 +++--
 source/winbindd/winbindd_util.c |3 +--
 3 files changed, 5 insertions(+), 5 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/smbd/conn.c b/source/smbd/conn.c
index b9433bb..7f34d2b 100644
--- a/source/smbd/conn.c
+++ b/source/smbd/conn.c
@@ -252,8 +252,8 @@ void conn_free_internal(connection_struct *conn)
/* Free vfs_connection_struct */
handle = conn-vfs_handles;
while(handle) {
-   DLIST_REMOVE(conn-vfs_handles, handle);
thandle = handle-next;
+   DLIST_REMOVE(conn-vfs_handles, handle);
if (handle-free_data)
handle-free_data(handle-data);
handle = thandle;
diff --git a/source/web/statuspage.c b/source/web/statuspage.c
index ce24c7c..e684a07 100644
--- a/source/web/statuspage.c
+++ b/source/web/statuspage.c
@@ -43,9 +43,10 @@ static void initPid2Machine (void)
 {
/* show machine name rather PID on table Open Files? */
if (PID_or_Machine) {
-   PIDMAP *p;
+   PIDMAP *p, *next;
 
-   for (p = pidmap; p != NULL; ) {
+   for (p = pidmap; p != NULL; p = next) {
+   next = p-next;
DLIST_REMOVE(pidmap, p);
SAFE_FREE(p-machine);
SAFE_FREE(p);
diff --git a/source/winbindd/winbindd_util.c b/source/winbindd/winbindd_util.c
index 7847d1d..ebcf625 100644
--- a/source/winbindd/winbindd_util.c
+++ b/source/winbindd/winbindd_util.c
@@ -1075,13 +1075,12 @@ void free_getent_state(struct getent_state *state)
temp = state;
 
while(temp != NULL) {
-   struct getent_state *next;
+   struct getent_state *next = temp-next;
 
/* Free sam entries then list entry */
 
SAFE_FREE(state-sam_entries);
DLIST_REMOVE(state, state);
-   next = temp-next;
 
SAFE_FREE(temp);
temp = next;


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch v3-0-test updated - release-3-0-32-43-g45514fc

2008-10-01 Thread Jeremy Allison
The branch, v3-0-test has been updated
   via  45514fc5d85221339a4ee55b9f9f78c917b2fd19 (commit)
  from  ffa704347a9cf10de20105e3dd07963f975cbd77 (commit)

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


- Log -
commit 45514fc5d85221339a4ee55b9f9f78c917b2fd19
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 13:23:36 2008 -0700

Fix use of DLIST_REMOVE as spotted by Constantine Vetoshev [EMAIL 
PROTECTED].
This API is unusual in that if used to remove a non-list head it nulls out
the next and prev pointers. This is what you want for debugging (don't want
an entry removed from the list to be still virtually linked into it) but
means there is no consistent idiom for use as the next and prev pointers
get trashed on removal from the list, meaning you must save them yourself.
You can use it one way when deleting everything via the head pointer, as
this preserves the next pointer, but you *must* use it another way when not
deleting everything via the head pointer. Fix all known uses of this (the 
main
one is in conn_free_internal() and would not free all the private data 
entries
for vfs modules. The other changes in web/statuspage.c and winbindd_util.c
are not strictly neccessary, as the head pointer is being used, but I've 
done
them for consistency. Long term we must revisit this as this API is too hard
to use correctly.
Jeremy.

---

Summary of changes:
 source/nsswitch/winbindd_util.c |3 +--
 source/smbd/conn.c  |2 +-
 source/web/statuspage.c |5 +++--
 3 files changed, 5 insertions(+), 5 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/nsswitch/winbindd_util.c b/source/nsswitch/winbindd_util.c
index 130d45c..e72cd7c 100644
--- a/source/nsswitch/winbindd_util.c
+++ b/source/nsswitch/winbindd_util.c
@@ -771,13 +771,12 @@ void free_getent_state(struct getent_state *state)
temp = state;
 
while(temp != NULL) {
-   struct getent_state *next;
+   struct getent_state *next = temp-next;
 
/* Free sam entries then list entry */
 
SAFE_FREE(state-sam_entries);
DLIST_REMOVE(state, state);
-   next = temp-next;
 
SAFE_FREE(temp);
temp = next;
diff --git a/source/smbd/conn.c b/source/smbd/conn.c
index 610e641..4a78b5a 100644
--- a/source/smbd/conn.c
+++ b/source/smbd/conn.c
@@ -261,8 +261,8 @@ void conn_free_internal(connection_struct *conn)
/* Free vfs_connection_struct */
handle = conn-vfs_handles;
while(handle) {
-   DLIST_REMOVE(conn-vfs_handles, handle);
thandle = handle-next;
+   DLIST_REMOVE(conn-vfs_handles, handle);
if (handle-free_data)
handle-free_data(handle-data);
handle = thandle;
diff --git a/source/web/statuspage.c b/source/web/statuspage.c
index e4d726c..7b21a63 100644
--- a/source/web/statuspage.c
+++ b/source/web/statuspage.c
@@ -42,9 +42,10 @@ static void initPid2Machine (void)
 {
/* show machine name rather PID on table Open Files? */
if (PID_or_Machine) {
-   PIDMAP *p;
+   PIDMAP *p, *next;
 
-   for (p = pidmap; p != NULL; ) {
+   for (p = pidmap; p != NULL; p = next) {
+   next = p-next;
DLIST_REMOVE(pidmap, p);
SAFE_FREE(p-machine);
SAFE_FREE(p);


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch v3-3-test updated - release-3-2-0pre2-4205-ge782150

2008-10-01 Thread Jeremy Allison
The branch, v3-3-test has been updated
   via  e782150175ff324b6e0ef1802a9e17f9a5aea560 (commit)
  from  737825183115732de1f1d6d2bd89ce6402a65b20 (commit)

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


- Log -
commit e782150175ff324b6e0ef1802a9e17f9a5aea560
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 15:00:22 2008 -0700

Whitespace cleanup.
Jeremy.

---

Summary of changes:
 source/printing/print_cups.c |   32 
 1 files changed, 16 insertions(+), 16 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/printing/print_cups.c b/source/printing/print_cups.c
index f9568f0..97584cb 100644
--- a/source/printing/print_cups.c
+++ b/source/printing/print_cups.c
@@ -7,12 +7,12 @@
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
- * 
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, see http://www.gnu.org/licenses/.
  */
@@ -109,7 +109,7 @@ bool cups_cache_reload(void)
{
  printer-name,
  printer-info
-   };   
+   };
bool ret = False;
 
DEBUG(5, (reloading cups printcap cache\n));
@@ -657,8 +657,8 @@ static int cups_job_submit(int snum, struct printjob *pjob)
ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_NAME, job-name, NULL,
 new_jobname);
 
-   /* 
-* add any options defined in smb.conf 
+   /*
+* add any options defined in smb.conf
 */
 
num_options = 0;
@@ -666,7 +666,7 @@ static int cups_job_submit(int snum, struct printjob *pjob)
num_options = cupsParseOptions(lp_cups_options(snum), num_options, 
options);
 
if ( num_options )
-   cupsEncodeOptions(request, num_options, options); 
+   cupsEncodeOptions(request, num_options, options);
 
/*
* Do the request and get back a response...
@@ -712,7 +712,7 @@ static int cups_job_submit(int snum, struct printjob *pjob)
 static int cups_queue_get(const char *sharename,
enum printing_types printing_type,
char *lpq_command,
-   print_queue_struct **q, 
+   print_queue_struct **q,
print_status_struct *status)
 {
fstring printername;
@@ -751,10 +751,10 @@ static int cups_queue_get(const char *sharename,
 
*q = NULL;
 
-   /* HACK ALERT!!!  The problem with support the 'printer name' 
-  option is that we key the tdb off the sharename.  So we will 
-  overload the lpq_command string to pass in the printername 
-  (which is basically what we do for non-cups printers ... using 
+   /* HACK ALERT!!!  The problem with support the 'printer name'
+  option is that we key the tdb off the sharename.  So we will
+  overload the lpq_command string to pass in the printername
+  (which is basically what we do for non-cups printers ... using
   the lpq_command to get the queue listing). */
 
fstrcpy( printername, lpq_command );
@@ -1316,22 +1316,22 @@ bool cups_pull_comment_location(NT_PRINTER_INFO_LEVEL_2 
*printer)
/* Grab the comment if we don't have one */
if ( (strcmp(attr-name, printer-info) == 0)
  (attr-value_tag == IPP_TAG_TEXT)
- !strlen(printer-comment) ) 
+ !strlen(printer-comment) )
{
DEBUG(5,(cups_pull_comment_location: Using 
cups comment: %s\n,
-attr-values[0].string.text)); 

+attr-values[0].string.text));
strlcpy(printer-comment,
attr-values[0].string.text,
sizeof(printer-comment));
}
 
-   /* Grab the location if we don't have one */ 
+   /* Grab the location if we don't have one */
if ( (strcmp(attr-name, printer-location) == 0)
- (attr-value_tag == IPP_TAG_TEXT) 
+ (attr-value_tag == IPP_TAG_TEXT)
  

[SCM] Samba Shared Repository - branch master updated - ea85ceefa945f1fee2bb10441a3cc8e35216733b

2008-10-01 Thread Jeremy Allison
The branch, master has been updated
   via  ea85ceefa945f1fee2bb10441a3cc8e35216733b (commit)
  from  e4f5bfb34b7a515e2cf107eb94489260594b8733 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit ea85ceefa945f1fee2bb10441a3cc8e35216733b
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 15:01:05 2008 -0700

Whitespace cleanup.
Jeremy.

---

Summary of changes:
 source3/printing/print_cups.c |   32 
 1 files changed, 16 insertions(+), 16 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/printing/print_cups.c b/source3/printing/print_cups.c
index f9568f0..97584cb 100644
--- a/source3/printing/print_cups.c
+++ b/source3/printing/print_cups.c
@@ -7,12 +7,12 @@
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
- * 
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, see http://www.gnu.org/licenses/.
  */
@@ -109,7 +109,7 @@ bool cups_cache_reload(void)
{
  printer-name,
  printer-info
-   };   
+   };
bool ret = False;
 
DEBUG(5, (reloading cups printcap cache\n));
@@ -657,8 +657,8 @@ static int cups_job_submit(int snum, struct printjob *pjob)
ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_NAME, job-name, NULL,
 new_jobname);
 
-   /* 
-* add any options defined in smb.conf 
+   /*
+* add any options defined in smb.conf
 */
 
num_options = 0;
@@ -666,7 +666,7 @@ static int cups_job_submit(int snum, struct printjob *pjob)
num_options = cupsParseOptions(lp_cups_options(snum), num_options, 
options);
 
if ( num_options )
-   cupsEncodeOptions(request, num_options, options); 
+   cupsEncodeOptions(request, num_options, options);
 
/*
* Do the request and get back a response...
@@ -712,7 +712,7 @@ static int cups_job_submit(int snum, struct printjob *pjob)
 static int cups_queue_get(const char *sharename,
enum printing_types printing_type,
char *lpq_command,
-   print_queue_struct **q, 
+   print_queue_struct **q,
print_status_struct *status)
 {
fstring printername;
@@ -751,10 +751,10 @@ static int cups_queue_get(const char *sharename,
 
*q = NULL;
 
-   /* HACK ALERT!!!  The problem with support the 'printer name' 
-  option is that we key the tdb off the sharename.  So we will 
-  overload the lpq_command string to pass in the printername 
-  (which is basically what we do for non-cups printers ... using 
+   /* HACK ALERT!!!  The problem with support the 'printer name'
+  option is that we key the tdb off the sharename.  So we will
+  overload the lpq_command string to pass in the printername
+  (which is basically what we do for non-cups printers ... using
   the lpq_command to get the queue listing). */
 
fstrcpy( printername, lpq_command );
@@ -1316,22 +1316,22 @@ bool cups_pull_comment_location(NT_PRINTER_INFO_LEVEL_2 
*printer)
/* Grab the comment if we don't have one */
if ( (strcmp(attr-name, printer-info) == 0)
  (attr-value_tag == IPP_TAG_TEXT)
- !strlen(printer-comment) ) 
+ !strlen(printer-comment) )
{
DEBUG(5,(cups_pull_comment_location: Using 
cups comment: %s\n,
-attr-values[0].string.text)); 

+attr-values[0].string.text));
strlcpy(printer-comment,
attr-values[0].string.text,
sizeof(printer-comment));
}
 
-   /* Grab the location if we don't have one */ 
+   /* Grab the location if we don't have one */
if ( (strcmp(attr-name, printer-location) == 0)
- (attr-value_tag == IPP_TAG_TEXT) 
+ (attr-value_tag == IPP_TAG_TEXT)
  

[SCM] Samba Shared Repository - branch v3-2-test updated - release-3-2-0pre2-3063-gd596f78

2008-10-01 Thread Jeremy Allison
The branch, v3-2-test has been updated
   via  d596f78517f33f5a4c4849be3162d4f3031f045f (commit)
  from  347fc4ed725083f2c520afc39f7c81db0dd60324 (commit)

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


- Log -
commit d596f78517f33f5a4c4849be3162d4f3031f045f
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 15:00:35 2008 -0700

Whitespace cleanup.
Jeremy.

---

Summary of changes:
 source/printing/print_cups.c |   38 +++---
 1 files changed, 19 insertions(+), 19 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/printing/print_cups.c b/source/printing/print_cups.c
index 593c5c7..2acf2cb 100644
--- a/source/printing/print_cups.c
+++ b/source/printing/print_cups.c
@@ -7,12 +7,12 @@
  * it under the terms of the GNU General Public License as published by
  * the Free Software Foundation; either version 3 of the License, or
  * (at your option) any later version.
- * 
+ *
  * This program is distributed in the hope that it will be useful,
  * but WITHOUT ANY WARRANTY; without even the implied warranty of
  * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
  * GNU General Public License for more details.
- * 
+ *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, see http://www.gnu.org/licenses/.
  */
@@ -45,7 +45,7 @@ static http_t *cups_connect(void)
http_t *http;
char *server, *p;
int port;
-   
+
if (lp_cups_server() != NULL  strlen(lp_cups_server())  0) {
server = smb_xstrdup(lp_cups_server());
} else {
@@ -59,12 +59,12 @@ static http_t *cups_connect(void)
} else {
port = ippPort();
}
-   
+
DEBUG(10, (connecting to cups server %s:%d\n,
   server, port));
 
if ((http = httpConnect(server, port)) == NULL) {
-   DEBUG(0,(Unable to connect to CUPS server %s:%d - %s\n, 
+   DEBUG(0,(Unable to connect to CUPS server %s:%d - %s\n,
 server, port, strerror(errno)));
SAFE_FREE(server);
return NULL;
@@ -87,7 +87,7 @@ bool cups_cache_reload(void)
{
  printer-name,
  printer-info
-   };   
+   };
bool ret = False;
 
DEBUG(5, (reloading cups printcap cache\n));
@@ -635,8 +635,8 @@ static int cups_job_submit(int snum, struct printjob *pjob)
ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_NAME, job-name, NULL,
 new_jobname);
 
-   /* 
-* add any options defined in smb.conf 
+   /*
+* add any options defined in smb.conf
 */
 
num_options = 0;
@@ -644,7 +644,7 @@ static int cups_job_submit(int snum, struct printjob *pjob)
num_options = cupsParseOptions(lp_cups_options(snum), num_options, 
options);
 
if ( num_options )
-   cupsEncodeOptions(request, num_options, options); 
+   cupsEncodeOptions(request, num_options, options);
 
/*
* Do the request and get back a response...
@@ -690,7 +690,7 @@ static int cups_job_submit(int snum, struct printjob *pjob)
 static int cups_queue_get(const char *sharename,
enum printing_types printing_type,
char *lpq_command,
-   print_queue_struct **q, 
+   print_queue_struct **q,
print_status_struct *status)
 {
fstring printername;
@@ -729,10 +729,10 @@ static int cups_queue_get(const char *sharename,
 
*q = NULL;
 
-   /* HACK ALERT!!!  The problem with support the 'printer name' 
-  option is that we key the tdb off the sharename.  So we will 
-  overload the lpq_command string to pass in the printername 
-  (which is basically what we do for non-cups printers ... using 
+   /* HACK ALERT!!!  The problem with support the 'printer name'
+  option is that we key the tdb off the sharename.  So we will
+  overload the lpq_command string to pass in the printername
+  (which is basically what we do for non-cups printers ... using
   the lpq_command to get the queue listing). */
 
fstrcpy( printername, lpq_command );
@@ -1294,22 +1294,22 @@ bool cups_pull_comment_location(NT_PRINTER_INFO_LEVEL_2 
*printer)
/* Grab the comment if we don't have one */
if ( (strcmp(attr-name, printer-info) == 0)
  (attr-value_tag == IPP_TAG_TEXT)
- !strlen(printer-comment) ) 
+ !strlen(printer-comment) )
{

[SCM] Samba Shared Repository - branch master updated - 3bfb53caa86ece1d7ff1bf3c7759f003203d4247

2008-10-01 Thread Jeremy Allison
The branch, master has been updated
   via  3bfb53caa86ece1d7ff1bf3c7759f003203d4247 (commit)
  from  ea85ceefa945f1fee2bb10441a3cc8e35216733b (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit 3bfb53caa86ece1d7ff1bf3c7759f003203d4247
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 16:40:41 2008 -0700

Fix bug #5080. Access to cups-printers via samba broken with cups 1.3.4, 
Unsupported character set.
Cups 1.3.4 expects utf8 to be used in all messages to/from the server. We 
may be using a
different character set so we need to use talloc utf8 push/pull functions 
in all communication.
Needs more testing. Don't release until I've done a thorough test. I also 
have a version for 3.2.x.
Jeremy.

---

Summary of changes:
 source3/printing/print_cups.c |  341 ++---
 1 files changed, 251 insertions(+), 90 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/printing/print_cups.c b/source3/printing/print_cups.c
index 97584cb..b46d83b 100644
--- a/source3/printing/print_cups.c
+++ b/source3/printing/print_cups.c
@@ -2,6 +2,7 @@
  * Support code for the Common UNIX Printing System (CUPS)
  *
  * Copyright 1999-2003 by Michael R Sweet.
+ * Copyright 2008 Jeremy Allison.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -17,6 +18,10 @@
  * along with this program; if not, see http://www.gnu.org/licenses/.
  */
 
+/*
+ * JRA. Converted to utf8 pull/push.
+ */
+
 #include includes.h
 #include printing.h
 
@@ -51,24 +56,23 @@ cups_passwd_cb(const char *prompt)  /* I - Prompt */
return (NULL);
 }
 
-static http_t *cups_connect(void)
+static http_t *cups_connect(TALLOC_CTX *frame)
 {
-   http_t *http;
-   char *server, *p;
+   http_t *http = NULL;
+   char *server = NULL, *p = NULL;
int port;
int timeout = lp_cups_connection_timeout();
-
-   gotalarm = 0;
-
-   if (timeout) {
-CatchSignal(SIGALRM, SIGNAL_CAST gotalarm_sig);
-alarm(timeout);
-}
+   size_t size;
 
if (lp_cups_server() != NULL  strlen(lp_cups_server())  0) {
-   server = smb_xstrdup(lp_cups_server());
+   if (!push_utf8_talloc(frame, server, lp_cups_server(), size)) 
{
+   return NULL;
+   }
} else {
-   server = smb_xstrdup(cupsServer());
+   server = talloc_strdup(frame,cupsServer());
+   }
+   if (server) {
+   return NULL;
}
 
p = strchr(server, ':');
@@ -82,6 +86,13 @@ static http_t *cups_connect(void)
DEBUG(10, (connecting to cups server %s:%d\n,
   server, port));
 
+   gotalarm = 0;
+
+   if (timeout) {
+CatchSignal(SIGALRM, SIGNAL_CAST gotalarm_sig);
+alarm(timeout);
+}
+
http = httpConnect(server, port);
 
CatchSignal(SIGALRM, SIGNAL_CAST SIG_IGN);
@@ -92,12 +103,12 @@ static http_t *cups_connect(void)
 server, port, strerror(errno)));
}
 
-   SAFE_FREE(server);
return http;
 }
 
 bool cups_cache_reload(void)
 {
+   TALLOC_CTX *frame = talloc_stackframe();
http_t  *http = NULL;   /* HTTP connection to server */
ipp_t   *request = NULL,/* IPP Request */
*response = NULL;   /* IPP Response */
@@ -111,6 +122,7 @@ bool cups_cache_reload(void)
  printer-info
};
bool ret = False;
+   size_t size;
 
DEBUG(5, (reloading cups printcap cache\n));
 
@@ -124,7 +136,7 @@ bool cups_cache_reload(void)
* Try to connect to the server...
*/
 
-   if ((http = cups_connect()) == NULL) {
+   if ((http = cups_connect(frame)) == NULL) {
goto out;
}
 
@@ -145,7 +157,7 @@ bool cups_cache_reload(void)
language = cupsLangDefault();
 
ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_CHARSET,
- attributes-charset, NULL, cupsLangEncoding(language));
+ attributes-charset, NULL, utf-8);
 
ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_LANGUAGE,
  attributes-natural-language, NULL, language-language);
@@ -185,12 +197,24 @@ bool cups_cache_reload(void)
 
while (attr != NULL  attr-group_tag == IPP_TAG_PRINTER) {
if (strcmp(attr-name, printer-name) == 0 
-   attr-value_tag == IPP_TAG_NAME)
-   name = attr-values[0].string.text;
+   attr-value_tag == IPP_TAG_NAME) {
+

[SCM] Samba Shared Repository - branch v3-3-test updated - release-3-2-0pre2-4206-g2a2bb6b

2008-10-01 Thread Jeremy Allison
The branch, v3-3-test has been updated
   via  2a2bb6b0757b1f2a6199474f93ea8d598575d650 (commit)
  from  e782150175ff324b6e0ef1802a9e17f9a5aea560 (commit)

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


- Log -
commit 2a2bb6b0757b1f2a6199474f93ea8d598575d650
Author: Jeremy Allison [EMAIL PROTECTED]
Date:   Wed Oct 1 16:39:29 2008 -0700

Fix bug #5080. Access to cups-printers via samba broken with cups 1.3.4, 
Unsupported character set.
Cups 1.3.4 expects utf8 to be used in all messages to/from the server. We 
may be using a
different character set so we need to use talloc utf8 push/pull functions 
in all communication.
Needs more testing. Don't release until I've done a thorough test. I also 
have a version for 3.2.x.
Jeremy.

---

Summary of changes:
 source/printing/print_cups.c |  341 +++---
 1 files changed, 251 insertions(+), 90 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source/printing/print_cups.c b/source/printing/print_cups.c
index 97584cb..b46d83b 100644
--- a/source/printing/print_cups.c
+++ b/source/printing/print_cups.c
@@ -2,6 +2,7 @@
  * Support code for the Common UNIX Printing System (CUPS)
  *
  * Copyright 1999-2003 by Michael R Sweet.
+ * Copyright 2008 Jeremy Allison.
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
@@ -17,6 +18,10 @@
  * along with this program; if not, see http://www.gnu.org/licenses/.
  */
 
+/*
+ * JRA. Converted to utf8 pull/push.
+ */
+
 #include includes.h
 #include printing.h
 
@@ -51,24 +56,23 @@ cups_passwd_cb(const char *prompt)  /* I - Prompt */
return (NULL);
 }
 
-static http_t *cups_connect(void)
+static http_t *cups_connect(TALLOC_CTX *frame)
 {
-   http_t *http;
-   char *server, *p;
+   http_t *http = NULL;
+   char *server = NULL, *p = NULL;
int port;
int timeout = lp_cups_connection_timeout();
-
-   gotalarm = 0;
-
-   if (timeout) {
-CatchSignal(SIGALRM, SIGNAL_CAST gotalarm_sig);
-alarm(timeout);
-}
+   size_t size;
 
if (lp_cups_server() != NULL  strlen(lp_cups_server())  0) {
-   server = smb_xstrdup(lp_cups_server());
+   if (!push_utf8_talloc(frame, server, lp_cups_server(), size)) 
{
+   return NULL;
+   }
} else {
-   server = smb_xstrdup(cupsServer());
+   server = talloc_strdup(frame,cupsServer());
+   }
+   if (server) {
+   return NULL;
}
 
p = strchr(server, ':');
@@ -82,6 +86,13 @@ static http_t *cups_connect(void)
DEBUG(10, (connecting to cups server %s:%d\n,
   server, port));
 
+   gotalarm = 0;
+
+   if (timeout) {
+CatchSignal(SIGALRM, SIGNAL_CAST gotalarm_sig);
+alarm(timeout);
+}
+
http = httpConnect(server, port);
 
CatchSignal(SIGALRM, SIGNAL_CAST SIG_IGN);
@@ -92,12 +103,12 @@ static http_t *cups_connect(void)
 server, port, strerror(errno)));
}
 
-   SAFE_FREE(server);
return http;
 }
 
 bool cups_cache_reload(void)
 {
+   TALLOC_CTX *frame = talloc_stackframe();
http_t  *http = NULL;   /* HTTP connection to server */
ipp_t   *request = NULL,/* IPP Request */
*response = NULL;   /* IPP Response */
@@ -111,6 +122,7 @@ bool cups_cache_reload(void)
  printer-info
};
bool ret = False;
+   size_t size;
 
DEBUG(5, (reloading cups printcap cache\n));
 
@@ -124,7 +136,7 @@ bool cups_cache_reload(void)
* Try to connect to the server...
*/
 
-   if ((http = cups_connect()) == NULL) {
+   if ((http = cups_connect(frame)) == NULL) {
goto out;
}
 
@@ -145,7 +157,7 @@ bool cups_cache_reload(void)
language = cupsLangDefault();
 
ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_CHARSET,
- attributes-charset, NULL, cupsLangEncoding(language));
+ attributes-charset, NULL, utf-8);
 
ippAddString(request, IPP_TAG_OPERATION, IPP_TAG_LANGUAGE,
  attributes-natural-language, NULL, language-language);
@@ -185,12 +197,24 @@ bool cups_cache_reload(void)
 
while (attr != NULL  attr-group_tag == IPP_TAG_PRINTER) {
if (strcmp(attr-name, printer-name) == 0 
-   attr-value_tag == IPP_TAG_NAME)
-   name = attr-values[0].string.text;
+   attr-value_tag == IPP_TAG_NAME) {
+  

Build status as of Thu Oct 2 00:00:02 2008

2008-10-01 Thread build
URL: http://build.samba.org/

--- /home/build/master/cache/broken_results.txt.old 2008-10-01 
00:01:04.0 +
+++ /home/build/master/cache/broken_results.txt 2008-10-02 00:00:25.0 
+
@@ -1,22 +1,22 @@
-Build status as of Wed Oct  1 00:00:02 2008
+Build status as of Thu Oct  2 00:00:02 2008
 
 Build counts:
 Tree Total  Broken Panic 
 build_farm   0  0  0 
-ccache   33 8  0 
+ccache   30 8  0 
 ctdb 0  0  0 
 distcc   1  0  0 
-ldb  33 32 0 
-libreplace   30 12 0 
+ldb  31 32 0 
+libreplace   31 12 0 
 lorikeet-heimdal 29 20 0 
 pidl 19 19 0 
 ppp  14 0  0 
 rsync33 11 0 
 samba-docs   0  0  0 
 samba-gtk8  8  0 
-samba_3_X_devel 28 17 0 
+samba_3_X_devel 29 17 0 
 samba_3_X_test 29 17 0 
-samba_4_0_test 32 30 2 
+samba_4_0_test 32 29 1 
 smb-build31 7  0 
 talloc   32 29 0 
 tdb  30 12 0 


[SCM] Samba Shared Repository - branch master updated - 9097a67def72ea41e85a4acb5d7cbcc46adaaa84

2008-10-01 Thread Volker Lendecke
The branch, master has been updated
   via  9097a67def72ea41e85a4acb5d7cbcc46adaaa84 (commit)
  from  3bfb53caa86ece1d7ff1bf3c7759f003203d4247 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit 9097a67def72ea41e85a4acb5d7cbcc46adaaa84
Author: Volker Lendecke [EMAIL PROTECTED]
Date:   Thu Oct 2 02:17:04 2008 +0200

Fix bug 5798: CFLAGS info lost in configure

Michael, please check and merge to the other branches if it's right.

Thanks,

Volker

---

Summary of changes:
 source3/configure.in |2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/configure.in b/source3/configure.in
index f04ddbe..545a565 100644
--- a/source3/configure.in
+++ b/source3/configure.in
@@ -128,7 +128,7 @@ fi
 if test x$debug = xyes ; then
CFLAGS=${CFLAGS} -g
 else
-   CFLAGS=-O
+   CFLAGS=${CFLAGS} -O
 fi
 
 m4_include(../lib/socket_wrapper/config.m4)


-- 
Samba Shared Repository


[SCM] Samba Shared Repository - branch master updated - 8641b54a736c5c924bf38cf4574d1f8e34d2d0cd

2008-10-01 Thread Volker Lendecke
The branch, master has been updated
   via  8641b54a736c5c924bf38cf4574d1f8e34d2d0cd (commit)
  from  9097a67def72ea41e85a4acb5d7cbcc46adaaa84 (commit)

http://gitweb.samba.org/?p=samba.git;a=shortlog;h=master


- Log -
commit 8641b54a736c5c924bf38cf4574d1f8e34d2d0cd
Author: Volker Lendecke [EMAIL PROTECTED]
Date:   Thu Oct 2 03:37:52 2008 +0200

Attempt to fix the build on IRIX

Under irix, sa_family is a #define to sa_union.sa_generic.sa_family2

---

Summary of changes:
 source3/librpc/gen_ndr/nbt.h |2 +-
 source3/librpc/gen_ndr/ndr_nbt.c |6 +++---
 source3/librpc/idl/nbt.idl   |2 +-
 source3/libsmb/dsgetdcname.c |2 +-
 4 files changed, 6 insertions(+), 6 deletions(-)


Changeset truncated at 500 lines:

diff --git a/source3/librpc/gen_ndr/nbt.h b/source3/librpc/gen_ndr/nbt.h
index 264b00b..4b872d7 100644
--- a/source3/librpc/gen_ndr/nbt.h
+++ b/source3/librpc/gen_ndr/nbt.h
@@ -391,7 +391,7 @@ struct nbt_dgram_packet {
 }/* 
[public,flag(LIBNDR_FLAG_NOALIGN|LIBNDR_FLAG_BIGENDIAN|LIBNDR_PRINT_ARRAY_HEX)] 
*/;
 
 struct nbt_sockaddr {
-   uint32_t sa_family;
+   uint32_t sockaddr_family;
const char * pdc_ip;/* [flag(LIBNDR_FLAG_BIGENDIAN)] */
DATA_BLOB remaining;/* [flag(LIBNDR_FLAG_REMAINING)] */
 }/* [gensize,public] */;
diff --git a/source3/librpc/gen_ndr/ndr_nbt.c b/source3/librpc/gen_ndr/ndr_nbt.c
index 84cfabd..c02b539 100644
--- a/source3/librpc/gen_ndr/ndr_nbt.c
+++ b/source3/librpc/gen_ndr/ndr_nbt.c
@@ -1531,7 +1531,7 @@ _PUBLIC_ enum ndr_err_code ndr_push_nbt_sockaddr(struct 
ndr_push *ndr, int ndr_f
 {
if (ndr_flags  NDR_SCALARS) {
NDR_CHECK(ndr_push_align(ndr, 4));
-   NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, r-sa_family));
+   NDR_CHECK(ndr_push_uint32(ndr, NDR_SCALARS, 
r-sockaddr_family));
{
uint32_t _flags_save_ipv4address = ndr-flags;
ndr_set_flags(ndr-flags, LIBNDR_FLAG_BIGENDIAN);
@@ -1554,7 +1554,7 @@ _PUBLIC_ enum ndr_err_code ndr_pull_nbt_sockaddr(struct 
ndr_pull *ndr, int ndr_f
 {
if (ndr_flags  NDR_SCALARS) {
NDR_CHECK(ndr_pull_align(ndr, 4));
-   NDR_CHECK(ndr_pull_uint32(ndr, NDR_SCALARS, r-sa_family));
+   NDR_CHECK(ndr_pull_uint32(ndr, NDR_SCALARS, 
r-sockaddr_family));
{
uint32_t _flags_save_ipv4address = ndr-flags;
ndr_set_flags(ndr-flags, LIBNDR_FLAG_BIGENDIAN);
@@ -1577,7 +1577,7 @@ _PUBLIC_ void ndr_print_nbt_sockaddr(struct ndr_print 
*ndr, const char *name, co
 {
ndr_print_struct(ndr, name, nbt_sockaddr);
ndr-depth++;
-   ndr_print_uint32(ndr, sa_family, r-sa_family);
+   ndr_print_uint32(ndr, sockaddr_family, r-sockaddr_family);
ndr_print_ipv4address(ndr, pdc_ip, r-pdc_ip);
ndr_print_DATA_BLOB(ndr, remaining, r-remaining);
ndr-depth--;
diff --git a/source3/librpc/idl/nbt.idl b/source3/librpc/idl/nbt.idl
index 82571d9..f3590fc 100644
--- a/source3/librpc/idl/nbt.idl
+++ b/source3/librpc/idl/nbt.idl
@@ -339,7 +339,7 @@ interface nbt
 */
 
typedef [public,gensize] struct {
-   uint32  sa_family;
+   uint32  sockaddr_family;
[flag(NDR_BIG_ENDIAN)]  ipv4address pdc_ip;
[flag(NDR_REMAINING)]   DATA_BLOB remaining;
} nbt_sockaddr;
diff --git a/source3/libsmb/dsgetdcname.c b/source3/libsmb/dsgetdcname.c
index 89769d8..d2ede1d 100644
--- a/source3/libsmb/dsgetdcname.c
+++ b/source3/libsmb/dsgetdcname.c
@@ -200,7 +200,7 @@ static NTSTATUS store_cldap_reply(TALLOC_CTX *mem_ctx,
 
/* FIXME */
r-sockaddr_size = 0x10; /* the w32 winsock addr size */
-   r-sockaddr.sa_family = 2; /* AF_INET */
+   r-sockaddr.sockaddr_family = 2; /* AF_INET */
r-sockaddr.pdc_ip = talloc_strdup(mem_ctx, addr);
 
ndr_err = ndr_push_struct_blob(blob, mem_ctx, NULL, r,


-- 
Samba Shared Repository