s4:dsdb - use the more safe "samdb_msg_add_(u)int*" calls always where possible

[metze/samba/wip.git] / source4 / dsdb / samdb / ldb_modules / rootdse.c

diff --git a/source4/dsdb/samdb/ldb_modules/rootdse.c b/source4/dsdb/samdb/ldb_modules/rootdse.c
index 5c6090fc68f483663540a05982c7e173f47c0546..b986f778714fad12d3211898a89c5311690370c3 100644 (file)
--- a/source4/dsdb/samdb/ldb_modules/rootdse.c
+++ b/source4/dsdb/samdb/ldb_modules/rootdse.c
@@ -373,23 +373,24 @@ static int rootdse_add_dynamic(struct ldb_module *module, struct ldb_message *ms
        }
 
        if (do_attribute(attrs, "domainFunctionality")) {
-               if (ldb_msg_add_fmt(msg, "domainFunctionality",
-                                   "%d", dsdb_functional_level(ldb)) != LDB_SUCCESS) {
+               if (samdb_msg_add_int(ldb, msg, msg, "domainFunctionality",
+                                     dsdb_functional_level(ldb)) != LDB_SUCCESS) {
                        goto failed;
                }
        }
 
        if (do_attribute(attrs, "forestFunctionality")) {
-               if (ldb_msg_add_fmt(msg, "forestFunctionality",
-                                   "%d", dsdb_forest_functional_level(ldb)) != LDB_SUCCESS) {
+               if (samdb_msg_add_int(ldb, msg, msg, "forestFunctionality",
+                                     dsdb_forest_functional_level(ldb)) != LDB_SUCCESS) {
                        goto failed;
                }
        }
 
        if (do_attribute(attrs, "domainControllerFunctionality")
            && (val = talloc_get_type(ldb_get_opaque(ldb, "domainControllerFunctionality"), int))) {
-               if (ldb_msg_add_fmt(msg, "domainControllerFunctionality",
-                                   "%d", *val) != LDB_SUCCESS) {
+               if (samdb_msg_add_int(ldb, msg, msg,
+                                     "domainControllerFunctionality",
+                                     *val) != LDB_SUCCESS) {
                        goto failed;
                }
        }