Skip to content

Commit 4cf6f40

Browse files
committed
clippy warnings fixed
1 parent 1eebeed commit 4cf6f40

1 file changed

Lines changed: 20 additions & 25 deletions

File tree

src/netlink.rs

Lines changed: 20 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1218,19 +1218,17 @@ unsafe fn parse_neigh_nlmsg_v4(
12181218
let dptr = base.add(off + rta_hdr);
12191219
let dlen = rlen - rta_hdr;
12201220
match rta.rta_type {
1221-
1 /* NDA_DST */ => {
1222-
if dlen == 4 {
1223-
let mut a = [0u8; 4];
1224-
std::ptr::copy_nonoverlapping(dptr, a.as_mut_ptr(), 4);
1225-
if &a == target_ip4 { found_ip = true; }
1221+
1 if dlen == 4 => {
1222+
let mut a = [0u8; 4];
1223+
std::ptr::copy_nonoverlapping(dptr, a.as_mut_ptr(), 4);
1224+
if &a == target_ip4 {
1225+
found_ip = true;
12261226
}
12271227
}
1228-
2 /* NDA_LLADDR */ => {
1229-
if dlen == 6 {
1230-
let mut m = [0u8; 6];
1231-
std::ptr::copy_nonoverlapping(dptr, m.as_mut_ptr(), 6);
1232-
mac = Some(m);
1233-
}
1228+
2 if dlen == 6 => {
1229+
let mut m = [0u8; 6];
1230+
std::ptr::copy_nonoverlapping(dptr, m.as_mut_ptr(), 6);
1231+
mac = Some(m);
12341232
}
12351233
_ => {}
12361234
}
@@ -1380,19 +1378,17 @@ unsafe fn parse_neigh_nlmsg(
13801378
let dptr = base.add(off + rta_hdr);
13811379
let dlen = rlen - rta_hdr;
13821380
match rta.rta_type {
1383-
1 /* NDA_DST */ => {
1384-
if dlen == 16 {
1385-
let mut a = [0u8; 16];
1386-
std::ptr::copy_nonoverlapping(dptr, a.as_mut_ptr(), 16);
1387-
if &a == target_ip6 { found_ip = true; }
1381+
1 if dlen == 16 => {
1382+
let mut a = [0u8; 16];
1383+
std::ptr::copy_nonoverlapping(dptr, a.as_mut_ptr(), 16);
1384+
if &a == target_ip6 {
1385+
found_ip = true;
13881386
}
13891387
}
1390-
2 /* NDA_LLADDR */ => {
1391-
if dlen == 6 {
1392-
let mut m = [0u8; 6];
1393-
std::ptr::copy_nonoverlapping(dptr, m.as_mut_ptr(), 6);
1394-
mac = Some(m);
1395-
}
1388+
2 if dlen == 6 => {
1389+
let mut m = [0u8; 6];
1390+
std::ptr::copy_nonoverlapping(dptr, m.as_mut_ptr(), 6);
1391+
mac = Some(m);
13961392
}
13971393
_ => {}
13981394
}
@@ -1549,8 +1545,8 @@ pub fn nl_get_route(
15491545
.add(attr_offset + std::mem::size_of::<Rtattr>());
15501546
let dlen = rta_len - std::mem::size_of::<Rtattr>();
15511547
match rta.rta_type {
1552-
4 /* RTA_OIF */ => {
1553-
if dlen == 4 {
1548+
4
1549+
if dlen == 4 => {
15541550
let mut ifindex = [0u8; 4];
15551551
std::ptr::copy_nonoverlapping(data_ptr, ifindex.as_mut_ptr(), 4);
15561552
let idx = i32::from_ne_bytes(ifindex);
@@ -1559,7 +1555,6 @@ pub fn nl_get_route(
15591555
dev = Some(std::ffi::CStr::from_ptr(ifname.as_ptr()).to_string_lossy().into_owned());
15601556
}
15611557
}
1562-
}
15631558
5 /* RTA_GATEWAY */ => {
15641559
if is_v4 && dlen == 4 {
15651560
let mut octets = [0u8; 4];

0 commit comments

Comments
 (0)