See https://bugzilla.opensuse.org/show_bug.cgi?id=1151190 and its dependencies
for details.
Details
Helper still collects data correctly.
Diff Detail
- Repository
- R106 KSysguard
- Lint
Automatic diff as part of commit; lint not applicable. - Unit
Automatic diff as part of commit; unit tests not applicable.
Note that https://bugzilla.opensuse.org/show_bug.cgi?id=1152638 says pcap_breakloop() is unsupported when called from multiple threads. However, the actual libpcap documentation states:
Note also that, in a multi-threaded application, if one thread is blocked in pcap_dispatch(), pcap_loop(), pcap_next(3PCAP), or pcap_next_ex(3PCAP), a call to pcap_breakloop() in a different thread will not unblock that thread.
Which is a bit different, as it implies (to me) that the loop will still exit the next time the thread is woken up from its blocking call. Which is fine, the thread will be woken up when the next packet arrives and then exit.
Looks good to me. Thanks for the quick reaction (and sorry for my slow one!)
Which is a bit different, as it implies (to me) that the loop will still exit the next time the thread is woken up from its blocking call. Which is fine, the thread will be woken up when the next packet arrives and then exit.
Fair enough. Maybe I was a bit quick with my judgement there.