Call dns_adb_endudpfetch() on error path, if required
For UDP queries, after calling dns_adb_beginudpfetch() in fctx_query(),
make sure that dns_adb_endudpfetch() is also called on error path, in
order to adjust the quota back.
(cherry picked from commit 5da79e2be0)
This commit is contained in:
@@ -2279,7 +2279,7 @@ fctx_query(fetchctx_t *fctx, dns_adbaddrinfo_t *addrinfo,
|
||||
resquery_senddone, resquery_response, query, &query->id,
|
||||
&query->dispentry);
|
||||
if (result != ISC_R_SUCCESS) {
|
||||
goto cleanup_dispatch;
|
||||
goto cleanup_udpfetch;
|
||||
}
|
||||
|
||||
/* Connect the socket */
|
||||
@@ -2290,6 +2290,14 @@ fctx_query(fetchctx_t *fctx, dns_adbaddrinfo_t *addrinfo,
|
||||
|
||||
return (result);
|
||||
|
||||
cleanup_udpfetch:
|
||||
if (!RESQUERY_CANCELED(query)) {
|
||||
if ((query->options & DNS_FETCHOPT_TCP) == 0) {
|
||||
/* Inform the ADB that we're ending a UDP fetch */
|
||||
dns_adb_endudpfetch(fctx->adb, addrinfo);
|
||||
}
|
||||
}
|
||||
|
||||
cleanup_dispatch:
|
||||
fctx_detach(&query->fctx);
|
||||
|
||||
|
||||
Reference in New Issue
Block a user