[Date Prev][Date Next] [Chronological] [Thread] [Top]

ber_bv[r]chr() return type (was: commit: ldap/servers/slapd/overlays valsort.c)

Instead of this change (and some others I was about to make),
maybe ber_bvchr() and ber_bvrchr() should be changed to return
char* instead of void*?  lutil_memchr() would still return void*.

hallvard@OpenLDAP.org writes:
> 	valsort.c  1.13 -> 1.14
> Fix void* pointer arithmetic from ber_bvchr()
-                   end = ber_bvchr( &a->a_vals[i], '}' ) + 1;
+                   end = (char *) ber_bvchr( &a->a_vals[i],