Fix Ipv4Network::mask shift overflow bug

This commit is contained in:
Linus Färnstrand
2016-06-17 17:25:58 +02:00
parent 958a3f896e
commit dc763d2d03

View File

@ -59,7 +59,7 @@ impl Ipv4Network {
pub fn mask(&self) -> (Ipv4Addr, u32) { pub fn mask(&self) -> (Ipv4Addr, u32) {
let prefix = self.prefix; let prefix = self.prefix;
let mask = !(0xffffffff >> prefix); let mask = !(0xffffffff as u64 >> prefix) as u32;
(Ipv4Addr::from(mask), mask) (Ipv4Addr::from(mask), mask)
} }
@ -191,19 +191,26 @@ mod test {
} }
#[test] #[test]
fn parse_v4() { fn parse_v4_0bit() {
let cidr = Ipv4Network::from_cidr("0/0").unwrap(); let cidr = Ipv4Network::from_cidr("0/0").unwrap();
assert_eq!(cidr.ip(), Ipv4Addr::new(0, 0, 0, 0)); assert_eq!(cidr.ip(), Ipv4Addr::new(0, 0, 0, 0));
assert_eq!(cidr.prefix(), 0); assert_eq!(cidr.prefix(), 0);
} }
#[test] #[test]
fn parse_v4_2() { fn parse_v4_24bit() {
let cidr = Ipv4Network::from_cidr("127.1.0.0/24").unwrap(); let cidr = Ipv4Network::from_cidr("127.1.0.0/24").unwrap();
assert_eq!(cidr.ip(), Ipv4Addr::new(127, 1, 0, 0)); assert_eq!(cidr.ip(), Ipv4Addr::new(127, 1, 0, 0));
assert_eq!(cidr.prefix(), 24); assert_eq!(cidr.prefix(), 24);
} }
#[test]
fn parse_v4_32bit() {
let cidr = Ipv4Network::from_cidr("127.0.0.0/32").unwrap();
assert_eq!(cidr.ip(), Ipv4Addr::new(127, 0, 0, 0));
assert_eq!(cidr.prefix(), 32);
}
#[test] #[test]
fn parse_v4_fail_addr() { fn parse_v4_fail_addr() {
let cidr = Ipv4Network::from_cidr("10.a.b/8"); let cidr = Ipv4Network::from_cidr("10.a.b/8");