diff options
author | Lennart Poettering <lennart@poettering.net> | 2017-02-13 20:44:11 +0100 |
---|---|---|
committer | Lennart Poettering <lennart@poettering.net> | 2017-02-13 20:44:11 +0100 |
commit | 1a63fc5430d66ccd72b0dbe3f6709724a811df40 (patch) | |
tree | 425434c200abc98c8764f30eb35140ed2b810d2e /src/resolve/resolved-dns-transaction.c | |
parent | bceaa99d49538fc24151abed9f9d40c950f626a2 (diff) |
resolved: let's propagate errors from dns_scope_announce() and elsewhere
We don't actually make use of the return value for now, but it matches
our coding style elsewhere, and it actually shortens our code quite a
bit.
Also, add a missing OOM check after dns_answer_new().
Diffstat (limited to 'src/resolve/resolved-dns-transaction.c')
-rw-r--r-- | src/resolve/resolved-dns-transaction.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/resolve/resolved-dns-transaction.c b/src/resolve/resolved-dns-transaction.c index 0c7a8867fb..69e1d58ccf 100644 --- a/src/resolve/resolved-dns-transaction.c +++ b/src/resolve/resolved-dns-transaction.c @@ -364,7 +364,7 @@ void dns_transaction_complete(DnsTransaction *t, DnsTransactionState state) { dns_zone_item_notify(z); SWAP_TWO(t->notify_zone_items, t->notify_zone_items_done); if (t->probing) - dns_scope_announce(t->scope, false); + (void) dns_scope_announce(t->scope, false); SET_FOREACH_MOVE(d, t->notify_transactions_done, t->notify_transactions) dns_transaction_notify(d, t); |