From 7f63cebd331793d059b1dbfd2f7d7ce38105c4fe Mon Sep 17 00:00:00 2001 From: Andrew Tridgell Date: Wed, 13 Sep 2006 00:10:38 +0000 Subject: r18436: converted ldb to use talloc_move() instead of talloc_steal() when appropriate. Note that I also removed the error checks that were being done on the result of talloc_steal(). They are pointless as talloc_steal() doesn't have any failure modes that wouldn't cause a segv anyway, and they tend to clutter the code (This used to be commit c0d9e7d473b8e3eb2524a9fc29cf88680f994b36) --- source4/lib/ldb/tools/ldbsearch.c | 14 ++------------ 1 file changed, 2 insertions(+), 12 deletions(-) (limited to 'source4/lib/ldb/tools/ldbsearch.c') diff --git a/source4/lib/ldb/tools/ldbsearch.c b/source4/lib/ldb/tools/ldbsearch.c index e6b8a48a95..181d857efc 100644 --- a/source4/lib/ldb/tools/ldbsearch.c +++ b/source4/lib/ldb/tools/ldbsearch.c @@ -81,12 +81,7 @@ static int store_message(struct ldb_message *msg, struct search_context *sctx) { return -1; } - sctx->store[sctx->num_stored] = talloc_steal(sctx->store, msg); - if (!sctx->store[sctx->num_stored]) { - fprintf(stderr, "talloc_steal failed while storing messages\n"); - return -1; - } - + sctx->store[sctx->num_stored] = talloc_move(sctx->store, msg); sctx->num_stored++; sctx->store[sctx->num_stored] = NULL; @@ -101,12 +96,7 @@ static int store_referral(char *referral, struct search_context *sctx) { return -1; } - sctx->refs_store[sctx->refs] = talloc_steal(sctx->refs_store, referral); - if (!sctx->refs_store[sctx->refs]) { - fprintf(stderr, "talloc_steal failed while storing referrals\n"); - return -1; - } - + sctx->refs_store[sctx->refs] = talloc_move(sctx->refs_store, referral); sctx->refs++; sctx->refs_store[sctx->refs] = NULL; -- cgit