Commit - SSSD/sssd - 49d84c926b00ba1368372cdec255bceb58d66f43

    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>

src/db/sysdb.c

file modified

+20 -2

src/db/sysdb_ops.c

file modified

+7 -11

src/db/sysdb_private.h

file modified

+6 -2

src/db/sysdb_ranges.c

file modified

+8 -10

src/db/sysdb_services.c

file modified

+3 -6

SSSD / sssd

Source Code

Documentation

49d84c9 sysdb: add sysdb_{add,replace,delete}_ulong()

5 files Authored by fidencio 7 years ago, Committed by jhrozek 7 years ago,

`49d84c9` sysdb: add sysdb_{add,replace,delete}_ulong()