Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

PcapLiveDevice stores IP addresses in structured that may be overwritten. #1499

Closed
2 tasks done
Dimi1010 opened this issue Jul 17, 2024 · 3 comments · Fixed by #1497
Closed
2 tasks done

PcapLiveDevice stores IP addresses in structured that may be overwritten. #1499

Dimi1010 opened this issue Jul 17, 2024 · 3 comments · Fixed by #1497
Assignees
Labels

Comments

@Dimi1010
Copy link
Collaborator

Dimi1010 commented Jul 17, 2024

During initialization of PcapLiveDeviceList information of all devices is fetched via pcap_findalldevs. This information is used to populate PcapLiveDevice objects and then the list is freed. This results in an issue as the device instance holds its IP address information in pcap_addr_t structs that are technically dangling, but still working somehow.

The fix to this issue will be implemented as a two stage process.

@seladb
Copy link
Owner

seladb commented Jul 29, 2024

This issue was fixed in #1500

@seladb seladb closed this as completed Jul 29, 2024
@Dimi1010
Copy link
Collaborator Author

Dimi1010 commented Jul 29, 2024

This issue was fixed in #1500

It hasn't been fixed. That was only Task 1.
The underlying IP is still stored in pcap_addr_t which I can't remove until PcapLiveDevice::getAddresses() is removed which needs to go though its deprecation cycle.

@Dimi1010
Copy link
Collaborator Author

Fixed by #1497

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants