On Jun 10, 2024, at 12:39 PM, Alberto Perez Bogantes via tcpdump-workers 
<tcpdump-workers@lists.tcpdump.org> wrote:

> We've been working on adding a new feature to tcpdump that will allow IP
> address anonymization ...

...

> 
> I am sharing my GitHub project (https://github.com/aperezb21/tcpdump),
> which is forked from commit bb704ed32d770e84fdc340de8276c261bb6e9ee1,
> containing the current prototype. We welcome any discussion or feedback,
> both on or off-list.

The change at

        
https://github.com/the-tcpdump-group/tcpdump/commit/7ced1b157db335467915aee558e48e951c9b44e4

looks as if you're doing more than just anonymizing addresses in IP headers, 
which is, obviously, necessary.

However, the changes look a bit intrusive - why must all that stuff be moved 
from print-arp.c to arp.h, for example?  Perhaps distribution the anonymization 
code throughout the print-XXX.c files, rather than centralizing it, would work 
better.

There are also a *LOT* of white space changes to tcpdump.c; those shouldn't be 
done.
_______________________________________________
tcpdump-workers mailing list -- tcpdump-workers@lists.tcpdump.org
To unsubscribe send an email to tcpdump-workers-le...@lists.tcpdump.org
%(web_page_url)slistinfo%(cgiext)s/%(_internal_name)s

Reply via email to