#include "portable.h"
-#ifdef SLAPD_SQL
-
#include <stdio.h>
#include <sys/types.h>
#include "ac/string.h"
static backsql_api *backsqlapi;
int
-backsql_api_config( backsql_info *si, const char *name )
+backsql_api_config( backsql_info *bi, const char *name )
{
backsql_api *ba;
- assert( si );
+ assert( bi );
assert( name );
for ( ba = backsqlapi; ba; ba = ba->ba_next ) {
ba2 = ch_malloc( sizeof( backsql_api ) );
*ba2 = *ba;
- ba2->ba_next = si->si_api;
- si->si_api = ba2;
+ ba2->ba_next = bi->sql_api;
+ bi->sql_api = ba2;
return 0;
}
}
for ( ba2 = backsqlapi; ba2; ba2 = ba2->ba_next ) {
if ( strcasecmp( ba->ba_name, ba2->ba_name ) == 0 ) {
fprintf( stderr, "API module \"%s\" already defined\n", ba->ba_name );
- exit(EXIT_FAILURE);
+ exit( EXIT_FAILURE );
}
}
int
backsql_api_dn2odbc( Operation *op, SlapReply *rs, struct berval *dn )
{
- backsql_info *si = (backsql_info *)op->o_bd->be_private;
+ backsql_info *bi = (backsql_info *)op->o_bd->be_private;
backsql_api *ba;
int rc;
struct berval bv;
- ba = si->si_api;
+ ba = bi->sql_api;
if ( ba == NULL ) {
return 0;
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;
int
backsql_api_odbc2dn( Operation *op, SlapReply *rs, struct berval *dn )
{
- backsql_info *si = (backsql_info *)op->o_bd->be_private;
+ backsql_info *bi = (backsql_info *)op->o_bd->be_private;
backsql_api *ba;
int rc;
struct berval bv;
- ba = si->si_api;
+ ba = bi->sql_api;
if ( ba == NULL ) {
return 0;
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;
}
-#endif /* SLAPD_SQL */
-