diff -ru a/src/pk-transaction.c b/src/pk-transaction.c --- a/src/pk-transaction.c 2010-01-28 10:00:41.000000000 +1300 +++ b/src/pk-transaction.c 2010-02-24 17:57:49.782480634 +1300 @@ -2459,7 +2459,7 @@ pk_transaction_accept_eula (PkTransaction *transaction, const gchar *eula_id, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -2630,7 +2630,7 @@ pk_transaction_download_packages (PkTransaction *transaction, gchar **package_ids, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; gchar *directory = NULL; gint retval; @@ -2721,7 +2721,7 @@ pk_transaction_get_categories (PkTransaction *transaction, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -2777,7 +2777,7 @@ gboolean recursive, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -2864,7 +2864,7 @@ pk_transaction_get_details (PkTransaction *transaction, gchar **package_ids, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -2941,7 +2941,7 @@ pk_transaction_get_distro_upgrades (PkTransaction *transaction, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -2991,7 +2991,7 @@ pk_transaction_get_files (PkTransaction *transaction, gchar **package_ids, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -3068,7 +3068,7 @@ pk_transaction_get_packages (PkTransaction *transaction, const gchar *filter, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -3143,7 +3143,7 @@ pk_transaction_get_repo_list (PkTransaction *transaction, const gchar *filter, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -3201,7 +3201,7 @@ gboolean recursive, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -3289,7 +3289,7 @@ DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -3366,7 +3366,7 @@ pk_transaction_get_updates (PkTransaction *transaction, const gchar *filter, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; GPtrArray *updates_cache; g_return_if_fail (PK_IS_TRANSACTION (transaction)); @@ -3523,7 +3523,7 @@ { gchar *full_paths_temp; gboolean ret; - GError *error; + GError *error = NULL; GError *error_local = NULL; PkServicePack *service_pack; gchar *content_type = NULL; @@ -3631,7 +3631,7 @@ gchar **package_ids, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -3709,7 +3709,7 @@ DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -3777,7 +3777,7 @@ pk_transaction_refresh_cache (PkTransaction *transaction, gboolean force, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -3829,7 +3829,7 @@ DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -3907,7 +3907,7 @@ DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -3967,7 +3967,7 @@ DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -4027,7 +4027,7 @@ gchar **packages, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *packages_temp; guint i; guint length; @@ -4115,7 +4115,7 @@ DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -4173,7 +4173,7 @@ gchar **values, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -4240,7 +4240,7 @@ gchar **values, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; guint i; g_return_if_fail (PK_IS_TRANSACTION (transaction)); @@ -4319,7 +4319,7 @@ gchar **values, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; guint i; g_return_if_fail (PK_IS_TRANSACTION (transaction)); @@ -4398,7 +4398,7 @@ gchar **values, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -4463,7 +4463,7 @@ void pk_transaction_set_locale (PkTransaction *transaction, const gchar *code, DBusGMethodInvocation *context) { - GError *error; + GError *error = NULL; gboolean ret; g_return_if_fail (PK_IS_TRANSACTION (transaction)); @@ -4558,7 +4558,7 @@ void pk_transaction_set_hints (PkTransaction *transaction, gchar **hints, DBusGMethodInvocation *context) { - GError *error; + GError *error = NULL; gboolean ret; guint i; gchar **sections; @@ -4614,7 +4614,7 @@ { gchar *full_paths_temp; gboolean ret; - GError *error; + GError *error = NULL; GError *error_local = NULL; PkServicePack *service_pack; gchar *content_type; @@ -4720,7 +4720,7 @@ pk_transaction_simulate_install_packages (PkTransaction *transaction, gchar **package_ids, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -4796,7 +4796,7 @@ pk_transaction_simulate_remove_packages (PkTransaction *transaction, gchar **package_ids, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -4872,7 +4872,7 @@ pk_transaction_simulate_update_packages (PkTransaction *transaction, gchar **package_ids, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -4948,7 +4948,7 @@ pk_transaction_update_packages (PkTransaction *transaction, gboolean only_trusted, gchar **package_ids, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; gchar *package_ids_temp; guint length; guint max_length; @@ -5024,7 +5024,7 @@ pk_transaction_update_system (PkTransaction *transaction, gboolean only_trusted, DBusGMethodInvocation *context) { gboolean ret; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL); @@ -5081,7 +5081,7 @@ { gboolean ret; PkProvidesEnum provides; - GError *error; + GError *error = NULL; g_return_if_fail (PK_IS_TRANSACTION (transaction)); g_return_if_fail (transaction->priv->tid != NULL);