laforge submitted this change.

View Change

Approvals: pespin: Looks good to me, but someone else must approve Jenkins Builder: Verified laforge: Looks good to me, approved
netdev: netdev_mnl_set_mtu(): drop unneeded variables

Change-Id: I8f9d39733a80a0ec4e8c462f4ae32b4ac52b5849
---
M src/core/netdev.c
1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/core/netdev.c b/src/core/netdev.c
index 7e5a409..be7ad2e 100644
--- a/src/core/netdev.c
+++ b/src/core/netdev.c
@@ -541,8 +541,6 @@
char buf[MNL_SOCKET_BUFFER_SIZE];
struct nlmsghdr *nlh = mnl_nlmsg_put_header(buf);
struct ifinfomsg *ifm;
- unsigned int change = 0;
- unsigned int flags = 0;

nlh->nlmsg_type = RTM_NEWLINK;
nlh->nlmsg_flags = NLM_F_REQUEST | NLM_F_ACK;
@@ -550,8 +548,8 @@

ifm = mnl_nlmsg_put_extra_header(nlh, sizeof(*ifm));
ifm->ifi_family = AF_UNSPEC;
- ifm->ifi_change = change;
- ifm->ifi_flags = flags;
+ ifm->ifi_change = 0;
+ ifm->ifi_flags = 0;
ifm->ifi_index = if_index;

mnl_attr_put_u32(nlh, IFLA_MTU, mtu);

To view, visit change 42406. To unsubscribe, or for help writing mail filters, visit settings.

Gerrit-MessageType: merged
Gerrit-Project: libosmocore
Gerrit-Branch: master
Gerrit-Change-Id: I8f9d39733a80a0ec4e8c462f4ae32b4ac52b5849
Gerrit-Change-Number: 42406
Gerrit-PatchSet: 3
Gerrit-Owner: fixeria <vyanitskiy@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: laforge <laforge@osmocom.org>
Gerrit-Reviewer: pespin <pespin@sysmocom.de>