Revert "Add variable size ping"

This reverts commit eeacd42.

Author: Serock3

talpid-wireguard/src/connectivity_check.rs

@@ -511,13 +511,13 @@ mod test {
 
     #[derive(Default)]
     struct MockPinger {
-        on_send_ping: Option<Box<dyn FnMut(u16) + Send>>,
+        on_send_ping: Option<Box<dyn FnMut() + Send>>,
     }
 
     impl Pinger for MockPinger {
-        fn send_icmp_sized(&mut self, size: u16) -> Result<(), crate::ping_monitor::Error> {
+        fn send_icmp(&mut self) -> Result<(), crate::ping_monitor::Error> {
             if let Some(callback) = self.on_send_ping.as_mut() {
-                (callback)(size);
+                (callback)();
             }
             Ok(())
         }

talpid-wireguard/src/ping_monitor/icmp.rs

@@ -132,9 +132,8 @@ impl Pinger {
 }
 
 impl super::Pinger for Pinger {
-    fn send_icmp_sized(&mut self, size: u16) -> Result<()> {
-        const IPV4_HEADER_SIZE: u16 = 20;
-        let mut message = vec![0u8; (size - IPV4_HEADER_SIZE) as usize];
+    fn send_icmp(&mut self) -> Result<()> {
+        let mut message = [0u8; 50];
         self.construct_icmpv4_packet(&mut message)?;
         self.send_ping_request(&message, self.addr)
     }

talpid-wireguard/src/ping_monitor/mod.rs

@@ -11,11 +11,7 @@ pub use imp::Error;
 /// Trait for sending ICMP requests to get some traffic from a remote server
 pub trait Pinger: Send {
     /// Sends an ICMP packet
-    fn send_icmp(&mut self) -> Result<(), Error> {
-        self.send_icmp_sized(50)
-    }
-    /// Sends an ICMP packet of the given size
-    fn send_icmp_sized(&mut self, size: u16) -> Result<(), Error>;
+    fn send_icmp(&mut self) -> Result<(), Error>;
     /// Clears all resources used by the pinger.
     fn reset(&mut self) {}
 }