diff options
author | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-10-24 19:30:12 +0200 |
---|---|---|
committer | Matthias Dieter Wallnöfer <mdw@samba.org> | 2010-10-24 17:57:06 +0000 |
commit | 49dee0e453049a2b26aaacf81e61a0f11afccd91 (patch) | |
tree | 67cc5d76f097367c1af1fa56a18b6816735f7cb9 /source4/dsdb/samdb/ldb_modules/util.c | |
parent | 482c02284068810a57b35a509857fb1273d833b0 (diff) | |
download | samba-49dee0e453049a2b26aaacf81e61a0f11afccd91.tar.gz samba-49dee0e453049a2b26aaacf81e61a0f11afccd91.tar.bz2 samba-49dee0e453049a2b26aaacf81e61a0f11afccd91.zip |
s4:dsdb - use the more safe "samdb_msg_add_(u)int*" calls always where possible
This should prevent all possible integer storage problems in future.
Diffstat (limited to 'source4/dsdb/samdb/ldb_modules/util.c')
-rw-r--r-- | source4/dsdb/samdb/ldb_modules/util.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/source4/dsdb/samdb/ldb_modules/util.c b/source4/dsdb/samdb/ldb_modules/util.c index 57066d9c88..14fccb9658 100644 --- a/source4/dsdb/samdb/ldb_modules/util.c +++ b/source4/dsdb/samdb/ldb_modules/util.c @@ -746,7 +746,7 @@ int dsdb_module_save_partition_usn(struct ldb_module *module, struct ldb_dn *dn, return ldb_module_oom(module); } - ret = ldb_msg_add_fmt(msg, "uSNHighest", "%llu", (unsigned long long)uSN); + ret = samdb_msg_add_uint64(ldb, msg, msg, "uSNHighest", uSN); if (ret != LDB_SUCCESS) { talloc_free(msg); return ret; @@ -755,7 +755,8 @@ int dsdb_module_save_partition_usn(struct ldb_module *module, struct ldb_dn *dn, /* urgent_uSN is optional so may not be stored */ if (urgent_uSN) { - ret = ldb_msg_add_fmt(msg, "uSNUrgent", "%llu", (unsigned long long)urgent_uSN); + ret = samdb_msg_add_uint64(ldb, msg, msg, "uSNUrgent", + urgent_uSN); if (ret != LDB_SUCCESS) { talloc_free(msg); return ret; |