Openembedded Core Discussions
 help / color / mirror / Atom feed
From: Alexander Kanavin <alex.kanavin@gmail.com>
To: openembedded-core@lists.openembedded.org
Subject: [PATCH 23/24] libsecret: upgrade 0.19.1 -> 0.20.0
Date: Mon, 20 Jan 2020 18:25:09 +0100	[thread overview]
Message-ID: <20200120172510.22648-23-alex.kanavin@gmail.com> (raw)
In-Reply-To: <20200120172510.22648-1-alex.kanavin@gmail.com>

Add a backported patch to fix musl builds.

Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
---
 ...ection-Rename-internal-functions-to-.patch | 144 ++++++++++++++++++
 ...ibsecret_0.19.1.bb => libsecret_0.20.0.bb} |   5 +-
 2 files changed, 147 insertions(+), 2 deletions(-)
 create mode 100644 meta/recipes-gnome/libsecret/libsecret/0001-secret-file-collection-Rename-internal-functions-to-.patch
 rename meta/recipes-gnome/libsecret/{libsecret_0.19.1.bb => libsecret_0.20.0.bb} (81%)

diff --git a/meta/recipes-gnome/libsecret/libsecret/0001-secret-file-collection-Rename-internal-functions-to-.patch b/meta/recipes-gnome/libsecret/libsecret/0001-secret-file-collection-Rename-internal-functions-to-.patch
new file mode 100644
index 00000000000..269c12b0107
--- /dev/null
+++ b/meta/recipes-gnome/libsecret/libsecret/0001-secret-file-collection-Rename-internal-functions-to-.patch
@@ -0,0 +1,144 @@
+From 59bcb169c4777bb3b1a50fee2ae20e4e8574b749 Mon Sep 17 00:00:00 2001
+From: Daiki Ueno <dueno@src.gnome.org>
+Date: Tue, 14 Jan 2020 10:09:29 +0100
+Subject: [PATCH] secret-file-collection: Rename internal functions to avoid
+ conflicts
+
+As encrypt() is a standard POSIX function, its declaration had
+conflict with our own.  Let's be more verbose about naming of the
+internal crypto functions.
+
+Fixes #35.
+
+Upstream-Status: Backport [https://github.com/GNOME/libsecret/commit/cf21ad50b62f7c8e4b22ef374f0a73290a99bdb8]
+Signed-off-by: Alexander Kanavin <alex.kanavin@gmail.com>
+---
+ libsecret/secret-file-collection.c | 38 +++++++++++++++---------------
+ 1 file changed, 19 insertions(+), 19 deletions(-)
+
+diff --git a/libsecret/secret-file-collection.c b/libsecret/secret-file-collection.c
+index 79863ea..8ffb542 100644
+--- a/libsecret/secret-file-collection.c
++++ b/libsecret/secret-file-collection.c
+@@ -68,7 +68,7 @@ enum {
+ };
+ 
+ static gboolean
+-derive (SecretFileCollection *self)
++do_derive_key (SecretFileCollection *self)
+ {
+ 	const gchar *password;
+ 	gsize n_password;
+@@ -93,9 +93,9 @@ derive (SecretFileCollection *self)
+ }
+ 
+ static gboolean
+-calculate_mac (SecretFileCollection *self,
+-	       const guint8 *value, gsize n_value,
+-	       guint8 *buffer)
++do_calculate_mac (SecretFileCollection *self,
++		  const guint8 *value, gsize n_value,
++		  guint8 *buffer)
+ {
+ 	gcry_mac_hd_t hd;
+ 	gcry_error_t gcry;
+@@ -130,9 +130,9 @@ calculate_mac (SecretFileCollection *self,
+ }
+ 
+ static gboolean
+-decrypt (SecretFileCollection *self,
+-	 guint8 *data,
+-	 gsize n_data)
++do_decrypt (SecretFileCollection *self,
++	    guint8 *data,
++	    gsize n_data)
+ {
+ 	gcry_cipher_hd_t hd;
+ 	gcry_error_t gcry;
+@@ -164,9 +164,9 @@ decrypt (SecretFileCollection *self,
+ }
+ 
+ static gboolean
+-encrypt (SecretFileCollection *self,
+-	 guint8 *data,
+-	 gsize n_data)
++do_encrypt (SecretFileCollection *self,
++	    guint8 *data,
++	    gsize n_data)
+ {
+ 	gcry_cipher_hd_t hd;
+ 	gcry_error_t gcry;
+@@ -311,7 +311,7 @@ on_load_contents (GObject *source_object,
+ 			self->modified = g_date_time_new_now_utc ();
+ 			self->usage_count = 0;
+ 
+-			if (!derive (self)) {
++			if (!do_derive_key (self)) {
+ 				g_task_return_new_error (task,
+ 							 SECRET_ERROR,
+ 							 SECRET_ERROR_PROTOCOL,
+@@ -375,7 +375,7 @@ on_load_contents (GObject *source_object,
+ 	g_assert (n_data == salt_size);
+ 
+ 	self->salt = g_bytes_new (data, n_data);
+-	if (!derive (self)) {
++	if (!do_derive_key (self)) {
+ 		g_task_return_new_error (task,
+ 					 SECRET_ERROR,
+ 					 SECRET_ERROR_PROTOCOL,
+@@ -442,7 +442,7 @@ hash_attributes (SecretFileCollection *self,
+ 		GVariant *variant;
+ 
+ 		value = g_hash_table_lookup (attributes, l->data);
+-		if (!calculate_mac (self, (guint8 *)value, strlen (value), buffer)) {
++		if (!do_calculate_mac (self, (guint8 *)value, strlen (value), buffer)) {
+ 			g_list_free (keys);
+ 			return NULL;
+ 		}
+@@ -485,7 +485,7 @@ hashed_attributes_match (SecretFileCollection *self,
+ 			return FALSE;
+ 		}
+ 
+-		if (!calculate_mac (self, value, strlen ((char *)value), buffer)) {
++		if (!do_calculate_mac (self, value, strlen ((char *)value), buffer)) {
+ 			g_variant_unref (hashed_attribute);
+ 			return FALSE;
+ 		}
+@@ -584,7 +584,7 @@ secret_file_collection_replace (SecretFileCollection *self,
+ 	g_variant_store (serialized_item, data);
+ 	g_variant_unref (serialized_item);
+ 	memset (data + n_data, n_padded - n_data, n_padded - n_data);
+-	if (!encrypt (self, data, n_padded)) {
++	if (!do_encrypt (self, data, n_padded)) {
+ 		egg_secure_free (data);
+ 		g_set_error (error,
+ 			     SECRET_ERROR,
+@@ -593,8 +593,8 @@ secret_file_collection_replace (SecretFileCollection *self,
+ 		return FALSE;
+ 	}
+ 
+-	if (!calculate_mac (self, data, n_padded + IV_SIZE,
+-			    data + n_padded + IV_SIZE)) {
++	if (!do_calculate_mac (self, data, n_padded + IV_SIZE,
++			       data + n_padded + IV_SIZE)) {
+ 		egg_secure_free (data);
+ 		g_set_error (error,
+ 			     SECRET_ERROR,
+@@ -681,7 +681,7 @@ _secret_file_item_decrypt (GVariant *encrypted,
+ 	}
+ 	n_padded -= IV_SIZE + MAC_SIZE;
+ 
+-	if (!calculate_mac (collection, data, n_padded + IV_SIZE, mac)) {
++	if (!do_calculate_mac (collection, data, n_padded + IV_SIZE, mac)) {
+ 		egg_secure_free (data);
+ 		g_set_error (error,
+ 			     SECRET_ERROR,
+@@ -699,7 +699,7 @@ _secret_file_item_decrypt (GVariant *encrypted,
+ 		return FALSE;
+ 	}
+ 
+-	if (!decrypt (collection, data, n_padded)) {
++	if (!do_decrypt (collection, data, n_padded)) {
+ 		egg_secure_free (data);
+ 		g_set_error (error,
+ 			     SECRET_ERROR,
diff --git a/meta/recipes-gnome/libsecret/libsecret_0.19.1.bb b/meta/recipes-gnome/libsecret/libsecret_0.20.0.bb
similarity index 81%
rename from meta/recipes-gnome/libsecret/libsecret_0.19.1.bb
rename to meta/recipes-gnome/libsecret/libsecret_0.20.0.bb
index 3da2a4acac1..aa3dfab581d 100644
--- a/meta/recipes-gnome/libsecret/libsecret_0.19.1.bb
+++ b/meta/recipes-gnome/libsecret/libsecret_0.20.0.bb
@@ -13,8 +13,9 @@ DEPENDS += "glib-2.0 libgcrypt gettext-native"
 
 PACKAGECONFIG[manpages] = "--enable-manpages, --disable-manpages, libxslt-native xmlto-native"
 
-SRC_URI[archive.md5sum] = "ea673119c00570d6434f8fd3636f1eb8"
-SRC_URI[archive.sha256sum] = "8583e10179456ae2c83075d95455f156dc08db6278b32bf4bd61819335a30e3a"
+SRC_URI += " file://0001-secret-file-collection-Rename-internal-functions-to-.patch"
+SRC_URI[archive.md5sum] = "335750caeed47f50496b3b0e6a1875ff"
+SRC_URI[archive.sha256sum] = "f1187370b453106af878e30c284a121ba0c513da8bb4170b329d66e250bdae43"
 
 # http://errors.yoctoproject.org/Errors/Details/20228/
 ARM_INSTRUCTION_SET_armv4 = "arm"
-- 
2.17.1



  parent reply	other threads:[~2020-01-20 17:25 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-01-20 17:24 [PATCH 01/24] python3: update to 3.8.1 Alexander Kanavin
2020-01-20 17:24 ` [PATCH 02/24] python3: do not compile .pyc in parallel during do_install() Alexander Kanavin
2020-01-20 17:24 ` [PATCH 03/24] python3: correctly process ptest output with sed Alexander Kanavin
2020-01-20 17:24 ` [PATCH 04/24] gstreamer1.0-python: add a patch to fix python 3.8 builds Alexander Kanavin
2020-01-20 17:24 ` [PATCH 05/24] nss: update to 3.49.1 Alexander Kanavin
2020-01-21 12:04   ` Andreas Müller
2020-01-21 12:47     ` Richard Purdie
2020-01-21 19:35       ` Khem Raj
2020-01-21 20:22         ` Richard Purdie
2020-01-21 22:02           ` Khem Raj
2020-01-22  8:13             ` Khem Raj
2020-01-20 17:24 ` [PATCH 06/24] libcap: update to 2.31 Alexander Kanavin
2020-01-20 17:24 ` [PATCH 07/24] selftest: check maintainers.inc for entries without recipes Alexander Kanavin
2020-01-20 17:24 ` [PATCH 08/24] core-image-sato-sdk-ptest: do not pull in ptest-pkgs Alexander Kanavin
2020-01-20 17:24 ` [PATCH 09/24] ptests: exclude mdadm from ptesting Alexander Kanavin
2020-01-20 17:24 ` [PATCH 10/24] ptests: exclude lttng-tools Alexander Kanavin
2020-01-20 17:37   ` Jonathan Rajotte-Julien
2020-01-20 17:44     ` Alexander Kanavin
2020-01-24 12:47     ` Alexander Kanavin
2020-01-24 15:35       ` Jonathan Rajotte-Julien
2020-01-20 22:46   ` Richard Purdie
2020-01-20 17:24 ` [PATCH 11/24] openssh: applied upstream fix for "cert not yet valid" test Alexander Kanavin
2020-01-20 17:24 ` [PATCH 12/24] openssh: explicitly skip unit tests Alexander Kanavin
2020-01-20 17:24 ` [PATCH 13/24] parted: fix more ptests Alexander Kanavin
2020-01-20 17:25 ` [PATCH 14/24] qemu.inc: add vfat to MACHINE_FEATURES Alexander Kanavin
2020-01-20 17:25 ` [PATCH 15/24] rt-tests: exclude another development version (1.6) Alexander Kanavin
2020-01-20 17:25 ` [PATCH 16/24] btrfs-tools: upgrade 5.4 -> 5.4.1 Alexander Kanavin
2020-01-20 17:25 ` [PATCH 17/24] libpipeline: upgrade 1.5.1 -> 1.5.2 Alexander Kanavin
2020-01-20 17:25 ` [PATCH 18/24] msmtp: upgrade 1.8.6 -> 1.8.7 Alexander Kanavin
2020-01-20 17:25 ` [PATCH 19/24] ffmpeg: upgrade 4.2.1 -> 4.2.2 Alexander Kanavin
2020-01-20 17:25 ` [PATCH 20/24] epiphany: upgrade 3.34.2 -> 3.34.3.1 Alexander Kanavin
2020-01-20 17:25 ` [PATCH 21/24] libwebp: upgrade 1.0.3 -> 1.1.0 Alexander Kanavin
2020-01-20 17:25 ` [PATCH 22/24] createrepo-c: upgrade 0.15.4 -> 0.15.5 Alexander Kanavin
2020-01-20 17:25 ` Alexander Kanavin [this message]
2020-01-20 17:25 ` [PATCH 24/24] meson: update 0.52.1 -> 0.53.0 Alexander Kanavin
2020-01-21  8:24   ` Richard Purdie
2020-01-21 18:44   ` Khem Raj
2020-01-23 15:48     ` Alexander Kanavin
2020-01-23 15:52       ` Khem Raj
2020-01-20 17:32 ` ✗ patchtest: failure for "python3: update to 3.8.1..." and 23 more Patchwork

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20200120172510.22648-23-alex.kanavin@gmail.com \
    --to=alex.kanavin@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox