s3-privileges: remove trailing whitespace from privileges codes.
authorGünther Deschner <gd@samba.org>
Fri, 15 May 2009 23:21:08 +0000 (01:21 +0200)
committerGünther Deschner <gd@samba.org>
Fri, 15 May 2009 23:27:51 +0000 (01:27 +0200)
Guenther

source3/lib/privileges.c
source3/lib/privileges_basic.c

index c1bb783fbc2174ed28d97cedfe5e1b89952765bd..b3574da858deedb426b6395f37637c4b5b24f280 100644 (file)
@@ -5,22 +5,22 @@
    Copyright (C) Simo Sorce                    2002-2003
    Copyright (C) Gerald (Jerry) Carter          2005
    Copyright (C) Michael Adam                  2007
-   
+
    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
    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/>.
 */
 
+
 #include "includes.h"
 
 #define PRIVPREFIX              "PRIV_"
@@ -44,28 +44,28 @@ static bool get_privileges( const DOM_SID *sid, SE_PRIV *mask )
        TDB_DATA data;
 
        /* Fail if the admin has not enable privileges */
-       
+
        if ( !lp_enable_privileges() ) {
                return False;
        }
-       
+
        if ( db == NULL )
                return False;
 
        /* PRIV_<SID> (NULL terminated) as the key */
-       
+
        fstr_sprintf(keystr, "%s%s", PRIVPREFIX, sid_to_fstring(tmp, sid));
 
        data = dbwrap_fetch_bystring( db, talloc_tos(), keystr );
-       
+
        if ( !data.dptr ) {
                DEBUG(3, ("get_privileges: No privileges assigned to SID "
                          "[%s]\n", sid_string_dbg(sid)));
                return False;
        }
-       
+
        SMB_ASSERT( data.dsize == sizeof( SE_PRIV ) );
-       
+
        se_priv_copy( mask, (SE_PRIV*)data.dptr );
        TALLOC_FREE(data.dptr);
 
@@ -81,7 +81,7 @@ static bool set_privileges( const DOM_SID *sid, SE_PRIV *mask )
        struct db_context *db = get_account_pol_db();
        fstring tmp, keystr;
        TDB_DATA data;
-       
+
        if ( !lp_enable_privileges() )
                return False;
 
@@ -94,11 +94,11 @@ static bool set_privileges( const DOM_SID *sid, SE_PRIV *mask )
        }
 
        /* PRIV_<SID> (NULL terminated) as the key */
-       
+
        fstr_sprintf(keystr, "%s%s", PRIVPREFIX, sid_to_fstring(tmp, sid));
-       
+
        /* no packing.  static size structure, just write it out */
-       
+
        data.dptr  = (uint8 *)mask;
        data.dsize = sizeof(SE_PRIV);
 
@@ -117,7 +117,7 @@ bool get_privileges_for_sids(SE_PRIV *privileges, DOM_SID *slist, int scount)
        bool found = False;
 
        se_priv_copy( privileges, &se_priv_none );
-       
+
        for ( i=0; i<scount; i++ ) {
                /* don't add unless we actually have a privilege assigned */
 
@@ -127,7 +127,7 @@ bool get_privileges_for_sids(SE_PRIV *privileges, DOM_SID *slist, int scount)
                DEBUG(5,("get_privileges_for_sids: sid = %s\nPrivilege "
                         "set:\n", sid_string_dbg(&slist[i])));
                dump_se_priv( DBGC_ALL, 5, &mask );
-                       
+
                se_priv_add( privileges, &mask );
                found = True;
        }
@@ -146,9 +146,9 @@ static int priv_traverse_fn(struct db_record *rec, void *state)
        int  prefixlen = strlen(PRIVPREFIX);
        DOM_SID sid;
        fstring sid_string;
-       
+
        /* easy check first */
-       
+
        if (rec->value.dsize != sizeof(SE_PRIV) )
                return 0;
 
@@ -156,21 +156,21 @@ static int priv_traverse_fn(struct db_record *rec, void *state)
 
        if ( strncmp((char *)rec->key.dptr, PRIVPREFIX, prefixlen) != 0)
                return 0;
-               
+
        /* check to see if we are looking for a particular privilege */
 
        if ( !se_priv_equal(&priv->privilege, &se_priv_none) ) {
                SE_PRIV mask;
-               
+
                se_priv_copy( &mask, (SE_PRIV*)rec->value.dptr );
-               
-               /* if the SID does not have the specified privilege 
+
+               /* if the SID does not have the specified privilege
                   then just return */
-                  
+
                if ( !is_privilege_assigned( &mask, &priv->privilege) )
                        return 0;
        }
