author | drw <drw> | 2002-12-20 02:17:55 (UTC) |
---|---|---|
committer | drw <drw> | 2002-12-20 02:17:55 (UTC) |
commit | 9328bf79f49294e14c1753c9ee17ddd2985c1969 (patch) (unidiff) | |
tree | b0c9fa3e990b45cbfc9fb7fb33da73b9f6477df9 | |
parent | 3e556ed5f8c8b1236b9c6155b609930103d17b21 (diff) | |
download | opie-9328bf79f49294e14c1753c9ee17ddd2985c1969.zip opie-9328bf79f49294e14c1753c9ee17ddd2985c1969.tar.gz opie-9328bf79f49294e14c1753c9ee17ddd2985c1969.tar.bz2 |
Fix to prevent segfault when no process/module is selected in list.
-rw-r--r-- | noncore/settings/sysinfo/modulesinfo.cpp | 5 | ||||
-rw-r--r-- | noncore/settings/sysinfo/processinfo.cpp | 7 |
2 files changed, 11 insertions, 1 deletions
diff --git a/noncore/settings/sysinfo/modulesinfo.cpp b/noncore/settings/sysinfo/modulesinfo.cpp index 8b58fe9..29cce90 100644 --- a/noncore/settings/sysinfo/modulesinfo.cpp +++ b/noncore/settings/sysinfo/modulesinfo.cpp | |||
@@ -121,8 +121,13 @@ void ModulesInfo::updateData() | |||
121 | } | 121 | } |
122 | 122 | ||
123 | void ModulesInfo::slotSendClicked() | 123 | void ModulesInfo::slotSendClicked() |
124 | { | 124 | { |
125 | if ( !ModulesView->currentItem() ) | ||
126 | { | ||
127 | return; | ||
128 | } | ||
129 | |||
125 | QString capstr = tr( "You really want to execute\n" ); | 130 | QString capstr = tr( "You really want to execute\n" ); |
126 | capstr.append( CommandCB->currentText() ); | 131 | capstr.append( CommandCB->currentText() ); |
127 | capstr.append( "\nfor this module?" ); | 132 | capstr.append( "\nfor this module?" ); |
128 | 133 | ||
diff --git a/noncore/settings/sysinfo/processinfo.cpp b/noncore/settings/sysinfo/processinfo.cpp index 35d7ba1..9a5446a 100644 --- a/noncore/settings/sysinfo/processinfo.cpp +++ b/noncore/settings/sysinfo/processinfo.cpp | |||
@@ -143,13 +143,18 @@ void ProcessInfo::updateData() | |||
143 | } | 143 | } |
144 | 144 | ||
145 | void ProcessInfo::slotSendClicked() | 145 | void ProcessInfo::slotSendClicked() |
146 | { | 146 | { |
147 | QListViewItem *currprocess = ProcessView->currentItem(); | ||
148 | if ( !currprocess ) | ||
149 | { | ||
150 | return; | ||
151 | } | ||
152 | |||
147 | QString capstr = tr( "You really want to send\n" ); | 153 | QString capstr = tr( "You really want to send\n" ); |
148 | capstr.append( SignalCB->currentText() ); | 154 | capstr.append( SignalCB->currentText() ); |
149 | capstr.append( "\nto this process?" ); | 155 | capstr.append( "\nto this process?" ); |
150 | 156 | ||
151 | QListViewItem *currprocess = ProcessView->currentItem(); | ||
152 | 157 | ||
153 | if ( QMessageBox::warning( this, currprocess->text( 1 ), capstr, | 158 | if ( QMessageBox::warning( this, currprocess->text( 1 ), capstr, |
154 | QMessageBox::Yes | QMessageBox::Default, QMessageBox::No | QMessageBox::Escape ) == QMessageBox::Yes ) | 159 | QMessageBox::Yes | QMessageBox::Default, QMessageBox::No | QMessageBox::Escape ) == QMessageBox::Yes ) |
155 | { | 160 | { |