Attachment 575690 Details for Bug 809105 – An upstream patch fixing the problem. See comments for more information

[patch] An upstream patch fixing the problem. See comments for more information

openldap-use-strcast-in-where-conditions.patch (text/plain), 1.35 KB, created by Rene Snajder on 2012-04-06 10:00:01 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: Rene Snajder

Created: 2012-04-06 10:00:01 UTC

Size: 1.35 KB

patch

obsolete

>Cast the value in every UPPER to make it compatible with postgresql if "upper_needs_cast" is set.
>Before this patch strcast was ONLY applied to the "(ldap_entries.dn)" query in init.c. 
>Now it should also be applied everywhere where bam_sel_expr_u is used (that should cover all of the filters).
>
>This fixes a compatibility issue raised by SSSD 1.5.1_66.el6_2.3 when used with postgresql, where UPPER was applied on the os_user.gidnumber attribute, without a proper cast although upper_needs_cast was defined in the config.
>
>Rene Snajder
>April 6th 2012
>
>--- openldap-2.4.23/servers/slapd/back-sql/schema-map.c	2010-04-13 22:23:43.000000000 +0200
>+++ opeldap-2.4.23.new/servers/slapd/back-sql/schema-map.c	2012-04-06 11:06:26.997355184 +0200
>@@ -471,13 +471,15 @@
> 		{
> 			struct berbuf	bb = BB_NULL;
> 
>-			backsql_strfcat_x( &bb, NULL, "bcbc",
>-					&bas->bas_bi->sql_upper_func,
>-					'(' /* ) */ ,
>+                        backsql_strfcat_x( &bb, NULL, "bcbcb",
>+                                        &bas->bas_bi->sql_upper_func_open,
>+                                        ' ',
> 					&at_map->bam_sel_expr,
>-					/* ( */ ')' );
>+                                        ' ',
>+                                        &bas->bas_bi->sql_upper_func_close );
> 			at_map->bam_sel_expr_u = bb.bb_val;
> 		}
>+
> 	}
> 	backsql_FreeRow( &at_row );
> 	SQLFreeStmt( bas->bas_sth, SQL_CLOSE );
>
>

Actions: View | Diff

Attachments on bug 809105: 575690 | 577697