mdnsd.c: fix conflict detection to filter out all assigned IP addresses

[thom-nic]

• fixes conflicting name detected troglobit/mdnsd#82

[troglobit]

Nice work!

[thom-nic]

Hi @troglobit thanks for looking at this. This PR is a mirror of troglobit#88 which I had opened here in case I needed to start maintaining a divergent fork of the official repo.

What do you think of this versus #3 ? The caching behavior won't cause false positives (conflicts) due to the forced-refresh behavior. But could result in false negatives, I think: incorrectly treating the address as local, and not treating it as a conflict. This could occur if the host released an address and then received a packet from that address which was then ignored, before the cache was refreshed.

[troglobit]

Hi @troglobit thanks for looking at this. This PR is a mirror of troglobit#88 which I had opened here in case I needed to start maintaining a divergent fork of the official repo.

Aha, LOL! In my sleep deprived state I didn't understand why I couldn't merge this 😄

What do you think of this versus #3 ? The caching behavior won't cause false positives (conflicts) due to the forced-refresh behavior. But could result in false negatives, I think: incorrectly treating the address as local, and not treating it as a conflict. This could occur if the host released an address and then received a packet from that address which was then ignored, before the cache was refreshed.

I think this cached version is the way to go, definitely.