fix lint issues

This commit is contained in:
Siyuan Miao 2025-06-14 14:56:05 +02:00
parent cb7da61ab4
commit 6e25d44597
4 changed files with 15 additions and 12 deletions

View File

@ -18,7 +18,6 @@ const (
func afPacketComputeSize(targetSizeMb int, snaplen int, pageSize int) ( func afPacketComputeSize(targetSizeMb int, snaplen int, pageSize int) (
frameSize int, blockSize int, numBlocks int, err error) { frameSize int, blockSize int, numBlocks int, err error) {
if snaplen < pageSize { if snaplen < pageSize {
frameSize = pageSize / (pageSize / snaplen) frameSize = pageSize / (pageSize / snaplen)
} else { } else {

View File

@ -74,7 +74,10 @@ func (l *LLDP) Start() error {
l.l.Error().Err(err).Msg("unable to set up AF_PACKET") l.l.Error().Err(err).Msg("unable to set up AF_PACKET")
return err return err
} }
l.startCapture() if err := l.startCapture(); err != nil {
l.l.Error().Err(err).Msg("unable to start capture")
return err
}
} }
go l.neighbors.Start() go l.neighbors.Start()
@ -93,7 +96,7 @@ func (l *LLDP) Stop() error {
} }
if l.enableRx { if l.enableRx {
l.shutdownCapture() _ = l.shutdownCapture()
} }
l.neighbors.Stop() l.neighbors.Stop()

View File

@ -20,6 +20,8 @@ var (
// from lldpd // from lldpd
// https://github.com/lldpd/lldpd/blob/9034c9332cca0c8b1a20e1287f0e5fed81f7eb2a/src/daemon/lldpd.h#L246 // https://github.com/lldpd/lldpd/blob/9034c9332cca0c8b1a20e1287f0e5fed81f7eb2a/src/daemon/lldpd.h#L246
//
//nolint:govet
var bpfFilter = []bpf.RawInstruction{ var bpfFilter = []bpf.RawInstruction{
{0x30, 0, 0, 0x00000000}, {0x54, 0, 0, 0x00000001}, {0x15, 0, 16, 0x00000001}, {0x30, 0, 0, 0x00000000}, {0x54, 0, 0, 0x00000001}, {0x15, 0, 16, 0x00000001},
{0x28, 0, 0, 0x0000000c}, {0x15, 0, 6, 0x000088cc}, {0x28, 0, 0, 0x0000000c}, {0x15, 0, 6, 0x000088cc},
@ -110,7 +112,6 @@ func (l *LLDP) startCapture() error {
} }
} }
} }
}() }()
return nil return nil
@ -239,7 +240,7 @@ func (l *LLDP) handlePacketCDP(mac string, raw *layers.CiscoDiscovery, info *lay
} }
if len(info.MgmtAddresses) > 0 { if len(info.MgmtAddresses) > 0 {
n.ManagementAddress = fmt.Sprintf("%s", info.MgmtAddresses[0]) n.ManagementAddress = string(info.MgmtAddresses[0])
} }
l.addNeighbor(mac, *n, ttl) l.addNeighbor(mac, *n, ttl)

View File

@ -14,11 +14,7 @@ func (s *NetworkInterfaceState) shouldStartLLDP() bool {
s.l.Trace().Msgf("LLDP mode: %s", s.config.LLDPMode.String) s.l.Trace().Msgf("LLDP mode: %s", s.config.LLDPMode.String)
if s.config.LLDPMode.String == "disabled" { return s.config.LLDPMode.String != "disabled"
return false
}
return true
} }
func (s *NetworkInterfaceState) startLLDP() { func (s *NetworkInterfaceState) startLLDP() {
@ -27,7 +23,9 @@ func (s *NetworkInterfaceState) startLLDP() {
} }
s.l.Trace().Msg("starting LLDP") s.l.Trace().Msg("starting LLDP")
s.lldp.Start() if err := s.lldp.Start(); err != nil {
s.l.Error().Err(err).Msg("unable to start LLDP")
}
} }
func (s *NetworkInterfaceState) stopLLDP() { func (s *NetworkInterfaceState) stopLLDP() {
@ -35,7 +33,9 @@ func (s *NetworkInterfaceState) stopLLDP() {
return return
} }
s.l.Trace().Msg("stopping LLDP") s.l.Trace().Msg("stopping LLDP")
s.lldp.Stop() if err := s.lldp.Stop(); err != nil {
s.l.Error().Err(err).Msg("unable to stop LLDP")
}
} }
func (s *NetworkInterfaceState) GetLLDPNeighbors() ([]lldp.Neighbor, error) { func (s *NetworkInterfaceState) GetLLDPNeighbors() ([]lldp.Neighbor, error) {