diff options
| author | Nikola Pajkovsky <npajkovs@redhat.com> | 2010-03-01 15:33:21 +0100 |
|---|---|---|
| committer | Nikola Pajkovsky <npajkovs@redhat.com> | 2010-03-01 15:33:21 +0100 |
| commit | 7533baf4751cc405fdcce2122d7b0e3c1193f2b6 (patch) | |
| tree | 857ab1f3e51157e14a64454ea29c373dbbce06dd /src/Daemon/CommLayerServerDBus.cpp | |
| parent | d74e3534266c1a39cc1972f64e77426f2fabf564 (diff) | |
| parent | 755b62981c0aef4ff825f8f8465d3e38123889b3 (diff) | |
| download | abrt-7533baf4751cc405fdcce2122d7b0e3c1193f2b6.tar.gz abrt-7533baf4751cc405fdcce2122d7b0e3c1193f2b6.tar.xz abrt-7533baf4751cc405fdcce2122d7b0e3c1193f2b6.zip | |
Merge branch 'master' of ssh://git.fedorahosted.org/git/abrt
Diffstat (limited to 'src/Daemon/CommLayerServerDBus.cpp')
| -rw-r--r-- | src/Daemon/CommLayerServerDBus.cpp | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/src/Daemon/CommLayerServerDBus.cpp b/src/Daemon/CommLayerServerDBus.cpp index 47b6682..a397dd1 100644 --- a/src/Daemon/CommLayerServerDBus.cpp +++ b/src/Daemon/CommLayerServerDBus.cpp @@ -227,12 +227,11 @@ static int handle_Report(DBusMessage* call, DBusMessage* reply) map_crash_data_t argin1; r = load_val(&in_iter, argin1); - if (r == ABRT_DBUS_ERROR) + if (r != ABRT_DBUS_MORE_FIELDS) { error_msg("dbus call %s: parameter type mismatch", __func__ + 7); return -1; } - const char* comment = get_crash_data_item_content_or_NULL(argin1, FILENAME_COMMENT) ? : ""; const char* reproduce = get_crash_data_item_content_or_NULL(argin1, FILENAME_REPRODUCE) ? : ""; const char* errmsg = NULL; @@ -254,7 +253,16 @@ static int handle_Report(DBusMessage* call, DBusMessage* reply) return 0; } - /* Second parameter is optional */ + /* Second parameter: reporters to use */ + vector_string_t reporters; + r = load_val(&in_iter, reporters); + if (r == ABRT_DBUS_ERROR) + { + error_msg("dbus call %s: parameter type mismatch", __func__ + 7); + return -1; + } + + /* Third parameter is optional */ map_map_string_t user_conf_data; if (r == ABRT_DBUS_MORE_FIELDS) { |