Re: (ITS#5779) backsql_load_schema_map checks wrong schema column in db

[ando@sys-net.it]

robb@wtg.cw.com wrote:
> Full_Name: Robert Brooks
> Version: 2.4.12
> OS: Linux
> URL: ftp://ftp.openldap.org/incoming/
> Submission from: (NULL) (212.28.17.64)
> 
> 
> start-up of slapd reports
> 
> "backsql_load_schema_map(): required column #6 "expect_return" is empty"
> 
> however the value of ldap_oc_mappings.create_proc is actually being checked,
> this can be confirmed by changing create_proc value in database from empty value
> to non-empty.
> 
> problem seems to be around line 567 of schema-map.c
> 
> "                                { delete_proc_idx + 1, "expect_return" },"
> 
> 6 seems correct value, but actual result is not correct

I have applied a blind fix to back-sql/schema-map.c; please apply the
patch between r1.67 & r1.68 and report (presented below as a reference).

p.

diff -u -r1.67 schema-map.c
--- servers/slapd/back-sql/schema-map.c	21 Aug 2008 20:07:29 -0000	1.67
+++ servers/slapd/back-sql/schema-map.c	30 Oct 2008 13:02:24 -0000
@@ -570,7 +570,7 @@
  			int i;

  			for ( i = 0; required[ i ].name != NULL; i++ ) {
-				if ( oc_row.value_len[ i ] <= 0 ) {
+				if ( oc_row.value_len[ required[ i ].idx ] <= 0 ) {
  					Debug( LDAP_DEBUG_ANY,
  						"backsql_load_schema_map(): "
  						"required column #%d \"%s\" is empty\n",




Ing. Pierangelo Masarati
OpenLDAP Core Team

SysNet s.r.l.
via Dossi, 8 - 27100 Pavia - ITALIA
http://www.sys-net.it
-----------------------------------
Office:  +39 02 23998309
Mobile:  +39 333 4963172
Fax:     +39 0382 476497
Email:   ando@sys-net.it
-----------------------------------