Merge lp:~cyphermox/ubuntu/lucid/wpasupplicant/dbus-pkcs11-eapol into lp:ubuntu/lucid/wpasupplicant

Proposed by Mathieu Trudel-Lapierre
Status: Merged
Merge reported by: Mathieu Trudel-Lapierre
Merged at revision: not available
Proposed branch: lp:~cyphermox/ubuntu/lucid/wpasupplicant/dbus-pkcs11-eapol
Merge into: lp:ubuntu/lucid/wpasupplicant
Diff against target: 70 lines (+50/-0)
3 files modified
debian/changelog (+8/-0)
debian/patches/git_dbus_smartcard_eapol_d719934.patch (+41/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp:~cyphermox/ubuntu/lucid/wpasupplicant/dbus-pkcs11-eapol
Reviewer Review Type Date Requested Status
Ubuntu Sponsors Pending
Review via email: mp+87780@code.launchpad.net
To post a comment you must log in.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2010-03-07 01:11:22 +0000
+++ debian/changelog 2012-01-06 16:58:27 +0000
@@ -1,3 +1,11 @@
1wpasupplicant (0.6.9-3ubuntu3.1) lucid-proposed; urgency=low
2
3 * debian/patches/git_dbus_smartcard_eapol_d719934.patch: cherry-pick a patch
4 from Git to correctly re-initialize pointers in the WPA state machine to
5 avoid crashing when smartcard parameters change over DBUS (LP: #528087)
6
7 -- Mathieu Trudel-Lapierre <mathieu-tl@ubuntu.com> Fri, 06 Jan 2012 11:49:41 -0500
8
1wpasupplicant (0.6.9-3ubuntu3) lucid; urgency=low9wpasupplicant (0.6.9-3ubuntu3) lucid; urgency=low
210
3 * rebuild rest of main for armel armv7/thumb2 optimization;11 * rebuild rest of main for armel armv7/thumb2 optimization;
412
=== added file 'debian/patches/git_dbus_smartcard_eapol_d719934.patch'
--- debian/patches/git_dbus_smartcard_eapol_d719934.patch 1970-01-01 00:00:00 +0000
+++ debian/patches/git_dbus_smartcard_eapol_d719934.patch 2012-01-06 16:58:27 +0000
@@ -0,0 +1,41 @@
1From d7199342f0633b5ab147dca5b885530fe32ceaeb Mon Sep 17 00:00:00 2001
2From: David Smith <david.daniel.smith@gmail.com>
3Date: Wed, 11 Nov 2009 17:46:15 +0200
4Subject: [PATCH 1/1] Reset EAPOL pointer when handling DBus smartcard parameters
5
6Smartcard parameter update via DBus ended up re-initializing the EAPOL
7state machine without updating the pointer inside WPA state machine.
8This can trigger a segfault when EAP layer attempts to use the old
9reference. Fix this by re-initializing the pointer inside WPA state
10machine.
11---
12 wpa_supplicant/ctrl_iface_dbus_handlers.c | 4 ++++
13 1 files changed, 4 insertions(+), 0 deletions(-)
14
15diff --git a/wpa_supplicant/ctrl_iface_dbus_handlers.c b/wpa_supplicant/ctrl_iface_dbus_handlers.c
16index 2fd4e23..0e9d4af 100644
17--- a/wpa_supplicant/ctrl_iface_dbus_handlers.c
18+++ b/wpa_supplicant/ctrl_iface_dbus_handlers.c
19@@ -27,6 +27,7 @@
20 #include "wpas_glue.h"
21 #include "eapol_supp/eapol_supp_sm.h"
22 #include "wps_supplicant.h"
23+#include "wpa.h"
24
25 extern int wpa_debug_level;
26 extern int wpa_debug_show_keys;
27@@ -1270,8 +1271,11 @@ DBusMessage * wpas_dbus_iface_set_smartcard_modules(
28 wpa_s->conf->pkcs11_module_path = pkcs11_module_path;
29 #endif /* EAP_TLS_OPENSSL */
30
31+ wpa_sm_set_eapol(wpa_s->wpa, NULL);
32 eapol_sm_deinit(wpa_s->eapol);
33+ wpa_s->eapol = NULL;
34 wpa_supplicant_init_eapol(wpa_s);
35+ wpa_sm_set_eapol(wpa_s->wpa, wpa_s->eapol);
36
37 return wpas_dbus_new_success_reply(message);
38
39--
401.7.4-rc1
41
042
=== modified file 'debian/patches/series'
--- debian/patches/series 2010-03-05 12:45:09 +0000
+++ debian/patches/series 2012-01-06 16:58:27 +0000
@@ -6,3 +6,4 @@
610_multi_driver.patch610_multi_driver.patch
711_syslog.patch711_syslog.patch
812_no_syslog_flood.patch812_no_syslog_flood.patch
9git_dbus_smartcard_eapol_d719934.patch

Subscribers

People subscribed via source and target branches