<table><tr><td style="">ahiemstra created this revision.<br />Herald added a project: Plasma.<br />Herald added a subscriber: plasma-devel.<br />ahiemstra requested review of this revision.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D24358">View Revision</a></tr></table><br /><div><strong>REVISION SUMMARY</strong><div><p>In case of conflict in i18n, keep the version of the branch "ours"<br />
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"</p>

<p>SVN_SILENT made messages (.desktop file) - always resolve ours</p>

<p>In case of conflict in i18n, keep the version of the branch "ours"<br />
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"</p>

<p>SVN_SILENT made messages (.desktop file) - always resolve ours</p>

<p>In case of conflict in i18n, keep the version of the branch "ours"<br />
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"</p>

<p>SVN_SILENT made messages (.desktop file) - always resolve ours</p>

<p>In case of conflict in i18n, keep the version of the branch "ours"<br />
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"</p>

<p>SVN_SILENT made messages (.desktop file) - always resolve ours</p>

<p>In case of conflict in i18n, keep the version of the branch "ours"<br />
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"</p>

<p>SVN_SILENT made messages (.desktop file) - always resolve ours</p>

<p>In case of conflict in i18n, keep the version of the branch "ours"<br />
To resolve a particular conflict, "git checkout --ours path/to/file.desktop"</p>

<p>networkhelper: Properly lock access to Accumulator::m_data</p>

<p>Since there are different threads accessing it.</p>

<p>Fixes OpenSuse Bug 1152634</p>

<p>networkhelper: Close the pcap instance when stop() is called</p>

<p>This prevents leaking of the pcap instance should start() ever be called<br />
again after stop().</p>

<p>This fixes most of OpenSuse Bug 1152638</p>

<p>networkhelper: Limit maximum size of incoming packet queue</p>

<p>Should there be so much traffic that the rest of the code cannot keep<br />
up, there now should be no risk that memory usage will keep growing.</p>

<p>Fixes OpenSuse Bug 1152639</p>

<p>networkhelper: Remove unused m_active member</p>

<p>It was a leftover from when we manually did the loop.</p>

<p>Fixes OpenSuse Bug 1152641</p>

<p>networkhelper: Ensure readlinkat results in a null terminated string</p>

<p>Apparently, readlinkat() will not ensure the result is null terminated,<br />
so decrease write size by one so the terminating null does not get<br />
overwritten.</p>

<p>Fixes OpenSuse Bug 1152642</p>

<p>networkhelper: Check for data length before trying to access packet parts</p>

<p>This ensures we actually have packet data with the required length<br />
instead of just assuming it does.</p>

<p>Fixes OpenSuse Bug 1152644</p>

<p>networkhelper: Do not call pcap_freecode when compilation failed</p>

<p>Since the structure will be empty in that case.</p>

<p>Fixes OpenSuse Bug 1152646</p>

<p>networkhelper: Properly convert IPv6 u8 array to u32</p>

<p>And-ing the parts would result in empty bytes and implcit conversion<br />
might fail.</p>

<p>Fixes Comment 3 of OpenSuse Bug 1151190</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R106 KSysguard</div></div></div><br /><div><strong>BRANCH</strong><div><div>secteam_fixes</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D24358">https://phabricator.kde.org/D24358</a></div></div><br /><div><strong>AFFECTED FILES</strong><div><div>plugins/process/network/helper/Accumulator.cpp<br />
plugins/process/network/helper/Accumulator.h<br />
plugins/process/network/helper/Capture.cpp<br />
plugins/process/network/helper/Capture.h<br />
plugins/process/network/helper/ConnectionMapping.cpp<br />
plugins/process/network/helper/Packet.cpp<br />
plugins/process/network/helper/Packet.h<br />
plugins/process/network/networkplugin.json</div></div></div><br /><div><strong>To: </strong>ahiemstra<br /><strong>Cc: </strong>plasma-devel, LeGast00n, The-Feren-OS-Dev, jraleigh, fbampaloukas, GB_2, ragreen, ZrenBot, ngraham, alexeymin, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>