diff options
Diffstat (limited to 'packages')
-rw-r--r-- | packages/packagekit/files/racing_condition.patch | 244 | ||||
-rw-r--r-- | packages/packagekit/packagekit_git.bb | 3 |
2 files changed, 246 insertions, 1 deletions
diff --git a/packages/packagekit/files/racing_condition.patch b/packages/packagekit/files/racing_condition.patch new file mode 100644 index 0000000000..ae127da861 --- /dev/null +++ b/packages/packagekit/files/racing_condition.patch @@ -0,0 +1,244 @@ +diff --git a/backends/opkg/pk-backend-opkg.c b/backends/opkg/pk-backend-opkg.c +index a7b5244..9e71b90 100644 +--- a/backends/opkg/pk-backend-opkg.c ++++ b/backends/opkg/pk-backend-opkg.c +@@ -20,6 +20,7 @@ + * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA 02111-1307, USA. + */ + ++#include <unistd.h> + #include <gmodule.h> + #include <glib.h> + #include <string.h> +@@ -47,6 +48,22 @@ typedef struct { + PkBackend *backend; + } SearchParams; + ++static int __opkg_lock__ = 0; ++static inline void opkg_thread_lock() ++{ ++ while ( __opkg_lock__ ) ++ usleep(500); ++ __opkg_lock__ = 1; ++} ++ ++static inline void opkg_thread_unlock() ++{ ++ if (! __opkg_lock__ ) { ++ pk_debug("Warning: opkg_thread_unlock without lock before!!\n"); ++ } ++ __opkg_lock__ = 0; ++} ++ + static void + opkg_unknown_error (PkBackend *backend, gint error_code, gchar *failed_cmd) + { +@@ -236,6 +253,7 @@ backend_refresh_cache_thread (PkBackend *backend) + opkg_unknown_error (backend, ret, "Refreshing cache"); + } + pk_backend_finished (backend); ++ opkg_thread_unlock(); + + return (ret == 0); + } +@@ -246,6 +264,7 @@ backend_refresh_cache_thread (PkBackend *backend) + static void + backend_refresh_cache (PkBackend *backend, gboolean force) + { ++ opkg_thread_lock(); + pk_backend_set_status (backend, PK_STATUS_ENUM_REFRESH_CACHE); + pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); + +@@ -260,15 +279,26 @@ backend_refresh_cache (PkBackend *backend, gboolean force) + static void + pk_opkg_package_list_cb (opkg_t *opkg, opkg_package_t *pkg, void *data) + { +- SearchParams *params = (SearchParams*) data; ++ SearchParams *params; + gchar *uid; + gchar *haystack; + gint status, match; +- PkFilterEnum filters = params->filters; ++ PkFilterEnum filters; + ++ if (!opkg) { ++ pk_debug("opkg is NULL!!\n"); ++ return; ++ } ++ if (!data) { ++ pk_debug("data is NULL!!\n"); ++ return; ++ } + if (!pkg->name) + return; + ++ params = (SearchParams*) data; ++ filters = params->filters; ++ + switch (params->search_type) + { + case SEARCH_NAME: +@@ -336,6 +366,7 @@ backend_search_thread (PkBackend *backend) + opkg_list_packages (opkg, pk_opkg_package_list_cb, params); + + pk_backend_finished (params->backend); ++ opkg_thread_unlock(); + + g_free (params->needle); + g_free (params); +@@ -348,6 +379,7 @@ backend_search_name (PkBackend *backend, PkFilterEnum filters, const gchar *sear + { + SearchParams *params; + ++ opkg_thread_lock(); + + pk_backend_set_status (backend, PK_STATUS_ENUM_QUERY); + pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); +@@ -370,6 +402,7 @@ backend_search_description (PkBackend *backend, PkFilterEnum filters, const gcha + { + SearchParams *params; + ++ opkg_thread_lock(); + + pk_backend_set_status (backend, PK_STATUS_ENUM_QUERY); + pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); +@@ -389,6 +422,7 @@ backend_search_group (PkBackend *backend, PkFilterEnum filters, const gchar *sea + { + SearchParams *params; + ++ opkg_thread_lock(); + + pk_backend_set_status (backend, PK_STATUS_ENUM_QUERY); + pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); +@@ -431,12 +465,15 @@ backend_install_packages_thread (PkBackend *backend) + } + + pk_backend_finished (backend); ++ opkg_thread_unlock(); + return (err == 0); + } + + static void + backend_install_packages (PkBackend *backend, gchar **package_ids) + { ++ opkg_thread_lock(); ++ + pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); + pk_backend_set_status (backend, PK_STATUS_ENUM_INSTALL); + +@@ -493,6 +530,7 @@ backend_remove_packages_thread (PkBackend *backend) + } + + pk_backend_finished (backend); ++ opkg_thread_unlock(); + return (err == 0); + } + +@@ -500,6 +538,8 @@ static void + backend_remove_packages (PkBackend *backend, gchar **package_ids, gboolean allow_deps, gboolean autoremove) + { + gpointer *params; ++ ++ opkg_thread_lock(); + + pk_backend_set_status (backend, PK_STATUS_ENUM_REMOVE); + pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); +@@ -540,12 +580,15 @@ backend_update_system_thread (PkBackend *backend) + opkg_unknown_error (backend, err, "Upgrading system"); + + pk_backend_finished (backend); ++ opkg_thread_unlock(); + return (err != 0); + } + + static void + backend_update_system (PkBackend *backend) + { ++ opkg_thread_lock(); ++ + pk_backend_set_status (backend, PK_STATUS_ENUM_UPDATE); + pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); + +@@ -571,6 +614,7 @@ backend_update_package_thread (PkBackend *backend) + "Package not found"); + pk_package_id_free (pi); + pk_backend_finished (backend); ++ opkg_thread_unlock(); + return FALSE; + } + +@@ -581,6 +625,7 @@ backend_update_package_thread (PkBackend *backend) + + pk_package_id_free (pi); + pk_backend_finished (backend); ++ opkg_thread_unlock(); + return (err != 0); + } + +@@ -589,10 +634,12 @@ backend_update_packages (PkBackend *backend, gchar **package_ids) + { + gint i; + +- pk_backend_set_status (backend, PK_STATUS_ENUM_UPDATE); +- pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); +- + for (i = 0; package_ids[i]; i++) { ++ ++ opkg_thread_lock(); ++ ++ pk_backend_set_status (backend, PK_STATUS_ENUM_UPDATE); ++ pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); + pk_backend_set_string (backend, "pkgid", package_ids[i]); + pk_backend_thread_create (backend, backend_update_package_thread); + } +@@ -626,12 +673,14 @@ backend_get_updates_thread (PkBackend *backend) + { + opkg_list_upgradable_packages (opkg, pk_opkg_list_upgradable_cb, backend); + pk_backend_finished (backend); ++ opkg_thread_unlock(); + return TRUE; + } + + static void + backend_get_updates (PkBackend *backend, PkFilterEnum filters) + { ++ opkg_thread_lock(); + pk_backend_set_status (backend, PK_STATUS_ENUM_UPDATE); + pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); + +@@ -672,6 +721,7 @@ backend_get_details_thread (PkBackend *backend) + { + pk_backend_error_code (backend, PK_ERROR_ENUM_PACKAGE_ID_INVALID, "invalid package id"); + pk_backend_finished (backend); ++ opkg_thread_unlock(); + return FALSE; + } + +@@ -683,6 +733,7 @@ backend_get_details_thread (PkBackend *backend) + { + pk_backend_error_code (backend, PK_ERROR_ENUM_PACKAGE_NOT_FOUND, NULL); + pk_backend_finished (backend); ++ opkg_thread_unlock(); + return FALSE; + } + +@@ -698,15 +749,18 @@ backend_get_details_thread (PkBackend *backend) + } + + pk_backend_details (backend, newid, NULL, group, pkg->description, pkg->url, pkg->size); ++ pk_backend_finished (backend); ++ opkg_thread_unlock(); ++ + g_free (newid); + opkg_package_free(pkg); +- pk_backend_finished (backend); + return TRUE; + } + + static void + backend_get_details (PkBackend *backend, const gchar *package_id) + { ++ opkg_thread_lock(); + pk_backend_set_percentage (backend, PK_BACKEND_PERCENTAGE_INVALID); + pk_backend_thread_create (backend, backend_get_details_thread); + } diff --git a/packages/packagekit/packagekit_git.bb b/packages/packagekit/packagekit_git.bb index e3d3335ce3..17871a3ab6 100644 --- a/packages/packagekit/packagekit_git.bb +++ b/packages/packagekit/packagekit_git.bb @@ -5,13 +5,14 @@ LICENSE = "GPL" DEPENDS = "dbus (>= 1.1.1) dbus-glib glib-2.0 sqlite3 opkg intltool intltool-native (>= 0.37.1)" RDEPENDS_${PN} = "opkg" PV = "0.2.3+gitr${SRCREV}" -PR = "r15" +PR = "r16" SRC_URI = "git://anongit.freedesktop.org/git/packagekit;protocol=git \ file://disable-docbook2man.patch;patch=1 \ file://repository-ping.patch;patch=1 \ file://force_depends.patch;patch=1 \ file://remove_search_memory_leak.patch;patch=1 \ + file://racing_condition.patch;patch=1 \ " |