summaryrefslogtreecommitdiffhomepage
diff options
context:
space:
mode:
authorLinus Färnstrand <linus@mullvad.net>2018-11-14 12:56:29 +0100
committerLinus Färnstrand <linus@mullvad.net>2018-11-14 17:20:48 +0100
commitbf2ee2e5d6368de4f6770c203bbe78fd874cda5b (patch)
treed5c7ac7d148266d1700f05f3db3ec2f2165bea54
parent60cb963caf8cedd2f2a0676810991654823c18f2 (diff)
downloadmullvadvpn-bf2ee2e5d6368de4f6770c203bbe78fd874cda5b.tar.xz
mullvadvpn-bf2ee2e5d6368de4f6770c203bbe78fd874cda5b.zip
Remove cache_dir from NetworkSecurity constructor
-rw-r--r--talpid-core/src/security/linux/mod.rs3
-rw-r--r--talpid-core/src/security/macos/dns.rs4
-rw-r--r--talpid-core/src/security/macos/mod.rs3
-rw-r--r--talpid-core/src/security/mod.rs8
-rw-r--r--talpid-core/src/security/windows/mod.rs3
-rw-r--r--talpid-core/src/tunnel_state_machine/mod.rs3
6 files changed, 10 insertions, 14 deletions
diff --git a/talpid-core/src/security/linux/mod.rs b/talpid-core/src/security/linux/mod.rs
index 7c7d1f64d9..a7897f0cc1 100644
--- a/talpid-core/src/security/linux/mod.rs
+++ b/talpid-core/src/security/linux/mod.rs
@@ -16,7 +16,6 @@ use std::{
ffi::CString,
io,
net::{IpAddr, Ipv4Addr},
- path::Path,
};
use talpid_types::net::{Endpoint, TransportProtocol};
@@ -78,7 +77,7 @@ pub struct NetworkSecurity {
impl NetworkSecurityT for NetworkSecurity {
type Error = Error;
- fn new(_cache_dir: impl AsRef<Path>) -> Result<Self> {
+ fn new() -> Result<Self> {
Ok(NetworkSecurity {
table_name: TABLE_NAME.clone(),
})
diff --git a/talpid-core/src/security/macos/dns.rs b/talpid-core/src/security/macos/dns.rs
index 56126fb20f..4937ffab03 100644
--- a/talpid-core/src/security/macos/dns.rs
+++ b/talpid-core/src/security/macos/dns.rs
@@ -17,10 +17,10 @@ use log::{debug, trace};
use std::{
collections::HashMap,
fmt,
+ net::IpAddr,
+ path::Path,
sync::{mpsc, Arc, Mutex},
thread,
- path::Path,
- net::IpAddr,
};
error_chain! {
diff --git a/talpid-core/src/security/macos/mod.rs b/talpid-core/src/security/macos/mod.rs
index 1fdb0c49b3..14a271ce89 100644
--- a/talpid-core/src/security/macos/mod.rs
+++ b/talpid-core/src/security/macos/mod.rs
@@ -3,7 +3,6 @@ extern crate tokio_core;
use super::{NetworkSecurityT, SecurityPolicy};
use std::net::Ipv4Addr;
-use std::path::Path;
use talpid_types::net;
mod dns;
@@ -26,7 +25,7 @@ pub struct NetworkSecurity {
impl NetworkSecurityT for NetworkSecurity {
type Error = Error;
- fn new(_cache_dir: impl AsRef<Path>) -> Result<Self> {
+ fn new() -> Result<Self> {
Ok(NetworkSecurity {
pf: pfctl::PfCtl::new()?,
pf_was_enabled: None,
diff --git a/talpid-core/src/security/mod.rs b/talpid-core/src/security/mod.rs
index 4d3a26cd13..d057fb8336 100644
--- a/talpid-core/src/security/mod.rs
+++ b/talpid-core/src/security/mod.rs
@@ -3,9 +3,9 @@ use ipnetwork::{IpNetwork, Ipv4Network, Ipv6Network};
#[cfg(unix)]
use lazy_static::lazy_static;
use std::fmt;
+use std::net::IpAddr;
#[cfg(unix)]
use std::net::{Ipv4Addr, Ipv6Addr};
-use std::net::IpAddr;
use std::path::Path;
use talpid_types::net::Endpoint;
@@ -115,9 +115,9 @@ pub struct NetworkSecurity {
impl NetworkSecurity {
/// Returns a new `NetworkSecurity`, ready to apply policies.
- pub fn new(cache_dir: impl AsRef<Path>) -> Result<Self, Error> {
+ pub fn new() -> Result<Self, Error> {
Ok(NetworkSecurity {
- inner: imp::NetworkSecurity::new(cache_dir)?,
+ inner: imp::NetworkSecurity::new()?,
})
}
@@ -176,7 +176,7 @@ trait NetworkSecurityT: Sized {
type Error: ::std::error::Error;
/// Create new instance
- fn new(cache_dir: impl AsRef<Path>) -> ::std::result::Result<Self, Self::Error>;
+ fn new() -> ::std::result::Result<Self, Self::Error>;
/// Enable the given SecurityPolicy
fn apply_policy(&mut self, policy: SecurityPolicy) -> ::std::result::Result<(), Self::Error>;
diff --git a/talpid-core/src/security/windows/mod.rs b/talpid-core/src/security/windows/mod.rs
index b6b99f8351..35dd57a1de 100644
--- a/talpid-core/src/security/windows/mod.rs
+++ b/talpid-core/src/security/windows/mod.rs
@@ -1,5 +1,4 @@
use std::net::IpAddr;
-use std::path::Path;
use std::ptr;
use log::{debug, error, trace};
@@ -65,7 +64,7 @@ pub struct NetworkSecurity(());
impl NetworkSecurityT for NetworkSecurity {
type Error = Error;
- fn new(_cache_dir: impl AsRef<Path>) -> Result<Self> {
+ fn new() -> Result<Self> {
unsafe {
WinFw_Initialize(
WINFW_TIMEOUT_SECONDS,
diff --git a/talpid-core/src/tunnel_state_machine/mod.rs b/talpid-core/src/tunnel_state_machine/mod.rs
index d6d9bfcc38..dcb4ada26c 100644
--- a/talpid-core/src/tunnel_state_machine/mod.rs
+++ b/talpid-core/src/tunnel_state_machine/mod.rs
@@ -191,8 +191,7 @@ impl TunnelStateMachine {
cache_dir: impl AsRef<Path>,
commands: mpsc::UnboundedReceiver<TunnelCommand>,
) -> Result<Self> {
- let security =
- NetworkSecurity::new(&cache_dir).chain_err(|| ErrorKind::NetworkSecurityError)?;
+ let security = NetworkSecurity::new().chain_err(|| ErrorKind::NetworkSecurityError)?;
let dns_monitor = DnsMonitor::new(cache_dir).chain_err(|| ErrorKind::DnsMonitorError)?;
let mut shared_values = SharedTunnelStateValues {
security,