From 670673cccc390e0d060b201553c9b878eec07f25 Mon Sep 17 00:00:00 2001
From: Roopesh Chander <roop@roopc.net>
Date: Wed, 16 Jan 2019 01:51:50 +0530
Subject: [PATCH] macOS: Use tunnelOverheadBytes for automatic MTU in macOS

---
 .../PacketTunnelSettingsGenerator.swift              | 12 ++++++++----
 1 file changed, 8 insertions(+), 4 deletions(-)

diff --git a/WireGuard/WireGuardNetworkExtension/PacketTunnelSettingsGenerator.swift b/WireGuard/WireGuardNetworkExtension/PacketTunnelSettingsGenerator.swift
index f6eb722..a1f90f1 100644
--- a/WireGuard/WireGuardNetworkExtension/PacketTunnelSettingsGenerator.swift
+++ b/WireGuard/WireGuardNetworkExtension/PacketTunnelSettingsGenerator.swift
@@ -70,7 +70,7 @@ class PacketTunnelSettingsGenerator {
         dnsSettings.matchDomains = [""] // All DNS queries must first go through the tunnel's DNS
         networkSettings.dnsSettings = dnsSettings
 
-        var mtu = tunnelConfiguration.interface.mtu ?? 0
+        let mtu = tunnelConfiguration.interface.mtu ?? 0
 
         /* 0 means automatic MTU. In theory, we should just do
          * `networkSettings.tunnelOverheadBytes = 80` but in
@@ -79,11 +79,15 @@ class PacketTunnelSettingsGenerator {
          * add a nob, maybe, or iOS will do probing for us.
          */
         if mtu == 0 {
-            mtu = 1280
+            #if os(iOS)
+            networkSettings.mtu = NSNumber(value: 1280)
+            #elseif os(OSX)
+            networkSettings.tunnelOverheadBytes = 80
+            #endif
+        } else {
+            networkSettings.mtu = NSNumber(value: mtu)
         }
 
-        networkSettings.mtu = NSNumber(value: mtu)
-
         let (ipv4Routes, ipv6Routes) = routes()
         let (ipv4IncludedRoutes, ipv6IncludedRoutes) = includedRoutes()