Bug/Patch

[Russell Suter <rsuter mxlogic com>]

All,

Apologies up front if this isn't proper procedure for submitting a 
patch, but I'm still new.

There is a slight annoyance/bug in the advanced page of the pptp setup 
page.  If one selects MPPE and chooses 128 bit, the next time they come 
into the screen, the MPPE check box is not set.  This is because the 
require-mppe key is not set.  The following patch will always set the 
require-mppe key if the check box is set:

Index: advanced-dialog.c
===================================================================
--- advanced-dialog.c    (revision 19)
+++ advanced-dialog.c    (working copy)
@@ -385,10 +385,11 @@
            g_hash_table_insert (hash, g_strdup 
(NM_PPTP_KEY_REQUIRE_MPPE_40), g_strdup ("yes"));
            break;
        default:
-            g_hash_table_insert (hash, g_strdup 
(NM_PPTP_KEY_REQUIRE_MPPE), g_strdup ("yes"));
            break;
        }

+        g_hash_table_insert (hash, g_strdup (NM_PPTP_KEY_REQUIRE_MPPE), 
g_strdup ("yes"));
+
        widget = glade_xml_get_widget (xml, "ppp_allow_stateful_mppe");
        if (gtk_toggle_button_get_active (GTK_TOGGLE_BUTTON (widget)))
            g_hash_table_insert (hash, g_strdup 
(NM_PPTP_KEY_MPPE_STATEFUL), g_strdup ("yes"));

This may not be the best way to fix the problem, but it does work.

--
Russ