summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJakub Hrozek <jhrozek@redhat.com>2012-03-19 08:03:32 +0100
committerStephen Gallagher <sgallagh@redhat.com>2012-03-21 11:22:39 -0400
commit6d46dc4f1d46ba4a3c568e41b62b6832474cb0e8 (patch)
tree6361e54f22d1259ec5bd94e9822dbabd9f78a067
parentafde61e7e656abded3eb2f393cb8390ba013c0a6 (diff)
downloadsssd-6d46dc4f1d46ba4a3c568e41b62b6832474cb0e8.tar.gz
sssd-6d46dc4f1d46ba4a3c568e41b62b6832474cb0e8.tar.xz
sssd-6d46dc4f1d46ba4a3c568e41b62b6832474cb0e8.zip
Save alias of the primary name, too
-rw-r--r--src/db/sysdb.c12
-rw-r--r--src/responder/nss/nsssrv_services.c22
2 files changed, 23 insertions, 11 deletions
diff --git a/src/db/sysdb.c b/src/db/sysdb.c
index c29382fe7..d3b479aa2 100644
--- a/src/db/sysdb.c
+++ b/src/db/sysdb.c
@@ -1680,7 +1680,7 @@ errno_t sysdb_attrs_get_aliases(TALLOC_CTX *mem_ctx,
{
TALLOC_CTX *tmp_ctx = NULL;
struct ldb_message_element *sysdb_name_el;
- size_t i, ai;
+ size_t i, j, ai;
errno_t ret;
const char **aliases = NULL;
const char *name;
@@ -1725,8 +1725,14 @@ errno_t sysdb_attrs_get_aliases(TALLOC_CTX *mem_ctx,
goto done;
}
- if (sss_utf8_case_eq((const uint8_t *) primary,
- (const uint8_t *) lower) == ENOMATCH) {
+ for (j=0; j < ai; j++) {
+ if (sss_utf8_case_eq((const uint8_t *) aliases[j],
+ (const uint8_t *) lower) == ENOMATCH) {
+ break;
+ }
+ }
+
+ if (ai == 0 || j < ai) {
aliases[ai] = talloc_strdup(aliases, lower);
if (!aliases[ai]) {
ret = ENOMEM;
diff --git a/src/responder/nss/nsssrv_services.c b/src/responder/nss/nsssrv_services.c
index 872b0701f..d7ba53c9f 100644
--- a/src/responder/nss/nsssrv_services.c
+++ b/src/responder/nss/nsssrv_services.c
@@ -597,9 +597,9 @@ fill_service(struct sss_packet *packet,
{
errno_t ret;
unsigned int msg_count = *count;
- size_t rzero, rsize;
+ size_t rzero, rsize, aptr;
unsigned int num, i, j;
- uint32_t num_aliases;
+ uint32_t num_aliases, written_aliases;
struct ldb_message *msg;
struct ldb_message_element *el;
TALLOC_CTX *tmp_ctx = NULL;
@@ -711,8 +711,9 @@ fill_service(struct sss_packet *packet,
num_aliases = el->num_values;
}
- /* Store the alias count */
- SAFEALIGN_SET_UINT32(&body[rzero + rsize], num_aliases, &rsize);
+ /* We'll store the alias count here */
+ aptr = rzero+rsize;
+ rsize += sizeof(uint32_t);
/* Store the primary name */
safealign_memcpy(&body[rzero + rsize],
@@ -726,11 +727,14 @@ fill_service(struct sss_packet *packet,
cased_proto.len,
&rsize);
+ written_aliases = 0;
for (j = 0; j < num_aliases; j++) {
- tmpstr = sss_get_cased_name(tmp_ctx,
- (const char *)el->values[j].data,
- dom->case_sensitive);
- to_sized_string(&alias, tmpstr);
+ if (sss_string_equal(dom->case_sensitive,
+ (const char *)el->values[j].data,
+ cased_name.str)) {
+ continue;
+ }
+ to_sized_string(&alias, (const char *)el->values[j].data);
ret = sss_packet_grow(packet, alias.len);
if (ret != EOK) {
@@ -745,8 +749,10 @@ fill_service(struct sss_packet *packet,
alias.len,
&rsize);
+ written_aliases++;
talloc_zfree(tmpstr);
}
+ SAFEALIGN_SET_UINT32(&body[aptr], written_aliases, &rsize);
num++;
}