ptloader segfaulting while looking up LDAP groups
Patrick Radtke
phr2101 at columbia.edu
Wed Aug 9 16:46:41 EDT 2006
>
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
More information about the Info-cyrus
mailing list