some done todos

[hippotat.git] / server / slocal.rs

diff --git a/server/slocal.rs b/server/slocal.rs
index a30aad6a61e76c2b6b877a5340cf7b693b47b969..19944119e00deb2a2b070b3c94bb950b2e10fd27 100644 (file)
--- a/server/slocal.rs
+++ b/server/slocal.rs
@@ -35,43 +35,38 @@ pub async fn run(global: Arc<Global>,
 
           goodness -= goodness >> GOODNESS_SHIFT;
 
-          match checkn(SlipNoConv, global.config.mtu, &data, |header|{
+          match process1(SlipNoConv, global.config.mtu, &data, |header|{
             let saddr = ip_packet_addr::<false>(header)?;
             let daddr = ip_packet_addr::<true>(header)?;
             Ok((saddr,daddr))
-          }, |(data, (saddr, daddr))| {
-            let global = &global;
-            let may_route = &may_route;
-            async move {
-              if ! global.config.vnetwork.iter().any(|n| n.contains(&saddr)) {
-                // pretent as if this came from route
-                trace!(
-                  target: "hippotatd",
- "discard to={:?} came=ipif user=local from-outside-vnets-={:?}",
-                  daddr, saddr);
-                return Ok(())
-              }
-
-              route_packet(
-                &global, "ipif", None,
-                data, daddr, may_route.clone()
-              ).await;
+          }) {
+            Err(PE::Empty) => { },
 
-              Ok(())
-            }
-          }, |pe| Ok(match pe {
-            PE::Empty => { },
-            other => throw!(other),
-          })).await {
-            Ok(()) => goodness += 1,
-            Err(e) => {
+            Err(pe) => {
               goodness -= 1;
               error!("[good={}] invalid data from local tx ipif {}",
-                     goodness, e);
+                     goodness, pe);
               if goodness < GOODNESS_MIN {
                 throw!(anyhow!("too many bad packets, too few good ones!"))
               }
             },
+
+            Ok((ref data, (ref saddr, ref daddr)))
+            if ! global.config.vnetwork.iter().any(|n| n.contains(saddr)) => {
+              // pretent as if this came from route
+              trace!(
+                target: "hippotatd",
+ "discard to={:?} came=ipif user=local len={} outside-vnets: from={:?}",
+                daddr, saddr, data.len());
+            },
+
+            Ok((data, (saddr, daddr))) => {
+              goodness += 1;
+              route_packet(
+                &global, "ipif", None,
+                data, daddr, may_route.clone()
+              ).await;
+            }
           }
         },
       }