chall32 / LDWin

Link Discovery for Windows
395 stars 51 forks source link

Support reporting all hosts when receiving LLDP packets from multiple sources. #17

Open ocelotsloth opened 7 years ago

ocelotsloth commented 7 years ago

Currently this fails if you are connecting through more than a single device which broadcasts LLDP.

In particular, this fails both when an LLDP daemon is installed on the windows host itself (LDWin picks up those packets) or if you are connecting to a switch through something like an IP Phone which also reports LLDP.

On the command line with a tool like lldpd this is resolved by simply printing multiple information blocks. Is there a simple way to add support for reporting multiple hosts broadcasting LLDP information?

Unfortunately I cannot simply look into it myself due to no source code.... ☹️ Perhaps look into releasing this under an open source license? I'm sure the community would be most appreciative 😄.

ocelotsloth commented 7 years ago

Currently this fails

To clarify, it doesn't outright fail. Instead it only shows the information for one of the reporting devices with no way to choose which one is displayed.

cascadeth commented 7 years ago

@ocelotsloth To clarify, the source code is available, but you might want to check in with the dev @chall32 about editing it. https://github.com/chall32/LDWin/blob/master/LDWin.au3 It is a compiled AutoIT script.

Currently LDWin uses tcpdump with -c 1(capture only 1 and end capturing). You could alter it to capture more than one, and then parse for which one you want. One might run into difficulties if a device broadcasts more often than others. You might see a lot from a talkative switch, but less from your phone for example, depending on how often each broadcast their LLDP packet. Hope this helps? :)