diff options
author | Fabiano FidĂȘncio <fidencio@redhat.com> | 2016-05-25 20:16:58 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2016-05-31 13:06:38 +0200 |
commit | 5b1e73bc40a55f2095660423a2a4623a93de1ef8 (patch) | |
tree | 9e1f0748de97711f2879f2b5a9cf92831ca8a1ca | |
parent | 149174acae677d1e72a0da431bf0850d55f2ccb4 (diff) | |
download | sssd-5b1e73bc40a55f2095660423a2a4623a93de1ef8.tar.gz sssd-5b1e73bc40a55f2095660423a2a4623a93de1ef8.tar.xz sssd-5b1e73bc40a55f2095660423a2a4623a93de1ef8.zip |
sysdb: move add_string() convenience to sysdb.c
Considering that sysdb.c is about utilities around our cache and that
sysdb_ops.c is about operations on objects, seems that add_string()
could fit better in sysdb.c.
This move is a suggestion from Jakub Hrozek.
Related: https://fedorahosted.org/sssd/ticket/1656
Signed-off-by: Fabiano FidĂȘncio <fabiano@fidencio.org>
Reviewed-by: Petr Cech <pcech@redhat.com>
-rw-r--r-- | src/db/sysdb.c | 13 | ||||
-rw-r--r-- | src/db/sysdb_ops.c | 13 |
2 files changed, 13 insertions, 13 deletions
diff --git a/src/db/sysdb.c b/src/db/sysdb.c index d4366a3c7..f83d236c8 100644 --- a/src/db/sysdb.c +++ b/src/db/sysdb.c @@ -2111,3 +2111,16 @@ done: *_usn = usn; return EOK; } + +int add_string(struct ldb_message *msg, int flags, + const char *attr, const char *value) +{ + int ret; + + ret = ldb_msg_add_empty(msg, attr, flags, NULL); + if (ret == LDB_SUCCESS) { + ret = ldb_msg_add_string(msg, attr, value); + if (ret == LDB_SUCCESS) return EOK; + } + return ENOMEM; +} diff --git a/src/db/sysdb_ops.c b/src/db/sysdb_ops.c index 843251b3e..1a93956b0 100644 --- a/src/db/sysdb_ops.c +++ b/src/db/sysdb_ops.c @@ -27,19 +27,6 @@ #include "util/cert.h" #include <time.h> -int add_string(struct ldb_message *msg, int flags, - const char *attr, const char *value) -{ - int ret; - - ret = ldb_msg_add_empty(msg, attr, flags, NULL); - if (ret == LDB_SUCCESS) { - ret = ldb_msg_add_string(msg, attr, value); - if (ret == LDB_SUCCESS) return EOK; - } - return ENOMEM; -} - int add_ulong(struct ldb_message *msg, int flags, const char *attr, unsigned long value) { |