registry: check for existence of key in regdb_get_secdesc() before proceeding.
[kai/samba.git] / source3 / registry / reg_objects.c
index 9cfeb7faa97baf29b61c973a2af613295262e246..47122ccad2a20739e0c1483403eed89300b7e099 100644 (file)
@@ -1,21 +1,20 @@
-/* 
+/*
  *  Unix SMB/CIFS implementation.
- *  RPC Pipe client / server routines
- *  Copyright (C) Gerald Carter                     2002.
+ *  Virtual Windows Registry Layer
+ *  Copyright (C) Gerald Carter                     2002-2005
  *
  *  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 2 of the License, or
+ *  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, write to the Free Software
- *  Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
+ *  along with this program; if not, see <http://www.gnu.org/licenses/>.
  */
 
 /* Implementation of registry frontend view functions. */
 #include "includes.h"
 
 #undef DBGC_CLASS
-#define DBGC_CLASS DBGC_RPC_SRV
+#define DBGC_CLASS DBGC_REGISTRY
+
+/**********************************************************************
+
+ Note that the REGSUB_CTR and REGVAL_CTR objects *must* be talloc()'d
+ since the methods use the object pointer as the talloc context for
+ internal private data.
 
+ There is no longer a regXXX_ctr_intit() and regXXX_ctr_destroy()
+ pair of functions.  Simply TALLOC_ZERO_P() and TALLOC_FREE() the
+ object.
+
+ **********************************************************************/
 
 /***********************************************************************
- Init the talloc context held by a REGSUBKEY_CTR structure
+ Add a new key to the array
  **********************************************************************/
 
-void regsubkey_ctr_init( REGSUBKEY_CTR *ctr )
+WERROR regsubkey_ctr_addkey( REGSUBKEY_CTR *ctr, const char *keyname )
 {
-       if ( !ctr->ctx )
-               ctr->ctx = talloc_init("regsubkey_ctr_init for ctr %p", ctr);
+       char **newkeys;
+
+       if ( !keyname ) {
+               return WERR_OK;
+       }
+
+       /* make sure the keyname is not already there */
+
+       if ( regsubkey_ctr_key_exists( ctr, keyname ) ) {
+               return WERR_OK;
+       }
+
+       if (!(newkeys = TALLOC_REALLOC_ARRAY(ctr, ctr->subkeys, char *,
+                                            ctr->num_subkeys+1))) {
+               return WERR_NOMEM;
+       }
+
+       ctr->subkeys = newkeys;
+
+       if (!(ctr->subkeys[ctr->num_subkeys] = talloc_strdup(ctr->subkeys,
+                                                            keyname ))) {
+               /*
+                * Don't shrink the new array again, this wastes a pointer
+                */
+               return WERR_NOMEM;
+       }
+       ctr->num_subkeys++;
+
+       return WERR_OK;
 }
 
-/***********************************************************************
Add a new key to the array
+ /***********************************************************************
Delete a key from the array
  **********************************************************************/
 
