Re: ptloader segfaulting while looking up LDAP groups

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




Sorry, I must have butchered the original patch when I was pulling it out of a larger patch (that does some Columbia specific stuff).

Igor's patch makes it look like the correct version.

-Patrick


The last commit has an issue.

Please try this patch and report back:

--- ldap.c.orig	2006-08-09 14:42:05.023665000 -0400
+++ ldap.c	2006-08-09 14:42:41.274455000 -0400
@@ -1065,11 +1065,11 @@
                 continue;

             strcpy((*newstate)->groups[i].id, "group:");
+
 	    int j;
-	    strcpy((*newstate)->groups[i].id, "group:");
-	    for(j =0; j < strlen(vals[i]); j++) {
-	      if(isupper(vals[i][j]))
-		vals[i][j]=tolower(vals[i][j]);
+	    for(j =0; j < strlen(vals[0]); j++) {
+	      if(isupper(vals[0][j]))
+		vals[0][j]=tolower(vals[0][j]);
 	    }

             strlcat((*newstate)->groups[i].id, vals[0],

--
Igor
----
Cyrus Home Page: http://asg.web.cmu.edu/cyrus
Cyrus Wiki/FAQ: http://cyruswiki.andrew.cmu.edu
List Archives/Info: http://asg.web.cmu.edu/cyrus/mailing-list.html

----
Cyrus Home Page: http://asg.web.cmu.edu/cyrus
Cyrus Wiki/FAQ: http://cyruswiki.andrew.cmu.edu
List Archives/Info: http://asg.web.cmu.edu/cyrus/mailing-list.html

[Index of Archives]     [Cyrus SASL]     [Squirrel Mail]     [Asterisk PBX]     [Video For Linux]     [Photo]     [Yosemite News]     [gtk]     [KDE]     [Gimp on Windows]     [Steve's Art]

  Powered by Linux