ldb_ldif: avoid strlen(NULL)
authorDouglas Bagnall <douglas.bagnall@catalyst.net.nz>
Thu, 18 Oct 2018 21:21:21 +0000 (10:21 +1300)
committerMartin Schwenke <martins@samba.org>
Fri, 19 Oct 2018 01:43:58 +0000 (03:43 +0200)
Signed-off-by: Douglas Bagnall <douglas.bagnall@catalyst.net.nz>
Reviewed-by: Martin Schwenke <martin@meltin.net>
Autobuild-User(master): Martin Schwenke <martins@samba.org>
Autobuild-Date(master): Fri Oct 19 03:43:58 CEST 2018 on sn-devel-144

lib/ldb/common/ldb_ldif.c

index ec1859b5c0d66dc6df96897d1486e1c551c7086e..e69467891c90eb4a1d3b7681f26629ba8a91f9b5 100644 (file)
@@ -313,7 +313,7 @@ static int ldb_ldif_write_trace(struct ldb_context *ldb,
 
        for (i=0;i<msg->num_elements;i++) {
                const struct ldb_schema_attribute *a;
-               size_t namelen = strlen(msg->elements[i].name);
+               size_t namelen;
 
                if (msg->elements[i].name == NULL) {
                        ldb_debug(ldb, LDB_DEBUG_ERROR,
@@ -322,6 +322,7 @@ static int ldb_ldif_write_trace(struct ldb_context *ldb,
                        return -1;
                }
 
+               namelen = strlen(msg->elements[i].name);
                a = ldb_schema_attribute_by_name(ldb, msg->elements[i].name);
 
                if (ldif->changetype == LDB_CHANGETYPE_MODIFY) {