Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Incorrect DNSc_GetHost() call in DNScCmd_GetHost() #1

Open
SeanAlling opened this issue Jul 17, 2020 · 0 comments · May be fixed by #2
Open

Incorrect DNSc_GetHost() call in DNScCmd_GetHost() #1

SeanAlling opened this issue Jul 17, 2020 · 0 comments · May be fixed by #2
Assignees
@wes-jmagasrevy
Milestone
V2.02.01

Comments

@SeanAlling
Copy link

Overview

When including the DNS shell commands in a build a compile time error will be generated in DNScCmd_GetHost() since no function matches DNSc_GetHost() that has 6 arguments.

Description

In DNScCmd_GetHost(), the following call to DNScCmd_GetHost() is

    status = DNSc_GetHost(p_argv[1],
                          addrs,
                         &addr_ctr,
                          DNSc_FLAG_NONE,
                          DEF_NULL,
                         &err);

This is providing 6 arguments to DNSc_GetHost(), but the function definition (shown below) specifies that 8 arguments are required.

DNSc_STATUS  DNSc_GetHost (const  CPU_CHAR       *p_host_name,
                                  CPU_CHAR       *p_res_host_name,
                                  CPU_INT32U      res_hostname_len,
                                  DNSc_ADDR_OBJ  *p_addrs,
                                  CPU_INT08U     *p_addr_nbr,
                                  DNSc_FLAGS      flags,
                                  DNSc_REQ_CFG   *p_cfg,
                                  DNSc_ERR       *p_err)
@wes-jmagasrevy wes-jmagasrevy self-assigned this Apr 27, 2021
@wes-jmagasrevy wes-jmagasrevy added this to the V2.02.01 milestone May 20, 2021
@feer9 feer9 linked a pull request Mar 18, 2024 that will close this issue
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@wes-jmagasrevy wes-jmagasrevy

Labels
None yet
Projects
None yet
Milestone
V2.02.01
Development

Successfully merging a pull request may close this issue.

Fix DNSc_GetHost() call in DNScCmd_GetHost() feer9/uC-DNSc
2 participants
@SeanAlling @wes-jmagasrevy