X-Git-Url: https://git.sur5r.net/?a=blobdiff_plain;f=servers%2Fslapd%2Fback-sql%2Fapi.c;h=6f87973d8a8f230a00c52c07a506a00c8f08c8ec;hb=5fcc9285fb8c549a5264921a2b61cfc40803d720;hp=1a1c32ab9368de151b744e13f24807e70771e550;hpb=396743f62f451f1d55df9d91ebbdaf21f075ec2c;p=openldap
diff --git a/servers/slapd/back-sql/api.c b/servers/slapd/back-sql/api.c
index 1a1c32ab93..6f87973d8a 100644
--- a/servers/slapd/back-sql/api.c
+++ b/servers/slapd/back-sql/api.c
@@ -1,7 +1,8 @@
/* This work is part of OpenLDAP Software .
*
- * Copyright 1999-2004 The OpenLDAP Foundation.
+ * Copyright 1999-2012 The OpenLDAP Foundation.
* Portions Copyright 1999 Dmitry Kovalev.
+ * Portions Copyright 2004 Pierangelo Masarati.
* All rights reserved.
*
* Redistribution and use in source and binary forms, with or without
@@ -14,7 +15,8 @@
*/
/* ACKNOWLEDGEMENTS:
* This work was initially developed by Dmitry Kovalev for inclusion
- * by OpenLDAP Software.
+ * by OpenLDAP Software. Additional significant contributors include
+ * Pierangelo Masarati.
*/
#include "portable.h"
@@ -29,12 +31,12 @@
static backsql_api *backsqlapi;
int
-backsql_api_config( backsql_info *bi, const char *name )
+backsql_api_config( backsql_info *bi, const char *name, int argc, char *argv[] )
{
backsql_api *ba;
- assert( bi );
- assert( name );
+ assert( bi != NULL );
+ assert( name != NULL );
for ( ba = backsqlapi; ba; ba = ba->ba_next ) {
if ( strcasecmp( name, ba->ba_name ) == 0 ) {
@@ -42,6 +44,21 @@ backsql_api_config( backsql_info *bi, const char *name )
ba2 = ch_malloc( sizeof( backsql_api ) );
*ba2 = *ba;
+
+ if ( ba2->ba_config ) {
+ if ( ( *ba2->ba_config )( ba2, argc, argv ) ) {
+ ch_free( ba2 );
+ return 1;
+ }
+ ba2->ba_argc = argc;
+ if ( argc ) {
+ int i;
+ ba2->ba_argv = ch_malloc( argc * sizeof(char *));
+ for ( i=0; iba_argv[i] = ch_strdup( argv[i] );
+ }
+ }
+
ba2->ba_next = bi->sql_api;
bi->sql_api = ba2;
return 0;
@@ -51,12 +68,35 @@ backsql_api_config( backsql_info *bi, const char *name )
return 1;
}
+int
+backsql_api_destroy( backsql_info *bi )
+{
+ backsql_api *ba;
+
+ assert( bi != NULL );
+
+ ba = bi->sql_api;
+
+ if ( ba == NULL ) {
+ return 0;
+ }
+
+ for ( ; ba; ba = ba->ba_next ) {
+ if ( ba->ba_destroy ) {
+ (void)( *ba->ba_destroy )( ba );
+ }
+ }
+
+ return 0;
+}
+
int
backsql_api_register( backsql_api *ba )
{
backsql_api *ba2;
- assert( ba );
+ assert( ba != NULL );
+ assert( ba->ba_private == NULL );
if ( ba->ba_name == NULL ) {
fprintf( stderr, "API module has no name\n" );
@@ -94,14 +134,30 @@ backsql_api_dn2odbc( Operation *op, SlapReply *rs, struct berval *dn )
for ( ; ba; ba = ba->ba_next ) {
if ( ba->ba_dn2odbc ) {
+ /*
+ * The dn2odbc() helper is supposed to rewrite
+ * the contents of bv, freeing the original value
+ * with ch_free() if required and replacing it
+ * with a newly allocated one using ch_malloc()
+ * or companion functions.
+ *
+ * NOTE: it is supposed to __always__ free
+ * the value of bv in case of error, and reset
+ * it with BER_BVZERO() .
+ */
rc = ( *ba->ba_dn2odbc )( op, rs, &bv );
if ( rc ) {
+ /* in case of error, dn2odbc() must cleanup */
+ assert( BER_BVISNULL( &bv ) );
+
return rc;
}
}
}
+ assert( !BER_BVISNULL( &bv ) );
+
*dn = bv;
return 0;
@@ -126,13 +182,28 @@ backsql_api_odbc2dn( Operation *op, SlapReply *rs, struct berval *dn )
for ( ; ba; ba = ba->ba_next ) {
if ( ba->ba_dn2odbc ) {
rc = ( *ba->ba_odbc2dn )( op, rs, &bv );
-
+ /*
+ * The odbc2dn() helper is supposed to rewrite
+ * the contents of bv, freeing the original value
+ * with ch_free() if required and replacing it
+ * with a newly allocated one using ch_malloc()
+ * or companion functions.
+ *
+ * NOTE: it is supposed to __always__ free
+ * the value of bv in case of error, and reset
+ * it with BER_BVZERO() .
+ */
if ( rc ) {
+ /* in case of error, odbc2dn() must cleanup */
+ assert( BER_BVISNULL( &bv ) );
+
return rc;
}
}
}
+ assert( !BER_BVISNULL( &bv ) );
+
*dn = bv;
return 0;