diff options
author | Fabiano FidĂȘncio <fidencio@redhat.com> | 2016-05-25 23:00:59 +0200 |
---|---|---|
committer | Jakub Hrozek <jhrozek@redhat.com> | 2016-05-31 13:07:02 +0200 |
commit | 49d84c926b00ba1368372cdec255bceb58d66f43 (patch) | |
tree | c8bfeb778ebf55009204da5fabf49f11f12eb117 /src/db/sysdb_ranges.c | |
parent | 7b9a4a89cb92a0281d73a2c2e79f5eeb317e1149 (diff) | |
download | sssd-49d84c926b00ba1368372cdec255bceb58d66f43.tar.gz sssd-49d84c926b00ba1368372cdec255bceb58d66f43.tar.xz sssd-49d84c926b00ba1368372cdec255bceb58d66f43.zip |
sysdb: add sysdb_{add,replace,delete}_ulong()
As the add_ulong() convenience can add, replace or remove a unsigned
long according to the operation received as its argument, some confusion
can easily happen due to its misleading name.
In order to improve the explicitness of our code, let's introduce
sysdb_add_ulong(), sysdb_replace_ulong() and sysdb_delete_ulong().
These new functions are basically wrappers of add_ulong() (now
sysdb_ldb_msg_ulong_helper()), calling it using the proper flag
according to each function.
Any code previously using add_ulong() is now adapted to use these brand
new functions.
Related: https://fedorahosted.org/sssd/ticket/1656
Signed-off-by: Fabiano FidĂȘncio <fabiano@fidencio.org>
Reviewed-by: Petr Cech <pcech@redhat.com>
Diffstat (limited to 'src/db/sysdb_ranges.c')
-rw-r--r-- | src/db/sysdb_ranges.c | 18 |
1 files changed, 8 insertions, 10 deletions
diff --git a/src/db/sysdb_ranges.c b/src/db/sysdb_ranges.c index da599bf30..511e4785d 100644 --- a/src/db/sysdb_ranges.c +++ b/src/db/sysdb_ranges.c @@ -206,8 +206,8 @@ errno_t sysdb_range_create(struct sysdb_ctx *sysdb, struct range_info *range) SYSDB_DOMAIN_ID_RANGE_CLASS); if (ret) goto done; - ret = add_ulong(msg, LDB_FLAG_MOD_ADD, SYSDB_SECONDARY_BASE_RID, - (unsigned long) range->secondary_base_rid); + ret = sysdb_add_ulong(msg, SYSDB_SECONDARY_BASE_RID, + (unsigned long) range->secondary_base_rid); if (ret) goto done; } else if (range->trusted_dom_sid != NULL && range->secondary_base_rid == 0) { @@ -222,20 +222,18 @@ errno_t sysdb_range_create(struct sysdb_ctx *sysdb, struct range_info *range) ret = sysdb_add_string(msg, SYSDB_NAME, range->name); if (ret) goto done; - ret = add_ulong(msg, LDB_FLAG_MOD_ADD, SYSDB_BASE_ID, - (unsigned long) range->base_id); + ret = sysdb_add_ulong(msg, SYSDB_BASE_ID, (unsigned long) range->base_id); if (ret) goto done; - ret = add_ulong(msg, LDB_FLAG_MOD_ADD, SYSDB_ID_RANGE_SIZE, - (unsigned long) range->id_range_size); + ret = sysdb_add_ulong(msg, SYSDB_ID_RANGE_SIZE, + (unsigned long) range->id_range_size); if (ret) goto done; - ret = add_ulong(msg, LDB_FLAG_MOD_ADD, SYSDB_BASE_RID, - (unsigned long) range->base_rid); + ret = sysdb_add_ulong(msg, SYSDB_BASE_RID, + (unsigned long) range->base_rid); if (ret) goto done; - ret = add_ulong(msg, LDB_FLAG_MOD_ADD, SYSDB_CREATE_TIME, - (unsigned long)time(NULL)); + ret = sysdb_add_ulong(msg, SYSDB_CREATE_TIME, (unsigned long)time(NULL)); if (ret) goto done; ret = sysdb_add_string(msg, SYSDB_ID_RANGE_TYPE, range->range_type); |