summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorBeniamino Galvani <bgalvani@redhat.com>2016-05-13 10:26:56 +0200
committerBeniamino Galvani <bgalvani@redhat.com>2016-05-13 14:51:10 +0200
commit8e60431a7d6fc4c5545e58464e10b9131cbd3e6a (patch)
tree2b2f1b25973d99192f60930fe19d0365ed1b97bf
parent3158a3957604af2bd5d4736c5c9a74b9a3f4cb5d (diff)
downloadnetwork-manager-applet-8e60431a7d6fc4c5545e58464e10b9131cbd3e6a.tar.gz
wireless-security: fix failed assertion in default_filter_privkey()
The @out_encrypted argument of nm_utils_file_is_private_key() must be initialized to FALSE. When using libnm-glib, keep the value set to TRUE since encryption detection is not supported for all file types. This fixes the following: libnm-CRITICAL **: nm_utils_file_is_private_key: assertion 'out_encrypted == NULL || *out_encrypted == FALSE' failed https://bugzilla.gnome.org/show_bug.cgi?id=763578
-rw-r--r--src/wireless-security/eap-method.c4
1 files changed, 3 insertions, 1 deletions
diff --git a/src/wireless-security/eap-method.c b/src/wireless-security/eap-method.c
index e771b58a..3c7b4544 100644
--- a/src/wireless-security/eap-method.c
+++ b/src/wireless-security/eap-method.c
@@ -393,7 +393,7 @@ default_filter_privkey (const GtkFileFilterInfo *filter_info, gpointer user_data
const char *extensions[] = { ".der", ".pem", ".p12", ".key", NULL };
#endif
gboolean require_encrypted = !!user_data;
- gboolean is_encrypted = TRUE;
+ gboolean is_encrypted;
if (!filter_info->filename)
return FALSE;
@@ -402,10 +402,12 @@ default_filter_privkey (const GtkFileFilterInfo *filter_info, gpointer user_data
if (!file_has_extension (filter_info->filename, extensions))
return FALSE;
+ is_encrypted = TRUE;
if ( !file_is_der_or_pem (filter_info->filename, TRUE, &is_encrypted)
&& !nm_utils_file_is_pkcs12 (filter_info->filename))
return FALSE;
#elif defined (LIBNM_BUILD)
+ is_encrypted = FALSE;
if (!nm_utils_file_is_private_key (filter_info->filename, &is_encrypted))
return FALSE;
#else