osmocom
/
wireshark
11
0
Fork 1
Code Issues Pull Requests Projects Releases Wiki Activity

Properly check the return value of color_filters_set_tmp(). 

It returns FALSE, not TRUE, on failure.

Bug: 12101
Change-Id: I2fb4f407d2040e7b84f17af197de81555d64c647
Reviewed-on: https://code.wireshark.org/review/13857
Reviewed-by: Guy Harris <guy@alum.mit.edu>
This commit is contained in:
Guy Harris 2016-02-09 13:17:18 -08:00
parent 25ce3a34bd
commit ab977a86d0
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
ui/qt/main_window_slots.cpp
View File

@ -2390,7 +2390,7 @@ void MainWindow::colorizeWithFilter()
if (ok) {
// Assume "Color X"
gchar *err_msg = NULL;
if (color_filters_set_tmp(color_number, filter.toUtf8().constData(), FALSE, &err_msg)) {
if (!color_filters_set_tmp(color_number, filter.toUtf8().constData(), FALSE, &err_msg)) {
simple_dialog(ESD_TYPE_ERROR, ESD_BTN_OK, "%s", err_msg);
g_free(err_msg);
}