-               
+
        fstrcpy( sid_string, (char *)&(rec->key.dptr[strlen(PRIVPREFIX)]) );
 
        /* this is a last ditch safety check to preventing returning
@@ -191,7 +191,7 @@ static int priv_traverse_fn(struct db_record *rec, void *state)
        {
                return 0;
        }
-       
+
        return 0;
 }
 
@@ -203,7 +203,7 @@ NTSTATUS privilege_enumerate_accounts(DOM_SID **sids, int *num_sids)
 {
        struct db_context *db = get_account_pol_db();
        PRIV_SID_LIST priv;
-       
+
        if (db == NULL) {
                return NT_STATUS_ACCESS_DENIED;
        }
@@ -215,7 +215,7 @@ NTSTATUS privilege_enumerate_accounts(DOM_SID **sids, int *num_sids)
        db->traverse_read(db, priv_traverse_fn, &priv);
 
        /* give the memory away; caller will free */
-       
+
        *sids      = priv.sids.list;
        *num_sids  = priv.sids.count;
 
@@ -258,7 +258,7 @@ NTSTATUS privilege_enum_sids(const SE_PRIV *mask, TALLOC_CTX *mem_ctx,
 bool grant_privilege(const DOM_SID *sid, const SE_PRIV *priv_mask)
 {
        SE_PRIV old_mask, new_mask;
-       
+
        ZERO_STRUCT( old_mask );
        ZERO_STRUCT( new_mask );
 
@@ -270,13 +270,13 @@ bool grant_privilege(const DOM_SID *sid, const SE_PRIV *priv_mask)
        se_priv_add( &new_mask, priv_mask );
 
        DEBUG(10,("grant_privilege: %s\n", sid_string_dbg(sid)));
-       
+
        DEBUGADD( 10, ("original privilege mask:\n"));
        dump_se_priv( DBGC_ALL, 10, &old_mask );
-       
+
        DEBUGADD( 10, ("new privilege mask:\n"));
        dump_se_priv( DBGC_ALL, 10, &new_mask );
-       
+
        return set_privileges( sid, &new_mask );
 }
 
@@ -304,22 +304,22 @@ bool grant_privilege_by_name(DOM_SID *sid, const char *name)
 bool revoke_privilege(const DOM_SID *sid, const SE_PRIV *priv_mask)
 {
        SE_PRIV mask;
-       
+
        /* if the user has no privileges, then we can't revoke any */
-       
+
        if ( !get_privileges( sid, &mask ) )
                return True;
-       
+
        DEBUG(10,("revoke_privilege: %s\n", sid_string_dbg(sid)));
-       
+
        DEBUGADD( 10, ("original privilege mask:\n"));
        dump_se_priv( DBGC_ALL, 10, &mask );
 
        se_priv_remove( &mask, priv_mask );
-       
+
        DEBUGADD( 10, ("new privilege mask:\n"));
        dump_se_priv( DBGC_ALL, 10, &mask );
-       
+
        return set_privileges( sid, &mask );
 }
 
@@ -360,13 +360,13 @@ NTSTATUS privilege_create_account(const DOM_SID *sid )
 }
 
 /****************************************************************************
- initialise a privilege list and set the talloc context 
+ initialise a privilege list and set the talloc context
  ****************************************************************************/
+
 NTSTATUS privilege_set_init(PRIVILEGE_SET *priv_set)
 {
        TALLOC_CTX *mem_ctx;
-       
+
        ZERO_STRUCTP( priv_set );
 
        mem_ctx = talloc_init("privilege set");
@@ -381,13 +381,13 @@ NTSTATUS privilege_set_init(PRIVILEGE_SET *priv_set)
 }
 
 /****************************************************************************
-  initialise a privilege list and with someone else's talloc context 
+  initialise a privilege list and with someone else's talloc context
 ****************************************************************************/
 
 NTSTATUS privilege_set_init_by_ctx(TALLOC_CTX *mem_ctx, PRIVILEGE_SET *priv_set)
 {
        ZERO_STRUCTP( priv_set );
-       
+
        priv_set->mem_ctx = mem_ctx;
        priv_set->ext_ctx = True;
 
@@ -435,7 +435,7 @@ NTSTATUS dup_luid_attr(TALLOC_CTX *mem_ctx, LUID_ATTR **new_la, LUID_ATTR *old_l
                (*new_la)[i].luid.low = old_la[i].luid.low;
                (*new_la)[i].attr = old_la[i].attr;
        }
-       
+
        return NT_STATUS_OK;
 }
 
