commit
b6ea1c4cb6
Binary file not shown.
BIN
ibus-1.5.22.tar.gz
Normal file
BIN
ibus-1.5.22.tar.gz
Normal file
Binary file not shown.
@ -1,108 +1,119 @@
|
|||||||
From 865b204f1c06b782cf7b4a479b358e76f4b3dfee Mon Sep 17 00:00:00 2001
|
From 023d50db40912e4f7ee333543e05995a9c730bae Mon Sep 17 00:00:00 2001
|
||||||
From: fujiwarat <takao.fujiwara1@gmail.com>
|
From: fujiwarat <takao.fujiwara1@gmail.com>
|
||||||
Date: Tue, 17 Jul 2018 13:39:30 +0900
|
Date: Fri, 15 May 2020 21:44:16 +0900
|
||||||
Subject: [PATCH] bus: Fix SEGV in bus_panel_proxy_focus_in()
|
Subject: [PATCH] Fix SEGV in bus_panel_proxy_focus_in()
|
||||||
|
|
||||||
|
rhbz#1350291 SEGV in BUS_IS_CONNECTION(skip_connection) in
|
||||||
|
bus_dbus_impl_dispatch_message_by_rule()
|
||||||
|
check if dbus_connection is closed in bus_dbus_impl_connection_filter_cb().
|
||||||
|
|
||||||
|
rhbz#1767976 SEGV in assert(connection != NULL) in
|
||||||
|
bus_dbus_impl_connection_filter_cb()
|
||||||
|
call bus_connection_set_filter() in bus_dbus_impl_destroy().
|
||||||
|
|
||||||
|
rhbz#1601577 rhbz#1797726 SEGV in ibus_engine_desc_get_layout() in
|
||||||
|
bus_engine_proxy_new_internal()
|
||||||
|
WIP: Added a GError to get the error message to check why the SEGV happened.
|
||||||
|
|
||||||
|
rhbz#1663528 SEGV in g_mutex_clear() in bus_dbus_impl_destroy()
|
||||||
|
If the mutex is not unlocked, g_mutex_clear() causes assert.
|
||||||
|
|
||||||
|
rhbz#1767691 SEGV in client/x11/main.c:_sighandler().
|
||||||
|
Do not call atexit functions in _sighandler().
|
||||||
|
|
||||||
|
rhbz#1795499 SEGV in ibus_bus_get_bus_address() because of no _bus->priv.
|
||||||
|
_changed_cb() should not be called after ibus_bus_destroy() is called.
|
||||||
|
|
||||||
|
rhbz#1771238 SEGV in assert(m_loop == null) in switcher.vala.
|
||||||
|
Grabbing keyboard could be failed and switcher received the keyboard
|
||||||
|
events and m_loop was not released.
|
||||||
|
|
||||||
|
rhbz#1797120 SEGV in assert(bus.is_connected()) in panel_binding_construct()
|
||||||
|
Check m_ibus in extension.vala:bus_name_acquired_cb()
|
||||||
|
|
||||||
BUG=rhbz#1349148
|
|
||||||
BUG=rhbz#1385349
|
|
||||||
BUG=rhbz#1350291
|
BUG=rhbz#1350291
|
||||||
BUG=rhbz#1406699
|
|
||||||
BUG=rhbz#1432252
|
|
||||||
BUG=rhbz#1601577
|
BUG=rhbz#1601577
|
||||||
|
BUG=rhbz#1663528
|
||||||
|
BUG=rhbz#1767691
|
||||||
|
BUG=rhbz#1795499
|
||||||
|
BUG=rhbz#1771238
|
||||||
|
BUG=rhbz#1767976
|
||||||
|
BUG=rhbz#1797120
|
||||||
---
|
---
|
||||||
bus/dbusimpl.c | 38 ++++++++++++++++++++++++++++++++------
|
bus/dbusimpl.c | 47 ++++++++++++++++++++++++---
|
||||||
bus/engineproxy.c | 5 ++++-
|
bus/engineproxy.c | 51 ++++++++++++++++++++++-------
|
||||||
bus/ibusimpl.c | 21 ++++++++++++++++++---
|
client/x11/main.c | 8 ++++-
|
||||||
3 files changed, 54 insertions(+), 10 deletions(-)
|
src/ibusbus.c | 5 +++
|
||||||
|
ui/gtk3/extension.vala | 4 +++
|
||||||
|
ui/gtk3/switcher.vala | 73 +++++++++++++++++++++++++-----------------
|
||||||
|
6 files changed, 141 insertions(+), 47 deletions(-)
|
||||||
|
|
||||||
diff --git a/bus/dbusimpl.c b/bus/dbusimpl.c
|
diff --git a/bus/dbusimpl.c b/bus/dbusimpl.c
|
||||||
index b54ef817..e4dd8683 100644
|
index 59787a80..af2fbde2 100644
|
||||||
--- a/bus/dbusimpl.c
|
--- a/bus/dbusimpl.c
|
||||||
+++ b/bus/dbusimpl.c
|
+++ b/bus/dbusimpl.c
|
||||||
@@ -2,7 +2,8 @@
|
@@ -610,6 +610,7 @@ static void
|
||||||
/* vim:set et sts=4: */
|
bus_dbus_impl_destroy (BusDBusImpl *dbus)
|
||||||
/* ibus - The Input Bus
|
|
||||||
* Copyright (C) 2008-2013 Peng Huang <shawn.p.huang@gmail.com>
|
|
||||||
- * Copyright (C) 2008-2013 Red Hat, Inc.
|
|
||||||
+ * Copyright (C) 2015-2017 Takao Fujiwara <takao.fujiwara1@gmail.com>
|
|
||||||
+ * Copyright (C) 2008-2017 Red Hat, Inc.
|
|
||||||
*
|
|
||||||
* This library is free software; you can redistribute it and/or
|
|
||||||
* modify it under the terms of the GNU Lesser General Public
|
|
||||||
@@ -344,6 +345,8 @@ bus_name_service_set_primary_owner (BusNameService *service,
|
|
||||||
BusConnectionOwner *owner,
|
|
||||||
BusDBusImpl *dbus)
|
|
||||||
{
|
{
|
||||||
+ gboolean has_old_owner = FALSE;
|
GList *p;
|
||||||
+
|
+ int i;
|
||||||
g_assert (service != NULL);
|
|
||||||
g_assert (owner != NULL);
|
|
||||||
g_assert (dbus != NULL);
|
|
||||||
@@ -351,6 +354,13 @@ bus_name_service_set_primary_owner (BusNameService *service,
|
|
||||||
BusConnectionOwner *old = service->owners != NULL ?
|
|
||||||
(BusConnectionOwner *)service->owners->data : NULL;
|
|
||||||
|
|
||||||
+ /* rhbz#1432252 If bus_connection_get_unique_name() == NULL,
|
for (p = dbus->objects; p != NULL; p = p->next) {
|
||||||
+ * "Hello" method is not received yet.
|
IBusService *object = (IBusService *) p->data;
|
||||||
|
@@ -633,6 +634,10 @@ bus_dbus_impl_destroy (BusDBusImpl *dbus)
|
||||||
|
|
||||||
|
for (p = dbus->connections; p != NULL; p = p->next) {
|
||||||
|
BusConnection *connection = BUS_CONNECTION (p->data);
|
||||||
|
+ /* rhbz#1767976 Fix connection == NULL in
|
||||||
|
+ * bus_dbus_impl_connection_filter_cb()
|
||||||
+ */
|
+ */
|
||||||
+ if (old != NULL && bus_connection_get_unique_name (old->conn) != NULL) {
|
+ bus_connection_set_filter (connection, NULL, NULL, NULL);
|
||||||
+ has_old_owner = TRUE;
|
g_signal_handlers_disconnect_by_func (connection,
|
||||||
|
bus_dbus_impl_connection_destroy_cb, dbus);
|
||||||
|
ibus_object_destroy (IBUS_OBJECT (connection));
|
||||||
|
@@ -647,12 +652,39 @@ bus_dbus_impl_destroy (BusDBusImpl *dbus)
|
||||||
|
dbus->unique_names = NULL;
|
||||||
|
dbus->names = NULL;
|
||||||
|
|
||||||
|
+ for (i = 0; g_idle_remove_by_data (dbus); i++) {
|
||||||
|
+ if (i > 1000) {
|
||||||
|
+ g_warning ("Too many idle threads were generated by " \
|
||||||
|
+ "bus_dbus_impl_forward_message_idle_cb and " \
|
||||||
|
+ "bus_dbus_impl_dispatch_message_by_rule_idle_cb");
|
||||||
|
+ break;
|
||||||
|
+ }
|
||||||
|
+ }
|
||||||
|
g_list_free_full (dbus->start_service_calls,
|
||||||
|
(GDestroyNotify) bus_method_call_free);
|
||||||
|
dbus->start_service_calls = NULL;
|
||||||
|
|
||||||
|
- g_mutex_clear (&dbus->dispatch_lock);
|
||||||
|
- g_mutex_clear (&dbus->forward_lock);
|
||||||
|
+ /* rhbz#1663528 Call g_mutex_trylock() before g_mutex_clear()
|
||||||
|
+ * because if the mutex is not unlocked, g_mutex_clear() causes assert.
|
||||||
|
+ */
|
||||||
|
+#define BUS_DBUS_MUTEX_SAFE_CLEAR(mtex) { \
|
||||||
|
+ int count = 0; \
|
||||||
|
+ while (!g_mutex_trylock ((mtex))) { \
|
||||||
|
+ g_usleep (1); \
|
||||||
|
+ if (count > 60) { \
|
||||||
|
+ g_warning (#mtex " is dead lock"); \
|
||||||
|
+ break; \
|
||||||
|
+ } \
|
||||||
|
+ ++count; \
|
||||||
|
+ } \
|
||||||
|
+ g_mutex_unlock ((mtex)); \
|
||||||
|
+ g_mutex_clear ((mtex)); \
|
||||||
+}
|
+}
|
||||||
+
|
+
|
||||||
if (old != NULL) {
|
+ BUS_DBUS_MUTEX_SAFE_CLEAR (&dbus->dispatch_lock);
|
||||||
g_signal_emit (dbus,
|
+ BUS_DBUS_MUTEX_SAFE_CLEAR (&dbus->forward_lock);
|
||||||
dbus_signals[NAME_LOST],
|
+
|
||||||
@@ -370,7 +380,8 @@ bus_name_service_set_primary_owner (BusNameService *service,
|
+#undef BUS_DBUS_MUTEX_SAFE_CLEAR
|
||||||
0,
|
|
||||||
owner->conn,
|
|
||||||
service->name,
|
|
||||||
- old != NULL ? bus_connection_get_unique_name (old->conn) : "",
|
|
||||||
+ has_old_owner ? bus_connection_get_unique_name (old->conn) :
|
|
||||||
+ "",
|
|
||||||
bus_connection_get_unique_name (owner->conn));
|
|
||||||
|
|
||||||
if (old != NULL && old->do_not_queue != 0) {
|
/* FIXME destruct _lock and _queue members. */
|
||||||
@@ -427,6 +438,7 @@ bus_name_service_remove_owner (BusNameService *service,
|
IBUS_OBJECT_CLASS(bus_dbus_impl_parent_class)->destroy ((IBusObject *) dbus);
|
||||||
BusDBusImpl *dbus)
|
@@ -1483,13 +1515,20 @@ bus_dbus_impl_connection_filter_cb (GDBusConnection *dbus_connection,
|
||||||
{
|
|
||||||
GSList *owners;
|
|
||||||
+ gboolean has_new_owner = FALSE;
|
|
||||||
|
|
||||||
g_assert (service != NULL);
|
|
||||||
g_assert (owner != NULL);
|
|
||||||
@@ -439,6 +451,13 @@ bus_name_service_remove_owner (BusNameService *service,
|
|
||||||
BusConnectionOwner *_new = NULL;
|
|
||||||
if (owners->next != NULL) {
|
|
||||||
_new = (BusConnectionOwner *)owners->next->data;
|
|
||||||
+ /* rhbz#1406699 If bus_connection_get_unique_name() == NULL,
|
|
||||||
+ * "Hello" method is not received yet.
|
|
||||||
+ */
|
|
||||||
+ if (_new != NULL &&
|
|
||||||
+ bus_connection_get_unique_name (_new->conn) != NULL) {
|
|
||||||
+ has_new_owner = TRUE;
|
|
||||||
+ }
|
|
||||||
}
|
|
||||||
|
|
||||||
if (dbus != NULL) {
|
|
||||||
@@ -447,7 +466,7 @@ bus_name_service_remove_owner (BusNameService *service,
|
|
||||||
0,
|
|
||||||
owner->conn,
|
|
||||||
service->name);
|
|
||||||
- if (_new != NULL) {
|
|
||||||
+ if (has_new_owner) {
|
|
||||||
g_signal_emit (dbus,
|
|
||||||
dbus_signals[NAME_ACQUIRED],
|
|
||||||
0,
|
|
||||||
@@ -460,7 +479,7 @@ bus_name_service_remove_owner (BusNameService *service,
|
|
||||||
_new != NULL ? _new->conn : NULL,
|
|
||||||
service->name,
|
|
||||||
bus_connection_get_unique_name (owner->conn),
|
|
||||||
- _new != NULL ? bus_connection_get_unique_name (_new->conn) : "");
|
|
||||||
+ has_new_owner ? bus_connection_get_unique_name (_new->conn) : "");
|
|
||||||
|
|
||||||
}
|
|
||||||
}
|
|
||||||
@@ -1464,13 +1483,20 @@ bus_dbus_impl_connection_filter_cb (GDBusConnection *dbus_connection,
|
|
||||||
gboolean incoming,
|
gboolean incoming,
|
||||||
gpointer user_data)
|
gpointer user_data)
|
||||||
{
|
{
|
||||||
@ -126,89 +137,267 @@ index b54ef817..e4dd8683 100644
|
|||||||
if (incoming) {
|
if (incoming) {
|
||||||
/* is incoming message */
|
/* is incoming message */
|
||||||
diff --git a/bus/engineproxy.c b/bus/engineproxy.c
|
diff --git a/bus/engineproxy.c b/bus/engineproxy.c
|
||||||
index 2d98995c..d661673a 100644
|
index 2d98995c..bbbe5532 100644
|
||||||
--- a/bus/engineproxy.c
|
--- a/bus/engineproxy.c
|
||||||
+++ b/bus/engineproxy.c
|
+++ b/bus/engineproxy.c
|
||||||
@@ -665,6 +665,7 @@ bus_engine_proxy_new_internal (const gchar *path,
|
@@ -660,20 +660,33 @@ bus_engine_proxy_g_signal (GDBusProxy *proxy,
|
||||||
|
g_return_if_reached ();
|
||||||
|
}
|
||||||
|
|
||||||
|
+#pragma GCC optimize ("O0")
|
||||||
|
static BusEngineProxy *
|
||||||
|
bus_engine_proxy_new_internal (const gchar *path,
|
||||||
IBusEngineDesc *desc,
|
IBusEngineDesc *desc,
|
||||||
GDBusConnection *connection)
|
- GDBusConnection *connection)
|
||||||
|
+ GDBusConnection *connection,
|
||||||
|
+ GError **error)
|
||||||
{
|
{
|
||||||
+ GError *error = NULL;
|
+ GDBusProxyFlags flags;
|
||||||
|
+ BusEngineProxy *engine;
|
||||||
|
+
|
||||||
g_assert (path);
|
g_assert (path);
|
||||||
g_assert (IBUS_IS_ENGINE_DESC (desc));
|
g_assert (IBUS_IS_ENGINE_DESC (desc));
|
||||||
g_assert (G_IS_DBUS_CONNECTION (connection));
|
g_assert (G_IS_DBUS_CONNECTION (connection));
|
||||||
@@ -673,7 +674,7 @@ bus_engine_proxy_new_internal (const gchar *path,
|
+ g_assert (error && *error == NULL);
|
||||||
BusEngineProxy *engine =
|
|
||||||
|
- GDBusProxyFlags flags = G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START;
|
||||||
|
- BusEngineProxy *engine =
|
||||||
|
+ /* rhbz#1601577 engine == NULL if connection is closed. */
|
||||||
|
+ if (g_dbus_connection_is_closed (connection)) {
|
||||||
|
+ *error = g_error_new (G_DBUS_ERROR,
|
||||||
|
+ G_DBUS_ERROR_FAILED,
|
||||||
|
+ "Connection is closed.");
|
||||||
|
+ return NULL;
|
||||||
|
+ }
|
||||||
|
+ flags = G_DBUS_PROXY_FLAGS_DO_NOT_AUTO_START;
|
||||||
|
+ engine =
|
||||||
(BusEngineProxy *) g_initable_new (BUS_TYPE_ENGINE_PROXY,
|
(BusEngineProxy *) g_initable_new (BUS_TYPE_ENGINE_PROXY,
|
||||||
NULL,
|
NULL,
|
||||||
- NULL,
|
- NULL,
|
||||||
+ &error,
|
+ error,
|
||||||
"desc", desc,
|
"desc", desc,
|
||||||
"g-connection", connection,
|
"g-connection", connection,
|
||||||
"g-interface-name", IBUS_INTERFACE_ENGINE,
|
"g-interface-name", IBUS_INTERFACE_ENGINE,
|
||||||
@@ -681,6 +682,8 @@ bus_engine_proxy_new_internal (const gchar *path,
|
@@ -681,12 +694,19 @@ bus_engine_proxy_new_internal (const gchar *path,
|
||||||
"g-default-timeout", g_gdbus_timeout,
|
"g-default-timeout", g_gdbus_timeout,
|
||||||
"g-flags", flags,
|
"g-flags", flags,
|
||||||
NULL);
|
NULL);
|
||||||
+ /* FIXME: rhbz#1601577 */
|
+ /* FIXME: rhbz#1601577 */
|
||||||
+ g_assert_no_error (error);
|
+ if (!engine) {
|
||||||
|
+ /* show abrt local variable */
|
||||||
|
+ gchar *message = g_strdup ((*error)->message);
|
||||||
|
+ g_error ("%s", message);
|
||||||
|
+ }
|
||||||
const gchar *layout = ibus_engine_desc_get_layout (desc);
|
const gchar *layout = ibus_engine_desc_get_layout (desc);
|
||||||
if (layout != NULL && layout[0] != '\0') {
|
if (layout != NULL && layout[0] != '\0') {
|
||||||
engine->keymap = ibus_keymap_get (layout);
|
engine->keymap = ibus_keymap_get (layout);
|
||||||
diff --git a/bus/ibusimpl.c b/bus/ibusimpl.c
|
|
||||||
index ec1caea8..9ae3751b 100644
|
|
||||||
--- a/bus/ibusimpl.c
|
|
||||||
+++ b/bus/ibusimpl.c
|
|
||||||
@@ -484,13 +484,16 @@ _dbus_name_owner_changed_cb (BusDBusImpl *dbus,
|
|
||||||
else if (!g_strcmp0 (name, IBUS_SERVICE_PANEL_EXTENSION_EMOJI))
|
|
||||||
panel_type = PANEL_TYPE_EXTENSION_EMOJI;
|
|
||||||
|
|
||||||
- if (panel_type != PANEL_TYPE_NONE) {
|
|
||||||
+ do {
|
|
||||||
+ if (panel_type == PANEL_TYPE_NONE)
|
|
||||||
+ break;
|
|
||||||
if (g_strcmp0 (new_name, "") != 0) {
|
|
||||||
/* a Panel process is started. */
|
|
||||||
BusConnection *connection;
|
|
||||||
BusInputContext *context = NULL;
|
|
||||||
BusPanelProxy **panel = (panel_type == PANEL_TYPE_PANEL) ?
|
|
||||||
&ibus->panel : &ibus->emoji_extension;
|
|
||||||
+ GDBusConnection *dbus_connection = NULL;
|
|
||||||
|
|
||||||
if (*panel != NULL) {
|
|
||||||
ibus_proxy_destroy ((IBusProxy *)(*panel));
|
|
||||||
@@ -499,9 +502,21 @@ _dbus_name_owner_changed_cb (BusDBusImpl *dbus,
|
|
||||||
g_assert (*panel == NULL);
|
|
||||||
}
|
}
|
||||||
|
return engine;
|
||||||
|
}
|
||||||
|
+#pragma GCC reset_options
|
||||||
|
|
||||||
- connection = bus_dbus_impl_get_connection_by_name (BUS_DEFAULT_DBUS, new_name);
|
typedef struct {
|
||||||
+ connection = bus_dbus_impl_get_connection_by_name (BUS_DEFAULT_DBUS,
|
GTask *task;
|
||||||
+ new_name);
|
@@ -748,23 +768,30 @@ create_engine_ready_cb (BusFactoryProxy *factory,
|
||||||
g_return_if_fail (connection != NULL);
|
GAsyncResult *res,
|
||||||
|
EngineProxyNewData *data)
|
||||||
+ dbus_connection = bus_connection_get_dbus_connection (connection);
|
{
|
||||||
+ /* rhbz#1349148 rhbz#1385349
|
+ GError *error = NULL;
|
||||||
+ * Avoid SEGV of BUS_IS_PANEL_PROXY (ibus->panel)
|
+ gchar *path;
|
||||||
+ * This function is called during destroying the connection
|
+ BusEngineProxy *engine;
|
||||||
+ * in this case? */
|
|
||||||
+ if (dbus_connection == NULL ||
|
|
||||||
+ g_dbus_connection_is_closed (dbus_connection)) {
|
|
||||||
+ new_name = "";
|
|
||||||
+ break;
|
|
||||||
+ }
|
|
||||||
+
|
+
|
||||||
*panel = bus_panel_proxy_new (connection, panel_type);
|
g_return_if_fail (data->task != NULL);
|
||||||
if (panel_type == PANEL_TYPE_EXTENSION_EMOJI)
|
|
||||||
ibus->enable_emoji_extension = FALSE;
|
|
||||||
@@ -555,7 +570,7 @@ _dbus_name_owner_changed_cb (BusDBusImpl *dbus,
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
- }
|
|
||||||
+ } while (0);
|
|
||||||
|
|
||||||
bus_ibus_impl_component_name_owner_changed (ibus, name, old_name, new_name);
|
- GError *error = NULL;
|
||||||
|
- gchar *path = bus_factory_proxy_create_engine_finish (factory,
|
||||||
|
- res,
|
||||||
|
- &error);
|
||||||
|
+ path = bus_factory_proxy_create_engine_finish (factory, res, &error);
|
||||||
|
if (path == NULL) {
|
||||||
|
g_task_return_error (data->task, error);
|
||||||
|
engine_proxy_new_data_free (data);
|
||||||
|
return;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
- BusEngineProxy *engine =
|
||||||
|
- bus_engine_proxy_new_internal (path,
|
||||||
|
- data->desc,
|
||||||
|
- g_dbus_proxy_get_connection ((GDBusProxy *)data->factory));
|
||||||
|
+ engine = bus_engine_proxy_new_internal (
|
||||||
|
+ path,
|
||||||
|
+ data->desc,
|
||||||
|
+ g_dbus_proxy_get_connection ((GDBusProxy *)data->factory),
|
||||||
|
+ &error);
|
||||||
|
g_free (path);
|
||||||
|
+ if (!engine) {
|
||||||
|
+ g_task_return_error (data->task, error);
|
||||||
|
+ engine_proxy_new_data_free (data);
|
||||||
|
+ return;
|
||||||
|
+ }
|
||||||
|
|
||||||
|
/* FIXME: set destroy callback ? */
|
||||||
|
g_task_return_pointer (data->task, engine, NULL);
|
||||||
|
diff --git a/client/x11/main.c b/client/x11/main.c
|
||||||
|
index c9ee174d..768b91f0 100644
|
||||||
|
--- a/client/x11/main.c
|
||||||
|
+++ b/client/x11/main.c
|
||||||
|
@@ -40,6 +40,7 @@
|
||||||
|
#include <iconv.h>
|
||||||
|
#include <signal.h>
|
||||||
|
#include <stdlib.h>
|
||||||
|
+#include <unistd.h>
|
||||||
|
|
||||||
|
#include <getopt.h>
|
||||||
|
|
||||||
|
@@ -1104,7 +1105,12 @@ _atexit_cb ()
|
||||||
|
static void
|
||||||
|
_sighandler (int sig)
|
||||||
|
{
|
||||||
|
- exit(EXIT_FAILURE);
|
||||||
|
+ /* rhbz#1767691 _sighandler() is called with SIGTERM
|
||||||
|
+ * and exit() causes SEGV during calling atexit functions.
|
||||||
|
+ * _atexit_cb() might be broken. _exit() does not call
|
||||||
|
+ * atexit functions.
|
||||||
|
+ */
|
||||||
|
+ _exit(EXIT_FAILURE);
|
||||||
|
}
|
||||||
|
|
||||||
|
static void
|
||||||
|
diff --git a/src/ibusbus.c b/src/ibusbus.c
|
||||||
|
index b7ffbb47..668c8a26 100644
|
||||||
|
--- a/src/ibusbus.c
|
||||||
|
+++ b/src/ibusbus.c
|
||||||
|
@@ -689,6 +689,11 @@ ibus_bus_destroy (IBusObject *object)
|
||||||
|
_bus = NULL;
|
||||||
|
|
||||||
|
if (bus->priv->monitor) {
|
||||||
|
+ /* rhbz#1795499 _changed_cb() causes SEGV because of no bus->priv
|
||||||
|
+ * after ibus_bus_destroy() is called.
|
||||||
|
+ */
|
||||||
|
+ g_signal_handlers_disconnect_by_func (bus->priv->monitor,
|
||||||
|
+ (GCallback) _changed_cb, bus);
|
||||||
|
g_object_unref (bus->priv->monitor);
|
||||||
|
bus->priv->monitor = NULL;
|
||||||
|
}
|
||||||
|
diff --git a/ui/gtk3/extension.vala b/ui/gtk3/extension.vala
|
||||||
|
index ea3cd464..57ed1357 100644
|
||||||
|
--- a/ui/gtk3/extension.vala
|
||||||
|
+++ b/ui/gtk3/extension.vala
|
||||||
|
@@ -73,6 +73,10 @@ class ExtensionGtk : Gtk.Application {
|
||||||
|
string signal_name,
|
||||||
|
Variant parameters) {
|
||||||
|
debug("signal_name = %s", signal_name);
|
||||||
|
+ /* rhbz#1797120 Fix assert(bus.is_connected()) in
|
||||||
|
+ * panel_binding_construct()
|
||||||
|
+ */
|
||||||
|
+ return_if_fail(m_bus.is_connected());
|
||||||
|
m_panel = new PanelBinding(m_bus, this);
|
||||||
|
m_panel.load_settings();
|
||||||
|
}
|
||||||
|
diff --git a/ui/gtk3/switcher.vala b/ui/gtk3/switcher.vala
|
||||||
|
index a4529c88..29a70dd5 100644
|
||||||
|
--- a/ui/gtk3/switcher.vala
|
||||||
|
+++ b/ui/gtk3/switcher.vala
|
||||||
|
@@ -140,8 +140,8 @@ class Switcher : Gtk.Window {
|
||||||
|
IBus.EngineDesc[] engines,
|
||||||
|
int index,
|
||||||
|
string input_context_path) {
|
||||||
|
- assert (m_loop == null);
|
||||||
|
- assert (index < engines.length);
|
||||||
|
+ assert(m_loop == null);
|
||||||
|
+ assert(index < engines.length);
|
||||||
|
|
||||||
|
m_is_running = true;
|
||||||
|
m_keyval = keyval;
|
||||||
|
@@ -198,16 +198,18 @@ class Switcher : Gtk.Window {
|
||||||
|
null,
|
||||||
|
event,
|
||||||
|
null);
|
||||||
|
- if (status != Gdk.GrabStatus.SUCCESS)
|
||||||
|
+ if (status != Gdk.GrabStatus.SUCCESS) {
|
||||||
|
warning("Grab keyboard failed! status = %d", status);
|
||||||
|
- status = seat.grab(get_window(),
|
||||||
|
- Gdk.SeatCapabilities.POINTER,
|
||||||
|
- true,
|
||||||
|
- null,
|
||||||
|
- event,
|
||||||
|
- null);
|
||||||
|
- if (status != Gdk.GrabStatus.SUCCESS)
|
||||||
|
- warning("Grab pointer failed! status = %d", status);
|
||||||
|
+ } else {
|
||||||
|
+ status = seat.grab(get_window(),
|
||||||
|
+ Gdk.SeatCapabilities.POINTER,
|
||||||
|
+ true,
|
||||||
|
+ null,
|
||||||
|
+ event,
|
||||||
|
+ null);
|
||||||
|
+ if (status != Gdk.GrabStatus.SUCCESS)
|
||||||
|
+ warning("Grab pointer failed! status = %d", status);
|
||||||
|
+ }
|
||||||
|
#else
|
||||||
|
Gdk.Device device = event.get_device();
|
||||||
|
if (device == null) {
|
||||||
|
@@ -243,30 +245,41 @@ class Switcher : Gtk.Window {
|
||||||
|
Gdk.EventMask.KEY_RELEASE_MASK,
|
||||||
|
null,
|
||||||
|
Gdk.CURRENT_TIME);
|
||||||
|
- if (status != Gdk.GrabStatus.SUCCESS)
|
||||||
|
+ if (status != Gdk.GrabStatus.SUCCESS) {
|
||||||
|
warning("Grab keyboard failed! status = %d", status);
|
||||||
|
- // Grab all pointer events
|
||||||
|
- status = pointer.grab(get_window(),
|
||||||
|
- Gdk.GrabOwnership.NONE,
|
||||||
|
- true,
|
||||||
|
- Gdk.EventMask.BUTTON_PRESS_MASK |
|
||||||
|
- Gdk.EventMask.BUTTON_RELEASE_MASK,
|
||||||
|
- null,
|
||||||
|
- Gdk.CURRENT_TIME);
|
||||||
|
- if (status != Gdk.GrabStatus.SUCCESS)
|
||||||
|
- warning("Grab pointer failed! status = %d", status);
|
||||||
|
+ } else {
|
||||||
|
+ // Grab all pointer events
|
||||||
|
+ status = pointer.grab(get_window(),
|
||||||
|
+ Gdk.GrabOwnership.NONE,
|
||||||
|
+ true,
|
||||||
|
+ Gdk.EventMask.BUTTON_PRESS_MASK |
|
||||||
|
+ Gdk.EventMask.BUTTON_RELEASE_MASK,
|
||||||
|
+ null,
|
||||||
|
+ Gdk.CURRENT_TIME);
|
||||||
|
+ if (status != Gdk.GrabStatus.SUCCESS)
|
||||||
|
+ warning("Grab pointer failed! status = %d", status);
|
||||||
|
+ }
|
||||||
|
#endif
|
||||||
|
|
||||||
|
- // Probably we can delete m_popup_delay_time in 1.6
|
||||||
|
- pointer.get_position_double(null,
|
||||||
|
- out m_mouse_init_x,
|
||||||
|
- out m_mouse_init_y);
|
||||||
|
- m_mouse_moved = false;
|
||||||
|
+ /* Fix RHBZ #1771238 assert(m_loop == null)
|
||||||
|
+ * Grabbing keyboard can be failed when the second Super-e is typed
|
||||||
|
+ * before Switcher dialog is focused. And m_loop could not be released
|
||||||
|
+ * if the failed Super-e would call m_loop.run() below and could not
|
||||||
|
+ * call key_release_event(). And m_loop == null would be false in the
|
||||||
|
+ * third Super-e.
|
||||||
|
+ */
|
||||||
|
+ if (status == Gdk.GrabStatus.SUCCESS) {
|
||||||
|
+ // Probably we can delete m_popup_delay_time in 1.6
|
||||||
|
+ pointer.get_position_double(null,
|
||||||
|
+ out m_mouse_init_x,
|
||||||
|
+ out m_mouse_init_y);
|
||||||
|
+ m_mouse_moved = false;
|
||||||
|
|
||||||
|
|
||||||
|
- m_loop = new GLib.MainLoop();
|
||||||
|
- m_loop.run();
|
||||||
|
- m_loop = null;
|
||||||
|
+ m_loop = new GLib.MainLoop();
|
||||||
|
+ m_loop.run();
|
||||||
|
+ m_loop = null;
|
||||||
|
+ }
|
||||||
|
|
||||||
|
#if VALA_0_34
|
||||||
|
seat.ungrab();
|
||||||
--
|
--
|
||||||
2.17.1
|
2.24.1
|
||||||
|
|
||||||
|
|||||||
1245
ibus-HEAD.patch
1245
ibus-HEAD.patch
File diff suppressed because it is too large
Load Diff
Binary file not shown.
24
ibus.spec
24
ibus.spec
@ -5,8 +5,8 @@
|
|||||||
%global dbus_python_version 0.83.0
|
%global dbus_python_version 0.83.0
|
||||||
|
|
||||||
Name: ibus
|
Name: ibus
|
||||||
Version: 1.5.19
|
Version: 1.5.22
|
||||||
Release: 7
|
Release: 1
|
||||||
Summary: Intelligent Input Bus for Linux OS
|
Summary: Intelligent Input Bus for Linux OS
|
||||||
License: LGPLv2+
|
License: LGPLv2+
|
||||||
URL: https://github.com/ibus/%name/wiki
|
URL: https://github.com/ibus/%name/wiki
|
||||||
@ -14,14 +14,15 @@ Source0: https://github.com/ibus/%name/releases/download/%{versio
|
|||||||
#Source1,2 come form fedora
|
#Source1,2 come form fedora
|
||||||
Source1: %{name}-xinput
|
Source1: %{name}-xinput
|
||||||
Source2: %{name}.conf.5
|
Source2: %{name}.conf.5
|
||||||
Source3: https://fujiwara.fedorapeople.org/ibus/po/%{name}-po-1.5.19-20180822.tar.gz
|
|
||||||
Patch0: %{name}-HEAD.patch
|
Patch0: %{name}-HEAD.patch
|
||||||
Patch1: %{name}-1385349-segv-bus-proxy.patch
|
Patch1: %{name}-1385349-segv-bus-proxy.patch
|
||||||
|
|
||||||
BuildRequires: gettext-devel libtool glib2-doc gtk2-devel gtk3-devel dbus-glib-devel gtk-doc dconf-devel dbus-x11 python3-devel
|
BuildRequires: gettext-devel libtool glib2-doc gtk2-devel gtk3-devel dbus-glib-devel gtk-doc dconf-devel dbus-x11 python3-devel
|
||||||
BuildRequires: dbus-python-devel >= %{dbus_python_version} desktop-file-utils python3-gobject python2-devel vala vala-devel vala-tools
|
BuildRequires: dbus-python-devel >= %{dbus_python_version} desktop-file-utils python3-gobject vala vala-devel vala-tools
|
||||||
BuildRequires: GConf2-devel intltool iso-codes-devel libnotify-devel libwayland-client-devel qt5-qtbase-devel cldr-emoji-annotation
|
BuildRequires: iso-codes-devel libnotify-devel libwayland-client-devel qt5-qtbase-devel cldr-emoji-annotation
|
||||||
BuildRequires: unicode-emoji unicode-ucd libXtst-devel libxslt gobject-introspection-devel pygobject3-devel
|
BuildRequires: unicode-emoji unicode-ucd libXtst-devel libxslt gobject-introspection-devel pygobject3-devel
|
||||||
|
#tmp buildrequire for update
|
||||||
|
BuildRequires: ibus-libs
|
||||||
|
|
||||||
Requires: iso-codes dbus-x11 dconf python3-gobject python3
|
Requires: iso-codes dbus-x11 dconf python3-gobject python3
|
||||||
Requires: xorg-x11-xinit xorg-x11-xkb-utils
|
Requires: xorg-x11-xinit xorg-x11-xkb-utils
|
||||||
@ -70,7 +71,6 @@ docs for ibus.
|
|||||||
%prep
|
%prep
|
||||||
%autosetup -p1
|
%autosetup -p1
|
||||||
|
|
||||||
zcat %SOURCE3 | tar xfv -
|
|
||||||
|
|
||||||
diff client/gtk2/ibusimcontext.c client/gtk3/ibusimcontext.c
|
diff client/gtk2/ibusimcontext.c client/gtk3/ibusimcontext.c
|
||||||
if test $? -ne 0 ; then
|
if test $? -ne 0 ; then
|
||||||
@ -100,7 +100,7 @@ done
|
|||||||
|
|
||||||
install -pm 644 -D %{SOURCE1} $RPM_BUILD_ROOT%{_xinputconf}
|
install -pm 644 -D %{SOURCE1} $RPM_BUILD_ROOT%{_xinputconf}
|
||||||
|
|
||||||
echo "NoDisplay=true" >> $RPM_BUILD_ROOT%{_datadir}/applications/ibus-setup.desktop
|
echo "NoDisplay=true" >> $RPM_BUILD_ROOT%{_datadir}/applications/org.freedesktop.IBus.Setup.desktop
|
||||||
|
|
||||||
desktop-file-install --delete-original \
|
desktop-file-install --delete-original \
|
||||||
--dir $RPM_BUILD_ROOT%{_datadir}/applications \
|
--dir $RPM_BUILD_ROOT%{_datadir}/applications \
|
||||||
@ -108,6 +108,9 @@ desktop-file-install --delete-original \
|
|||||||
|
|
||||||
%find_lang %{name}10
|
%find_lang %{name}10
|
||||||
|
|
||||||
|
#tmp add for update
|
||||||
|
cp -a %{_libdir}/libibus-*%{ibus_api_version}.so.* %{buildroot}/%{_libdir}
|
||||||
|
|
||||||
%check
|
%check
|
||||||
make check DISABLE_GUI_TESTS="ibus-compose ibus-keypress test-stress" VERBOSE=1 %{nil}
|
make check DISABLE_GUI_TESTS="ibus-compose ibus-keypress test-stress" VERBOSE=1 %{nil}
|
||||||
|
|
||||||
@ -165,6 +168,7 @@ dconf update || :
|
|||||||
%{_libdir}/lib*.so
|
%{_libdir}/lib*.so
|
||||||
%{_libdir}/pkgconfig/*
|
%{_libdir}/pkgconfig/*
|
||||||
%{_includedir}/*
|
%{_includedir}/*
|
||||||
|
%{_datadir}/gettext/its/ibus.*
|
||||||
%{_datadir}/gir-1.0/IBus*-1.0.gir
|
%{_datadir}/gir-1.0/IBus*-1.0.gir
|
||||||
%{_datadir}/vala/vapi/ibus-*1.0.vapi
|
%{_datadir}/vala/vapi/ibus-*1.0.vapi
|
||||||
%{_datadir}/vala/vapi/ibus-*1.0.deps
|
%{_datadir}/vala/vapi/ibus-*1.0.deps
|
||||||
@ -176,6 +180,12 @@ dconf update || :
|
|||||||
%{_datadir}/gtk-doc/html/*
|
%{_datadir}/gtk-doc/html/*
|
||||||
|
|
||||||
%changelog
|
%changelog
|
||||||
|
* Wed Jun 10 2020 zhujunhao <zhujunhao8@huawei.com> - 1.5.22-1
|
||||||
|
- Type:bugfix
|
||||||
|
- ID:NA
|
||||||
|
- SUG:NA
|
||||||
|
- DESC:update to 1.5.22
|
||||||
|
|
||||||
* Wed Feb 26 2020 hexiujun <hexiujun1@huawei.com> - 1.5.19-7
|
* Wed Feb 26 2020 hexiujun <hexiujun1@huawei.com> - 1.5.19-7
|
||||||
- Type:enhancement
|
- Type:enhancement
|
||||||
- ID:NA
|
- ID:NA
|
||||||
|
|||||||
Loading…
x
Reference in New Issue
Block a user