[SSSD] [PATCH] Ability to set a domain as case sensitive or insensitive

Stephen Gallagher sgallagh at redhat.com
Wed Dec 14 18:37:47 UTC 2011


On Wed, 2011-12-14 at 14:15 +0100, Jakub Hrozek wrote:
> Thank you for the review. New patches are attached.

Patch 0008:

+static bool string_equal(bool cs, const char *s1, const char *s2)
+{
+    if (cs) {
+        return strcasecmp(s1, s2) == 0;
+    }
+
+    return sss_utf8_case_eq((const uint8_t *)s1, (const uint8_t *)s2)
== EOK;
+}
+


I'm pretty sure you wanted:


+    if (cs) {
+        return strcmp(s1, s2) == 0;
+    }

not strcasecmp() there.


Everything else looks good.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 198 bytes
Desc: This is a digitally signed message part
URL: <https://lists.fedorahosted.org/pipermail/sssd-devel/attachments/20111214/5b1aeff9/attachment.sig>


More information about the sssd-devel mailing list