@@ -445,7 +445,7 @@ NTSTATUS dup_luid_attr(TALLOC_CTX *mem_ctx, LUID_ATTR **new_la, LUID_ATTR *old_l
 bool is_privileged_sid( const DOM_SID *sid )
 {
        SE_PRIV mask;
-       
+
        return get_privileges( sid, &mask );
 }
 
@@ -459,6 +459,6 @@ bool grant_all_privileges( const DOM_SID *sid )
        if (!se_priv_put_all_privileges(&mask)) {
                return False;
        }
-       
+
        return grant_privilege( sid, &mask );
 }
index 865c1f655c90f7fd2ebd298ae1b5456c4a46cbaa..323983b1683b90a72b5f4c8168350a9eb5b09ad4 100644 (file)
@@ -5,23 +5,23 @@
    Copyright (C) Simo Sorce                    2002-2003
    Copyright (C) Gerald (Jerry) Carter          2005
    Copyright (C) Michael Adam                  2007
-   
+
    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
    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/>.
 */
 
 /*
- * Basic privileges functions (mask-operations and conversion 
+ * Basic privileges functions (mask-operations and conversion
  * functions between the different formats (se_priv, privset, luid)
  * moved here * from lib/privileges.c to minimize linker deps.
  *
@@ -29,7 +29,7 @@
  *
  * some extra functions to hide privs array from lib/privileges.c
  */
+
 #include "includes.h"
 
 const SE_PRIV se_priv_all         = SE_ALL_PRIVS;
@@ -49,7 +49,7 @@ const SE_PRIV se_take_ownership  = SE_TAKE_OWNERSHIP;
 
 /********************************************************************
  This is a list of privileges reported by a WIndows 2000 SP4 AD DC
- just for reference purposes (and I know the LUID is not guaranteed 
+ just for reference purposes (and I know the LUID is not guaranteed
  across reboots):
 
             SeCreateTokenPrivilege  Create a token object ( 0x0, 0x2 )
@@ -87,11 +87,11 @@ const SE_PRIV se_take_ownership  = SE_TAKE_OWNERSHIP;
 /* we have to define the LUID here due to a horrible check by printmig.exe
    that requires the SeBackupPrivilege match what is in Windows.  So match
    those that we implement and start Samba privileges at 0x1001 */
-   
+
 PRIVS privs[] = {
-#if 0  /* usrmgr will display these twice if you include them.  We don't 
+#if 0  /* usrmgr will display these twice if you include them.  We don't
           use them but we'll keep the bitmasks reserved in privileges.h anyways */
-          
+
        {SE_NETWORK_LOGON,      "SeNetworkLogonRight",          "Access this computer from network",       { 0x0, 0x0 }},
        {SE_INTERACTIVE_LOGON,  "SeInteractiveLogonRight",      "Log on locally",                          { 0x0, 0x0 }},
        {SE_BATCH_LOGON,        "SeBatchLogonRight",            "Log on as a batch job",                   { 0x0, 0x0 }},
@@ -102,7 +102,7 @@ PRIVS privs[] = {
         {SE_BACKUP,             "SeBackupPrivilege",            "Back up files and directories",          { 0x0, 0x0011 }},
         {SE_RESTORE,            "SeRestorePrivilege",           "Restore files and directories",          { 0x0, 0x0012 }},
        {SE_REMOTE_SHUTDOWN,    "SeRemoteShutdownPrivilege",    "Force shutdown from a remote system",     { 0x0, 0x0018 }},
-       
+
        {SE_PRINT_OPERATOR,     "SePrintOperatorPrivilege",     "Manage printers",                         { 0x0, 0x1001 }},
        {SE_ADD_USERS,          "SeAddUsersPrivilege",          "Add users and groups to the domain",      { 0x0, 0x1002 }},
        {SE_DISK_OPERATOR,      "SeDiskOperatorPrivilege",      "Manage disk shares",                      { 0x0, 0x1003 }},
@@ -118,9 +118,9 @@ bool se_priv_copy( SE_PRIV *dst, const SE_PRIV *src )
 {
        if ( !dst || !src )
                return False;
-               
+
        memcpy( dst, src, sizeof(SE_PRIV) );
-       
+
        return True;
 }
 
@@ -137,7 +137,7 @@ bool se_priv_put_all_privileges(SE_PRIV *mask)
                return False;
        }
        for ( i=0; i<num_privs; i++ ) {
-               se_priv_add(mask, &privs[i].se_priv); 
+               se_priv_add(mask, &privs[i].se_priv);
        }
        return True;
 }
