Hello.
On 27.04.2018 23:24, Alexander Aring wrote:
> This patch fix ifname setting, we should use the ifname stored inside
> this string and not give the programming declaration where to find the
> string. Founded by code inspection.
>
> Signed-off-by: Alexander Aring <aring@xxxxxxxxxxxx>
> ---
> wpan-ping/wpan-ping.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/wpan-ping/wpan-ping.c b/wpan-ping/wpan-ping.c
> index f825b22..791346c 100644
> --- a/wpan-ping/wpan-ping.c
> +++ b/wpan-ping/wpan-ping.c
> @@ -194,7 +194,7 @@ static int get_interface_info(struct config *conf) {
> nl_socket_modify_cb(conf->nl_sock, NL_CB_VALID, NL_CB_CUSTOM, nl_msg_cb, conf);
> msg = nlmsg_alloc();
> genlmsg_put(msg, NL_AUTO_PORT, NL_AUTO_SEQ, conf->nl802154_id, 0, NLM_F_DUMP, NL802154_CMD_GET_INTERFACE, 0);
> - nla_put_string(msg, NL802154_ATTR_IFNAME, "conf->interface");
> + nla_put_string(msg, NL802154_ATTR_IFNAME, conf->interface);
> nl_send_sync(conf->nl_sock, msg);
>
> nl802154_cleanup(conf);
>
Taking this fix out of the RFC series. Thanks for catching this.
Applied to wpan-tools repo.
regards
Stefan Schmidt
--
To unsubscribe from this list: send the line "unsubscribe linux-wpan" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
