From 0e8b2815a8778af199a202eabee60881912175b5 Mon Sep 17 00:00:00 2001 From: Marcel Holtmann Date: Fri, 2 Apr 2010 11:43:20 -0700 Subject: [PATCH] Always reset option_rval for every option --- gatchat/ppp_cp.c | 10 ++++++++-- 1 file changed, 8 insertions(+), 2 deletions(-) diff --git a/gatchat/ppp_cp.c b/gatchat/ppp_cp.c index 16c0603..34a9cfc 100644 --- a/gatchat/ppp_cp.c +++ b/gatchat/ppp_cp.c @@ -1158,7 +1158,7 @@ static guint8 pppcp_process_configure_request(struct pppcp_data *data, gint len; int i = 0; struct ppp_option *option; - enum option_rval rval = OPTION_ERR; + enum option_rval rval; struct pppcp_action *action = data->action; pppcp_trace(data); @@ -1178,6 +1178,9 @@ static guint8 pppcp_process_configure_request(struct pppcp_data *data, if (action->option_scan) rval = action->option_scan(option, data); + else + rval = OPTION_ERR; + switch (rval) { case OPTION_ACCEPT: data->acceptable_options = @@ -1290,7 +1293,7 @@ static guint8 pppcp_process_configure_nak(struct pppcp_data *data, struct ppp_option *naked_option; struct ppp_option *config_option; guint i = 0; - enum option_rval rval = OPTION_ERR; + enum option_rval rval; struct pppcp_action *action = data->action; pppcp_trace(data); @@ -1317,6 +1320,9 @@ static guint8 pppcp_process_configure_nak(struct pppcp_data *data, if (action->option_scan) rval = action->option_scan(naked_option, data); + else + rval = OPTION_ERR; + if (rval == OPTION_ACCEPT) { /* * check the current config options to see if they -- 2.7.4