Skip to content

Commit

Permalink
auto-redirect: Fix nftables available check
Browse files Browse the repository at this point in the history
  • Loading branch information
nekohasekai committed Jul 4, 2024
1 parent 60a8276 commit 5d44041
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions redirect_linux.go
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,6 @@ import (
"os"
"os/exec"
"runtime"
"time"

"github.com/sagernet/nftables"
"github.com/sagernet/sing/common"
Expand Down Expand Up @@ -81,7 +80,8 @@ func NewAutoRedirect(options AutoRedirectOptions) (AutoRedirect, error) {
if r.useNFTables {
err = r.initializeNFTables()
if err != nil && err != os.ErrInvalid {
r.logger.Debug("device has no nftables support: ", err)
r.useNFTables = false
r.logger.Debug("missing nftables support: ", err)
}
}
if len(r.tunOptions.Inet4Address) > 0 {
Expand Down Expand Up @@ -131,7 +131,6 @@ func (r *autoRedirect) Start() error {
}
r.redirectServer = server
}
startAt := time.Now()
var err error
if r.useNFTables {
r.cleanupNFTables()
Expand All @@ -140,11 +139,7 @@ func (r *autoRedirect) Start() error {
r.cleanupIPTables()
err = r.setupIPTables()
}
if err != nil {
return err
}
r.logger.Debug("auto-redirect configured in ", time.Since(startAt))
return nil
return err
}

func (r *autoRedirect) Close() error {
Expand Down

0 comments on commit 5d44041

Please sign in to comment.