Make protocol version 2 the default protocol, and only run on version 1 if V1 is...
authorHolger Hetterich <hhetter@novell.com>
Wed, 18 May 2011 13:24:23 +0000 (15:24 +0200)
committerJeremy Allison <jra@samba.org>
Wed, 18 May 2011 19:36:44 +0000 (21:36 +0200)
I haven't received a single line of feedback on protocol v1
for at least 1 1/2 years, whereas protocol v2 has an active
userbase and more people developing around it.

This patch includes a manpage update, describing the new
version handling, as well as documenting the recent changes
making the module transfer the IP address of the client machine
as submitted with
464c69609aa7e582f484c1d357b7c6d3eb2bcbe3.

Signed-off-by: Jeremy Allison <jra@samba.org>
Autobuild-User: Jeremy Allison <jra@samba.org>
Autobuild-Date: Wed May 18 21:36:44 CEST 2011 on sn-devel-104

docs-xml/manpages-3/vfs_smb_traffic_analyzer.8.xml
source3/modules/vfs_smb_traffic_analyzer.c

index 882ee6af3f8d66f4381d0b985bf35f71aaa7a1fa..945070393cd9763b863f12b6e0093cf82d42610a 100644 (file)
@@ -45,6 +45,8 @@
        protocol, supporting only a small list of VFS operations, and had
        several drawbacks. The protocol version 2 is a try to solve the
        problems version 1 had while at the same time adding new features.
+       With the release of Samba 3.6.0, the module will run protocol version 2
+       by default.
        </para>
 </refsect1>
 
@@ -77,6 +79,7 @@
        <listitem><para><command>SHARE</command> - the name of the share on which the VFS operation occured</para></listitem>
        <listitem><para><command>FILENAME</command> - the name of the file that was used by the VFS operation</para></listitem>
        <listitem><para><command>TIMESTAMP</command> - a timestamp, formatted as "yyyy-mm-dd hh-mm-ss.ms" indicating when the VFS operation occured</para></listitem>
+       <listitem><para><command>IP</command> - The IP Address (v4 or v6) of the client machine that initiated the VFS operation.</para></listitem>
        </itemizedlist>
 
        </para>
                <varlistentry>
                <term>smb_traffic_analyzer:protocol_version = STRING</term>
                <listitem>
-               <para>If STRING matches to V1 or is not given at all, the module
-               will use version 1 of the protocol. If STRING matches to "V2"
-               the module will use version 2 of the protocol.
+               <para>If STRING matches to V1, the module will use version 1 of the
+               protocol. If STRING is not given, the module will use version 2 of the
+               protocol, which is the default.
                </para>
                </listitem>
                </varlistentry>
        <smbconfsection name="[example_share]"/>
        <smbconfoption name="path">/data/example</smbconfoption>
        <smbconfoption name="vfs_objects">smb_traffic_analyzer</smbconfoption>
-       <smbconfoption name="smb_traffic_analyzer:protocol_version">V2</smbconfoption>
        <smbconfoption name="smb_traffic_analyzer:host">examplehost</smbconfoption>
        <smbconfoption name="smb_traffic_analyzer:port">3491</smbconfoption>
        </programlisting>
index d1340c310c920cc1f06c9f2e0fec465b77f4aafd..4146d78624f7381cd2424292b58a93211c7c75d4 100644 (file)
@@ -452,7 +452,7 @@ static void smb_traffic_analyzer_send_data(vfs_handle_struct *handle,
                                        "protocol_version", NULL );
 
 
-       if ( protocol_version == NULL || strcmp( protocol_version,"V1") == 0) {
+       if (protocol_version != NULL && strcmp(protocol_version,"V1") == 0) {
 
                struct rw_data *s_data = (struct rw_data *) data;
 
@@ -489,7 +489,10 @@ static void smb_traffic_analyzer_send_data(vfs_handle_struct *handle,
                return;
                }
 
-       } else if ( strcmp( protocol_version, "V2") == 0) {
+       } else {
+               /**
+                * Protocol 2 is used by default.
+                */
 
                switch( vfs_operation ) {
                case vfs_id_open: ;
@@ -557,10 +560,6 @@ static void smb_traffic_analyzer_send_data(vfs_handle_struct *handle,
                        return;
                }
 
-       } else {
-               DEBUG(1, ("smb_traffic_analyzer_send_data_socket: "
-                       "error, unknown protocol given!\n"));
-               return;
        }
 
        if (!str) {