Skip to contentSkip to conversation

ppp: fix missing breaks in switchs

PATCH In review Kristen Carlson Accardi 2 participants 2 messages Apr 2, 2010

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 ppp: fix missing breaks in switchs

---
 gatchat/ppp_cp.c |    9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)
gatchat/ppp_cp.c +4 -5
index d5f7c92..cf2f5fa 100644
@@ -591,7 +591,7 @@ static void pppcp_down_event(struct pppcp_data *data, guint8 *packet, guint len)
 	case OPENED:
 		pppcp_transition_state(STARTING, data);
 		pppcp_this_layer_down(data);
-		/* fall through */
+		break;
 	case INITIAL:
 	case STARTING:
 		/* illegal */
@@ -840,7 +840,7 @@ static void pppcp_rca_event(struct pppcp_data *data, guint8 *packet, guint len)
 		/* scr/6x */
 		pppcp_send_configure_request(data);
 		pppcp_transition_state(REQSENT, data);
-		/* fall through */
+		break;
 	case ACKSENT:
 		pppcp_initialize_restart_count(&data->config_timer_data);
 		pppcp_this_layer_up(data);
@@ -870,7 +870,7 @@ static void pppcp_rcn_event(struct pppcp_data *data, guint8 *packet, guint len)
 	case CLOSING:
 	case STOPPING:
 		pppcp_transition_state(data->state, data);
-		/* fall through */
+		break;
 	case REQSENT:
 		pppcp_initialize_restart_count(&data->config_timer_data);
 		pppcp_send_configure_request(data);
@@ -905,10 +905,9 @@ static void pppcp_rtr_event(struct pppcp_data *data, guint8 *packet, guint len)
 	switch (data->state) {
 	case CLOSED:
 	case STOPPED:
-		pppcp_send_terminate_ack(data, packet);
-		/* fall through */
 	case CLOSING:
 	case STOPPING:
+		pppcp_send_terminate_ack(data, packet);
 		break;
 	case REQSENT:
 	case ACKRCVD:
Signature · 1 line
1.6.6.1
#1 lore raw

Hi Kristen,

Quoted 3 lines
---
 gatchat/ppp_cp.c |    9 ++++-----
 1 files changed, 4 insertions(+), 5 deletions(-)

patch has been applied. Thanks.

Regards

Marcel