<table><tr><td style="">aacid added a comment.
</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/D12716">View Revision</a></tr></table><br /><div><div><p>The other option would be not using findNetworkInterface at all since we're already doing qobject_cast<NetworkManager::Device*>(sender()) we already have the device, we don't really need to do a round-trip, but wihtout knowing this code at all it seems like if NetworkManager::findNetworkInterface doesn't find the interface it's because it's being dropped (which makes sense with the bug being about "crash when discoonecting VPN") so it seems better to just ignore the connection at that point.</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R116 Plasma Network Management Applet</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D12716">https://phabricator.kde.org/D12716</a></div></div><br /><div><strong>To: </strong>aacid, Plasma, jgrulich<br /><strong>Cc: </strong>plasma-devel, ragreen, Pitel, ZrenBot, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart<br /></div>