Clearly mark switch cases that fall through.

diff --git a/src/libcharon/sa/tasks/child_delete.c b/src/libcharon/sa/tasks/child_delete.c
index 45e97e4..e6834a9 100644 (file)
--- a/src/libcharon/sa/tasks/child_delete.c
+++ b/src/libcharon/sa/tasks/child_delete.c
@@ -163,6 +163,7 @@ static void process_payloads(private_child_delete_t *this, message_t *message)
                                                                                                                   protocol, spi);
                                                        continue;
                                                }
+                                               /* fall through */
                                        case CHILD_INSTALLED:
                                                if (!this->initiator)
                                                {       /* reestablish installed children if required */

diff --git a/src/libfreeswan/pfkey_v2_parse.c b/src/libfreeswan/pfkey_v2_parse.c
index 49d5cdf..a143003 100644 (file)
--- a/src/libfreeswan/pfkey_v2_parse.c
+++ b/src/libfreeswan/pfkey_v2_parse.c
@@ -1247,6 +1247,7 @@ pfkey_msg_parse(struct sadb_msg *pfkey_msg,
                                  pfkey_msg->sadb_msg_type,
                                  pfkey_v2_sadb_type_string(pfkey_msg->sadb_msg_type));
                }
+               /* fall through */
        case SADB_ACQUIRE:
        case SADB_REGISTER:
        case SADB_EXPIRE:

diff --git a/src/starter/args.c b/src/starter/args.c
index 87307f1..4d8003a 100644 (file)
--- a/src/starter/args.c
+++ b/src/starter/args.c
@@ -542,6 +542,7 @@ bool assign_arg(kw_token_t token, kw_token_t first, kw_list_t *kw, char *base,
                                }
                        }
                }
+               /* fall through */
        default:
                return TRUE;
        }

diff --git a/src/whack/whack.c b/src/whack/whack.c
index c5fe3b4..ac2d3ea 100644 (file)
--- a/src/whack/whack.c
+++ b/src/whack/whack.c
@@ -1155,6 +1155,7 @@ int main(int argc, char **argv)
 
                case OPT_STATUSALL:     /* --statusall */
                        msg.whack_statusall = TRUE;
+                       /* fall through */
 
                case OPT_STATUS:        /* --status */
                        msg.whack_status = TRUE;