[Fedora-directory-commits] ldapserver/ldap/servers/plugins/collation collate.c, 1.6, 1.7

[Pete Rowley (prowley)]

Author: prowley

Update of /cvs/dirsec/ldapserver/ldap/servers/plugins/collation
In directory cvs-int.fedora.redhat.com:/tmp/cvs-serv25844

Modified Files:
	collate.c 
Log Message:
173375: Problems with schema matching rules


Index: collate.c
===================================================================
RCS file: /cvs/dirsec/ldapserver/ldap/servers/plugins/collation/collate.c,v
retrieving revision 1.6
retrieving revision 1.7
diff -u -r1.6 -r1.7
--- collate.c	19 Apr 2005 22:07:29 -0000	1.6
+++ collate.c	6 Jan 2006 21:49:53 -0000	1.7
@@ -132,19 +132,34 @@
                     strcpy(nameOrder,"caseExactOrderingMatch");
                     strcpy(nameSubstring,"caseExactSubstringMatch");
                 }
- 
+
+		/* PAR: this looks broken
+                   the "extra" text based oids that are actually used
+                   to form the name and description are always derived
+                   from the language and country fields so there should
+                   be no need to have two separate code paths to
+                   set the name and description fields of the schema
+                   as language is always available, and if country is
+                   not, it is not in the name anyway.
+
+                   Is it safe to assume all matching rules will follow
+                   this convention?  The answer, or lack of it, probably
+                   explains the reasoning for doing things the way they
+                   are currently.
+                */ 
+                    
                 if(cargc > 7) {
-                    strcpy(nameOrder,"-");
+                    PL_strcatn(nameOrder,sizeof(nameOrder),"-");
                     PL_strcatn(nameOrder,sizeof(nameOrder),cargv[7]);
-                    strcpy(nameSubstring,"-");
+                    PL_strcatn(nameSubstring,sizeof(nameSubstring),"-");
                     PL_strcatn(nameSubstring,sizeof(nameSubstring),cargv[7]);
                     slapi_matchingrule_set(mrentry,SLAPI_MATCHINGRULE_NAME,
                                            (void *)slapi_ch_strdup(nameOrder));
                 }
-                else {
+                else  {
                     if(0 != cargv[1][0]) {
-                        strcpy(nameOrder,"-");
-                        strcpy(nameSubstring,"-");
+                        PL_strcatn(nameOrder,sizeof(nameOrder),"-");
+                        PL_strcatn(nameSubstring,sizeof(nameSubstring),"-");
                     } else {
 						nameOrder[0] = 0;
 						nameSubstring[0] = 0;