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