-int regsubkey_ctr_addkey( REGSUBKEY_CTR *ctr, const char *keyname )
+int regsubkey_ctr_delkey( REGSUBKEY_CTR *ctr, const char *keyname )
 {
-       uint32 len;
-       char **pp;
-       
-       if ( keyname )
-       {
-               len = strlen( keyname );
-
-               /* allocate a space for the char* in the array */
-               
-               if (  ctr->subkeys == 0 )
-                       ctr->subkeys = talloc( ctr->ctx, sizeof(char*) );
-               else {
-                       pp = talloc_realloc( ctr->ctx, ctr->subkeys, sizeof(char*)*(ctr->num_subkeys+1) );
-                       if ( pp )
-                               ctr->subkeys = pp;
-               }
+       int i;
 
-               /* allocate the string and save it in the array */
-               
-               ctr->subkeys[ctr->num_subkeys] = talloc( ctr->ctx, len+1 );
-               strncpy( ctr->subkeys[ctr->num_subkeys], keyname, len+1 );
-               ctr->num_subkeys++;
+       if ( !keyname )
+               return ctr->num_subkeys;
+
+       /* make sure the keyname is actually already there */
+
+       for ( i=0; i<ctr->num_subkeys; i++ ) {
+               if ( strequal( ctr->subkeys[i], keyname ) )
+                       break;
        }
-       
+
+       if ( i == ctr->num_subkeys )
+               return ctr->num_subkeys;
+
+       /* update if we have any keys left */
+       ctr->num_subkeys--;
+       if ( i < ctr->num_subkeys )
+               memmove(&ctr->subkeys[i], &ctr->subkeys[i+1],
+                       sizeof(char*) * (ctr->num_subkeys-i));
+
        return ctr->num_subkeys;
 }
+
+/***********************************************************************
+ Check for the existance of a key
+ **********************************************************************/
+
+bool regsubkey_ctr_key_exists( REGSUBKEY_CTR *ctr, const char *keyname )
+{
+       int     i;
+
+       if (!ctr->subkeys) {
+               return False;
+       }
+
+       for ( i=0; i<ctr->num_subkeys; i++ ) {
+               if ( strequal( ctr->subkeys[i],keyname ) )
+                       return True;
+       }
+
+       return False;
+}
+
 /***********************************************************************
  How many keys does the container hold ?
  **********************************************************************/
@@ -86,37 +140,14 @@ char* regsubkey_ctr_specific_key( REGSUBKEY_CTR *ctr, uint32 key_index )
 {
        if ( ! (key_index < ctr->num_subkeys) )
                return NULL;
-               
-       return ctr->subkeys[key_index];
-}
-
-/***********************************************************************
- free memory held by a REGSUBKEY_CTR structure
- **********************************************************************/
 
-void regsubkey_ctr_destroy( REGSUBKEY_CTR *ctr )
-{
-       if ( ctr ) {
-               talloc_destroy( ctr->ctx );     
-               ZERO_STRUCTP( ctr );
-       }
+       return ctr->subkeys[key_index];
 }
 
-
 /*
  * Utility functions for REGVAL_CTR
  */
 
-/***********************************************************************
- Init the talloc context held by a REGSUBKEY_CTR structure
- **********************************************************************/
-
-void regval_ctr_init( REGVAL_CTR *ctr )
-{
-       if ( !ctr->ctx )
-               ctr->ctx = talloc_init("regval_ctr_init for ctr %p", ctr);
-}
-
 /***********************************************************************
  How many keys does the container hold ?
  **********************************************************************/
@@ -134,42 +165,49 @@ int regval_ctr_numvals( REGVAL_CTR *ctr )
 REGISTRY_VALUE* dup_registry_value( REGISTRY_VALUE *val )
 {
        REGISTRY_VALUE  *copy = NULL;
-       
+
        if ( !val )
                return NULL;
-       
-       if ( !(copy = malloc( sizeof(REGISTRY_VALUE) )) ) {
+
+       if ( !(copy = SMB_MALLOC_P( REGISTRY_VALUE)) ) {
                DEBUG(0,("dup_registry_value: malloc() failed!\n"));
                return NULL;
        }
-       
+
        /* copy all the non-pointer initial data */
-       
+
        memcpy( copy, val, sizeof(REGISTRY_VALUE) );
-       if ( val->data_p ) 
+
+       copy->size = 0;
+       copy->data_p = NULL;
+
+       if ( val->data_p && val->size )
        {
-               if ( !(copy->data_p = memdup( val->data_p, val->size )) ) {
-                       DEBUG(0,("dup_registry_value: memdup() failed for [%d] bytes!\n",
-                               val->size));
+               if ( !(copy->data_p = (uint8 *)memdup( val->data_p,
+                                                      val->size )) ) {
+                       DEBUG(0,("dup_registry_value: memdup() failed for [%d] "
+                                "bytes!\n", val->size));
                        SAFE_FREE( copy );
+                       return NULL;
                }
+               copy->size = val->size;
        }
-       
-       return copy;    
+
+       return copy;
 }
 
 /**********************************************************************
- free the memory allocated to a REGISTRY_VALUE 
+ free the memory allocated to a REGISTRY_VALUE
  *********************************************************************/
+
 void free_registry_value( REGISTRY_VALUE *val )
 {
        if ( !val )
                return;
-               
+
        SAFE_FREE( val->data_p );
        SAFE_FREE( val );
-       
+
        return;
 }
 
@@ -184,7 +222,7 @@ uint8* regval_data_p( REGISTRY_VALUE *val )
 /**********************************************************************
  *********************************************************************/
 
-int regval_size( REGISTRY_VALUE *val )
+uint32 regval_size( REGISTRY_VALUE *val )
 {
        return val->size;
 }
@@ -207,63 +245,101 @@ uint32 regval_type( REGISTRY_VALUE *val )
 
 /***********************************************************************
  Retreive a pointer to a specific value.  Caller shoud dup the structure
- since this memory may go away with a regval_ctr_destroy()
+ since this memory will go away when the ctr is free()'d
  **********************************************************************/
 
 REGISTRY_VALUE* regval_ctr_specific_value( REGVAL_CTR *ctr, uint32 idx )
 {
        if ( !(idx < ctr->num_values) )
                return NULL;
-               
+
        return ctr->values[idx];
 }
 
 /***********************************************************************
- Retrive the TALLOC_CTX associated with a REGISTRY_VALUE 
+ Check for the existance of a value
  **********************************************************************/
 
-TALLOC_CTX* regval_ctr_getctx( REGVAL_CTR *val )
+bool regval_ctr_key_exists( REGVAL_CTR *ctr, const char *value )
 {
-       if ( !val )
+       int     i;
+
+       for ( i=0; i<ctr->num_values; i++ ) {
+               if ( strequal( ctr->values[i]->valuename, value) )
+                       return True;
+       }
+
+       return False;
+}
+
+/***********************************************************************
+ * compose a REGISTRY_VALUE from input data
+ **********************************************************************/
+
+REGISTRY_VALUE *regval_compose(TALLOC_CTX *ctx, const char *name, uint16 type,
+                              const char *data_p, size_t size)
+{
+       REGISTRY_VALUE *regval = TALLOC_P(ctx, REGISTRY_VALUE);
+
+       if (regval == NULL) {
                return NULL;
+       }
 
-       return val->ctx;
+       fstrcpy(regval->valuename, name);
+       regval->type = type;
+       if (size) {
+               regval->data_p = (uint8 *)TALLOC_MEMDUP(regval, data_p, size);
+               if (!regval->data_p) {
+                       TALLOC_FREE(regval);
+                       return NULL;
+               }
+       } else {
+               regval->data_p = NULL;
+       }
+       regval->size = size;
+
+       return regval;
 }
 
 /***********************************************************************
  Add a new registry value to the array
  **********************************************************************/
 
-int regval_ctr_addvalue( REGVAL_CTR *ctr, const char *name, uint16 type, 
+int regval_ctr_addvalue( REGVAL_CTR *ctr, const char *name, uint16 type,
                          const char *data_p, size_t size )
 {
-       REGISTRY_VALUE **ppreg;
-       
-       if ( name )
-       {
-               /* allocate a slot in the array of pointers */
-               
-               if (  ctr->num_values == 0 )
-                       ctr->values = talloc( ctr->ctx, sizeof(REGISTRY_VALUE*) );
-               else {
-                       ppreg = talloc_realloc( ctr->ctx, ctr->values, sizeof(REGISTRY_VALUE*)*(ctr->num_values+1) );
-                       if ( ppreg )
-                               ctr->values = ppreg;
-               }
+       if ( !name )
+               return ctr->num_values;
+
+       /* Delete the current value (if it exists) and add the new one */
+
+       regval_ctr_delvalue( ctr, name );
+
+       /* allocate a slot in the array of pointers */
 
-               /* allocate a new value and store the pointer in the arrya */
-               
-               ctr->values[ctr->num_values] = talloc( ctr->ctx, sizeof(REGISTRY_VALUE) );
-
-               /* init the value */
-       
-               fstrcpy( ctr->values[ctr->num_values]->valuename, name );
-               ctr->values[ctr->num_values]->type = type;
-               ctr->values[ctr->num_values]->data_p = talloc_memdup( ctr->ctx, data_p, size );
-               ctr->values[ctr->num_values]->size = size;
-               ctr->num_values++;
+       if (  ctr->num_values == 0 ) {
+               ctr->values = TALLOC_P( ctr, REGISTRY_VALUE *);
+       } else {
+               ctr->values = TALLOC_REALLOC_ARRAY(ctr, ctr->values,
+                                                  REGISTRY_VALUE *,
+                                                  ctr->num_values+1);
        }
 
+       if (!ctr->values) {
+               ctr->num_values = 0;
+               return 0;
+       }
+
+       /* allocate a new value and store the pointer in the arrya */
+
+       ctr->values[ctr->num_values] = regval_compose(ctr, name, type, data_p,
+                                                     size);
+       if (ctr->values[ctr->num_values] == NULL) {
+               ctr->num_values = 0;
+               return 0;
+       }
+       ctr->num_values++;
+
        return ctr->num_values;
 }
 
@@ -273,31 +349,9 @@ int regval_ctr_addvalue( REGVAL_CTR *ctr, const char *name, uint16 type,
 
 int regval_ctr_copyvalue( REGVAL_CTR *ctr, REGISTRY_VALUE *val )
 {
-       REGISTRY_VALUE **ppreg;
-       
-       if ( val )
-       {
-               /* allocate a slot in the array of pointers */
-               
-               if (  ctr->num_values == 0 )
-                       ctr->values = talloc( ctr->ctx, sizeof(REGISTRY_VALUE*) );
-               else {
-                       ppreg = talloc_realloc( ctr->ctx, ctr->values, sizeof(REGISTRY_VALUE*)*(ctr->num_values+1) );
-                       if ( ppreg )
-                               ctr->values = ppreg;
-               }
-
-               /* allocate a new value and store the pointer in the arrya */
-               
-               ctr->values[ctr->num_values] = talloc( ctr->ctx, sizeof(REGISTRY_VALUE) );
-
-               /* init the value */
-       
-               fstrcpy( ctr->values[ctr->num_values]->valuename, val->valuename );
-               ctr->values[ctr->num_values]->type = val->type;
-               ctr->values[ctr->num_values]->data_p = talloc_memdup( ctr->ctx, val->data_p, val->size );
-               ctr->values[ctr->num_values]->size = val->size;
-               ctr->num_values++;
+       if ( val ) {
+               regval_ctr_addvalue(ctr, val->valuename, val->type,
+                                   (char *)val->data_p, val->size);
        }
 
        return ctr->num_values;
@@ -311,64 +365,67 @@ int regval_ctr_copyvalue( REGVAL_CTR *ctr, REGISTRY_VALUE *val )
 int regval_ctr_delvalue( REGVAL_CTR *ctr, const char *name )
 {
        int     i;
-       
-       /* search for the value */
-       if (!(ctr->num_values))
-               return 0;
-       
+
        for ( i=0; i<ctr->num_values; i++ ) {
-               if ( strcmp( ctr->values[i]->valuename, name ) == 0)
+               if ( strequal( ctr->values[i]->valuename, name ) )
                        break;
        }
-       
+
        /* just return if we don't find it */
-       
+
        if ( i == ctr->num_values )
                return ctr->num_values;
-       
-       /* just shift everything down one */
-       
-       for ( /* use previous i */; i<(ctr->num_values-1); i++ )
-               memcpy( ctr->values[i], ctr->values[i+1], sizeof(REGISTRY_VALUE) );
-               
-       /* paranoia */
-       
-       ZERO_STRUCTP( ctr->values[i] );
-       
+
+       /* If 'i' was not the last element, just shift everything down one */
        ctr->num_values--;
-       
+       if ( i < ctr->num_values )
+               memmove(&ctr->values[i], &ctr->values[i+1],
+                       sizeof(REGISTRY_VALUE*)*(ctr->num_values-i));
+
        return ctr->num_values;
 }
 
 /***********************************************************************
Delete a single value from the registry container.
Retrieve single value from the registry container.
  No need to free memory since it is talloc'd.
  **********************************************************************/
 
 REGISTRY_VALUE* regval_ctr_getvalue( REGVAL_CTR *ctr, const char *name )
 {
        int     i;
-       
+
        /* search for the value */
-       
+
        for ( i=0; i<ctr->num_values; i++ ) {
                if ( strequal( ctr->values[i]->valuename, name ) )
                        return ctr->values[i];
        }
-       
+
        return NULL;
 }
 
 /***********************************************************************
- free memory held by a REGVAL_CTR structure
+ return the data_p as a uint32
  **********************************************************************/
 
-void regval_ctr_destroy( REGVAL_CTR *ctr )
+uint32 regval_dword( REGISTRY_VALUE *val )
 {
-       if ( ctr ) {
-               talloc_destroy( ctr->ctx );
-               ZERO_STRUCTP( ctr );
-       }
+       uint32 data;
+
+       data = IVAL( regval_data_p(val), 0 );
+
+       return data;
 }
 
+/***********************************************************************
+ return the data_p as a character string
+ **********************************************************************/
 
+char *regval_sz(REGISTRY_VALUE *val)
+{
+       char *data = NULL;
+
+       rpcstr_pull_talloc(talloc_tos(), &data,
+                       regval_data_p(val), regval_size(val),0);
+       return data;
+}