@@ -156,12 +156,12 @@ void se_priv_add( SE_PRIV *mask, const SE_PRIV *addpriv )
 }
 
 /***************************************************************************
- remove one SE_PRIV sytucture from another and store the resulting set 
+ remove one SE_PRIV sytucture from another and store the resulting set
  in mew_mask
 ****************************************************************************/
 
 void se_priv_remove( SE_PRIV *mask, const SE_PRIV *removepriv )
-{      
+{
        int i;
 
        for ( i=0; i<SE_PRIV_MASKSIZE; i++ ) {
@@ -174,9 +174,9 @@ void se_priv_remove( SE_PRIV *mask, const SE_PRIV *removepriv )
 ****************************************************************************/
 
 static void se_priv_invert( SE_PRIV *new_mask, const SE_PRIV *mask )
-{      
+{
        SE_PRIV allprivs;
-       
+
        se_priv_copy( &allprivs, &se_priv_all );
        se_priv_remove( &allprivs, mask );
        se_priv_copy( new_mask, &allprivs );
@@ -187,7 +187,7 @@ static void se_priv_invert( SE_PRIV *new_mask, const SE_PRIV *mask )
 ****************************************************************************/
 
 bool se_priv_equal( const SE_PRIV *mask1, const SE_PRIV *mask2 )
-{      
+{
        return ( memcmp(mask1, mask2, sizeof(SE_PRIV)) == 0 );
 }
 
@@ -199,18 +199,18 @@ static bool se_priv_empty( const SE_PRIV *mask )
 {
        SE_PRIV p1;
        int i;
-       
+
        se_priv_copy( &p1, mask );
 
        for ( i=0; i<SE_PRIV_MASKSIZE; i++ ) {
                p1.mask[i] &= se_priv_all.mask[i];
        }
-       
+
        return se_priv_equal( &p1, &se_priv_none );
 }
 
 /*********************************************************************
- Lookup the SE_PRIV value for a privilege name 
+ Lookup the SE_PRIV value for a privilege name
 *********************************************************************/
 
 bool se_priv_from_name( const char *name, SE_PRIV *mask )
@@ -234,13 +234,13 @@ bool se_priv_from_name( const char *name, SE_PRIV *mask )
 void dump_se_priv( int dbg_cl, int dbg_lvl, const SE_PRIV *mask )
 {
        int i;
-       
+
        DEBUGADDC( dbg_cl, dbg_lvl,("SE_PRIV "));
-       
+
        for ( i=0; i<SE_PRIV_MASKSIZE; i++ ) {
                DEBUGADDC( dbg_cl, dbg_lvl,(" 0x%x", mask->mask[i] ));
        }
-               
+
        DEBUGADDC( dbg_cl, dbg_lvl, ("\n"));
 }
 
@@ -255,24 +255,24 @@ bool is_privilege_assigned(const SE_PRIV *privileges,
 
        if ( !privileges || !check )
                return False;
-       
+
        /* everyone has privileges if you aren't checking for any */
-       
+
        if ( se_priv_empty( check ) ) {
                DEBUG(1,("is_privilege_assigned: no privileges in check_mask!\n"));
                return True;
        }
-       
+
        se_priv_copy( &p1, check );
-       
-       /* invert the SE_PRIV we want to check for and remove that from the 
-          original set.  If we are left with the SE_PRIV we are checking 
+
+       /* invert the SE_PRIV we want to check for and remove that from the
+          original set.  If we are left with the SE_PRIV we are checking
           for then return True */
-          
+
        se_priv_invert( &p1, check );
        se_priv_copy( &p2, privileges );
        se_priv_remove( &p2, &p1 );
-       
+
        return se_priv_equal( &p2, check );
 }
 
@@ -286,26 +286,26 @@ static bool is_any_privilege_assigned( SE_PRIV *privileges, const SE_PRIV *check
 
        if ( !privileges || !check )
                return False;
-       
+
        /* everyone has privileges if you aren't checking for any */
-       
+
        if ( se_priv_empty( check ) ) {
                DEBUG(1,("is_any_privilege_assigned: no privileges in check_mask!\n"));
                return True;
        }
-       
+
        se_priv_copy( &p1, check );
-       
-       /* invert the SE_PRIV we want to check for and remove that from the 
-          original set.  If we are left with the SE_PRIV we are checking 
+
+       /* invert the SE_PRIV we want to check for and remove that from the
+          original set.  If we are left with the SE_PRIV we are checking
           for then return True */
-          
+
        se_priv_invert( &p1, check );
        se_priv_copy( &p2, privileges );
        se_priv_remove( &p2, &p1 );
-       
+
        /* see if we have any bits left */
-       
+
        return !se_priv_empty( &p2 );
 }
 
@@ -322,7 +322,7 @@ const char* get_privilege_dispname( const char *name )
        }
 
        for ( i=0; !se_priv_equal(&privs[i].se_priv, &se_priv_end); i++ ) {
-       
+
                if ( strequal( privs[i].name, name ) ) {
                        return privs[i].description;
                }
@@ -332,9 +332,9 @@ const char* get_privilege_dispname( const char *name )
 }
 
 /****************************************************************************
- initialise a privilege list and set the talloc context 
+ initialise a privilege list and set the talloc context
  ****************************************************************************/
+
 /****************************************************************************
  Does the user have the specified privilege ?  We only deal with one privilege
  at a time here.
@@ -377,7 +377,7 @@ int count_all_privileges( void )
 /*********************************************************************
  Generate the LUID_ATTR structure based on a bitmask
  The assumption here is that the privilege has already been validated
- so we are guaranteed to find it in the list. 
+ so we are guaranteed to find it in the list.
 *********************************************************************/
 
 LUID_ATTR get_privilege_luid( SE_PRIV *mask )
@@ -386,9 +386,9 @@ LUID_ATTR get_privilege_luid( SE_PRIV *mask )
        int i;
 
        ZERO_STRUCT( priv_luid );
-       
+
        for ( i=0; !se_priv_equal(&privs[i].se_priv, &se_priv_end); i++ ) {
-       
+
                if ( se_priv_equal( &privs[i].se_priv, mask ) ) {
                        priv_luid.luid = privs[i].luid;
                        break;
@@ -414,7 +414,7 @@ const char *luid_to_privilege_name(const LUID *set)
                        return privs[i].name;
                }
        }
-       
+
        return NULL;
 }
 
@@ -433,7 +433,7 @@ static bool privilege_set_add(PRIVILEGE_SET *priv_set, LUID_ATTR set)
        if ( !new_set ) {
                DEBUG(0,("privilege_set_add: failed to allocate memory!\n"));
                return False;
-       }       
+       }
 
        new_set[priv_set->count].luid.high = set.luid.high;
        new_set[priv_set->count].luid.low = set.luid.low;
@@ -453,16 +453,16 @@ bool se_priv_to_privilege_set( PRIVILEGE_SET *set, SE_PRIV *mask )
        int i;
        uint32 num_privs = count_all_privileges();
        LUID_ATTR luid;
-       
+
        luid.attr = 0;
        luid.luid.high = 0;
-       
+
        for ( i=0; i<num_privs; i++ ) {
                if ( !is_privilege_assigned(mask, &privs[i].se_priv) )
                        continue;
-               
+
                luid.luid = privs[i].luid;
-               
+
                if ( !privilege_set_add( set, luid ) )
                        return False;
        }
@@ -477,7 +477,7 @@ static bool luid_to_se_priv( struct lsa_LUID *luid, SE_PRIV *mask )
 {
        int i;
        uint32 num_privs = count_all_privileges();
-       
+
        for ( i=0; i<num_privs; i++ ) {
                if ( luid->low == privs[i].luid.low ) {
                        se_priv_copy( mask, &privs[i].se_priv );
@@ -494,19 +494,19 @@ static bool luid_to_se_priv( struct lsa_LUID *luid, SE_PRIV *mask )
 bool privilege_set_to_se_priv( SE_PRIV *mask, struct lsa_PrivilegeSet *privset )
 {
        int i;
-       
+
        ZERO_STRUCTP( mask );
-       
+
        for ( i=0; i<privset->count; i++ ) {
                SE_PRIV r;
-       
+
                /* sanity check for invalid privilege.  we really
                   only care about the low 32 bits */
-                  
+
                if ( privset->set[i].luid.high != 0 )
                        return False;
-               
-               if ( luid_to_se_priv( &privset->set[i].luid, &r ) )             
+
+               if ( luid_to_se_priv( &privset->set[i].luid, &r ) )
                        se_priv_add( mask, &r );
        }