Skip to content

udp: fix destination address#265

Merged
nl5887 merged 1 commit intohoneytrap:masterfrom
CapacitorSet:fix-263
Nov 7, 2018
Merged

udp: fix destination address#265
nl5887 merged 1 commit intohoneytrap:masterfrom
CapacitorSet:fix-263

Conversation

@CapacitorSet
Copy link
Copy Markdown
Contributor

Fixes #263. The explanation for the bug is (from my comment on the issue):

listener/socket/socket.go:102 uses the UDP listening address as the value for Laddr, for which DummyUDPConn.LocalAddr is a wrapper. For whatever reason, it does not contain an actual IP.

With this fix, it now shows :: as a destination IP for UDP events, at least on my machine.

@nl5887
Copy link
Copy Markdown
Contributor

nl5887 commented Apr 19, 2018

This is a little bit different. When having a ListenUDP the LocalAddr isn't being filled correctly (I've experienced this in the agent), but maybe this is an Go issue itself.

@nl5887 nl5887 merged commit d90f39f into honeytrap:master Nov 7, 2018
@nl5887
Copy link
Copy Markdown
Contributor

nl5887 commented Nov 7, 2018

merged, thanks!

@CapacitorSet CapacitorSet deleted the fix-263 branch November 7, 2018 14:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

honeytrap: agent: destination address is 0.0.0.0

2 participants