diff --git a/dumpcap.c b/dumpcap.c index c843ddb631..b7675fd78b 100644 --- a/dumpcap.c +++ b/dumpcap.c @@ -547,9 +547,10 @@ relinquish_all_capabilities(void) #elif defined(__APPLE__) #define PLATFORM_PERMISSIONS_SUGGESTION \ "\n\n" \ - "If you installed Wireshark using the package from wireshark.org, "\ - "Try re-installing it and checking the box for the \"Set capture " \ - "permissions on startup\" item." + "If you installed Wireshark using the package from wireshark.org, " \ + "close this dialog and click on the \"installing ChmodBPF\" link in " \ + "\"You can fix this by installing ChmodBPF.\" on the main screen, " \ + "and then complete the installation procedure." #else #define PLATFORM_PERMISSIONS_SUGGESTION #endif diff --git a/ui/qt/interface_frame.cpp b/ui/qt/interface_frame.cpp index e06fdf6ab7..ca3a407cef 100644 --- a/ui/qt/interface_frame.cpp +++ b/ui/qt/interface_frame.cpp @@ -303,6 +303,12 @@ void InterfaceFrame::resetInterfaceTreeDisplay() if (!haveLocalCapturePermissions()) { #ifdef Q_OS_MAC + // + // NOTE: if you change this text, you must also change the + // definition of PLATFORM_PERMISSIONS_SUGGESTION that is + // used if __APPLE__ is defined, so that it reflects the + // new message text. + // QString install_chmodbpf_path = wsApp->applicationDirPath() + "/../Resources/Extras/Install ChmodBPF.pkg"; ui->warningLabel->setText(tr( "

" @@ -312,6 +318,11 @@ void InterfaceFrame::resetInterfaceTreeDisplay() "

") .arg(install_chmodbpf_path)); #else + // + // XXX - should this give similar platform-dependent recommendations, + // just as dumpcap gives platform-dependent recommendations in its + // PLATFORM_PERMISSIONS_SUGGESTION #define? + // ui->warningLabel->setText(tr("You don't have permission to capture on local interfaces.")); #endif }