s3:smbd/service apply some code formatting
authorChristian Ambach <ambi@samba.org>
Sun, 3 Apr 2016 03:16:45 +0000 (05:16 +0200)
committerJeremy Allison <jra@samba.org>
Tue, 3 May 2016 23:28:23 +0000 (01:28 +0200)
reduce indentation in switch statement, obey 80 char line limit, use C99 bool

Signed-off-by: Christian Ambach <ambi@samba.org>
Reviewed-by: Jeremy Allison <jra@samba.org>
source3/smbd/service.c

index e4a910afa0b455066a3996116423742a05ec0225..34cc369ce19f2266010101dd0f293f5feb4f47a9 100644 (file)
@@ -181,6 +181,7 @@ bool set_conn_connectpath(connection_struct *conn, const char *connectpath)
 bool set_current_service(connection_struct *conn, uint16_t flags, bool do_chdir)
 {
        int snum;
+       enum remote_arch_types ra_type;
 
        if (!conn)  {
                last_conn = NULL;
@@ -206,30 +207,35 @@ bool set_current_service(connection_struct *conn, uint16_t flags, bool do_chdir)
        last_conn = conn;
        last_flags = flags;
 
-       /* Obey the client case sensitivity requests - only for clients that support it. */
+       /*
+        * Obey the client case sensitivity requests - only for clients that
+        * support it. */
        switch (lp_case_sensitive(snum)) {
-               case Auto:
-                       {
-                               /* We need this uglyness due to DOS/Win9x clients that lie about case insensitivity. */
-                               enum remote_arch_types ra_type = get_remote_arch();
-                               if (conn->sconn->using_smb2) {
-                                       conn->case_sensitive = false;
-                               } else if ((ra_type != RA_SAMBA) && (ra_type != RA_CIFSFS)) {
-                                       /* Client can't support per-packet case sensitive pathnames. */
-                                       conn->case_sensitive = False;
-                               } else {
-                                       conn->case_sensitive = !(flags & FLAG_CASELESS_PATHNAMES);
-                               }
-                       }
-                       break;
-               case True:
-                       conn->case_sensitive = True;
-                       break;
-               default:
-                       conn->case_sensitive = False;
-                       break;
-       }
-       return(True);
+       case Auto:
+               /*
+                * We need this uglyness due to DOS/Win9x clients that lie
+                * about case insensitivity. */
+               ra_type = get_remote_arch();
+               if (conn->sconn->using_smb2) {
+                       conn->case_sensitive = false;
+               } else if ((ra_type != RA_SAMBA) && (ra_type != RA_CIFSFS)) {
+                       /*
+                        * Client can't support per-packet case sensitive
+                        * pathnames. */
+                       conn->case_sensitive = false;
+               } else {
+                       conn->case_sensitive =
+                                       !(flags & FLAG_CASELESS_PATHNAMES);
+               }
+       break;
+       case True:
+               conn->case_sensitive = true;
+               break;
+       default:
+               conn->case_sensitive = false;
+               break;
+       }
+       return true;
 }
 
 /****************************************************************************