mirror of
https://github.com/samba-team/samba.git
synced 2024-12-25 23:21:54 +03:00
r26574: Do not call functions in the variable declaration, fix checking for tmp_ctx,
and also makes code more readable and debuggable.
Eliminate silly parenthesys.
Simo.
(This used to be commit 166195b487
)
This commit is contained in:
parent
de9c8930a6
commit
2d400c1dd1
@ -961,16 +961,20 @@ static int ltdb_request(struct ldb_module *module, struct ldb_request *req)
|
||||
*/
|
||||
static int ltdb_sequence_number(struct ldb_module *module, struct ldb_request *req)
|
||||
{
|
||||
TALLOC_CTX *tmp_ctx = talloc_new(req);
|
||||
TALLOC_CTX *tmp_ctx;
|
||||
struct ldb_message *msg = NULL;
|
||||
struct ldb_dn *dn = ldb_dn_new(tmp_ctx, module->ldb, LTDB_BASEINFO);
|
||||
struct ldb_dn *dn;
|
||||
const char *date;
|
||||
int tret;
|
||||
|
||||
tmp_ctx = talloc_new(req);
|
||||
if (tmp_ctx == NULL) {
|
||||
talloc_free(tmp_ctx);
|
||||
return LDB_ERR_OPERATIONS_ERROR;
|
||||
}
|
||||
|
||||
dn = ldb_dn_new(tmp_ctx, module->ldb, LTDB_BASEINFO);
|
||||
|
||||
msg = talloc(tmp_ctx, struct ldb_message);
|
||||
if (msg == NULL) {
|
||||
talloc_free(tmp_ctx);
|
||||
@ -996,8 +1000,7 @@ static int ltdb_sequence_number(struct ldb_module *module, struct ldb_request *r
|
||||
req->op.seq_num.seq_num++;
|
||||
break;
|
||||
case LDB_SEQ_HIGHEST_TIMESTAMP:
|
||||
{
|
||||
const char *date = ldb_msg_find_attr_as_string(msg, LTDB_MOD_TIMESTAMP, NULL);
|
||||
date = ldb_msg_find_attr_as_string(msg, LTDB_MOD_TIMESTAMP, NULL);
|
||||
if (date) {
|
||||
req->op.seq_num.seq_num = ldb_string_to_time(date);
|
||||
} else {
|
||||
@ -1006,7 +1009,6 @@ static int ltdb_sequence_number(struct ldb_module *module, struct ldb_request *r
|
||||
}
|
||||
break;
|
||||
}
|
||||
}
|
||||
talloc_free(tmp_ctx);
|
||||
return LDB_SUCCESS;
|
||||
}
|
||||
|
Loading…
Reference in New Issue
Block a user