Skip to content

Commit

Permalink
RESOLV: removed unused argument
Browse files Browse the repository at this point in the history
`resolv_gethostbyname_dns_parse()` didn't use `status` and it was
always set to `ARES_SUCCESS` anyway.

Reviewed-by: Alejandro López <[email protected]>
Reviewed-by: Tomáš Halman <[email protected]>
  • Loading branch information
alexey-tikhonov committed Sep 18, 2024
1 parent c629868 commit 41dfdcc
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions src/resolv/async_resolv.c
Original file line number Diff line number Diff line change
Expand Up @@ -794,7 +794,7 @@ resolv_gethostbyname_dns_query_done(void *arg, int status, int timeouts,
unsigned char *abuf, int alen);
static int
resolv_gethostbyname_dns_parse(struct gethostbyname_dns_state *state,
int status, unsigned char *abuf, int alen);
unsigned char *abuf, int alen);

static struct tevent_req *
resolv_gethostbyname_dns_send(TALLOC_CTX *mem_ctx, struct tevent_context *ev,
Expand Down Expand Up @@ -935,7 +935,7 @@ resolv_gethostbyname_dns_query_done(void *arg, int status, int timeouts,
return;
}

ret = resolv_gethostbyname_dns_parse(state, status, abuf, alen);
ret = resolv_gethostbyname_dns_parse(state, abuf, alen);
if (ret != EOK) {
tevent_req_error(req, ret);
return;
Expand All @@ -946,11 +946,12 @@ resolv_gethostbyname_dns_query_done(void *arg, int status, int timeouts,

static int
resolv_gethostbyname_dns_parse(struct gethostbyname_dns_state *state,
int status, unsigned char *abuf, int alen)
unsigned char *abuf, int alen)
{
struct hostent *hostent = NULL;
int naddrttls;
errno_t ret;
int status;
void *addr = NULL;

naddrttls = DNS_HEADER_ANCOUNT(abuf);
Expand Down

0 comments on commit 41dfdcc

Please sign in to comment.