netfilter: nf_log_syslog: Don't ignore unknown protocols
authorPhil Sutter <phil@nwl.cc>
Thu, 24 Mar 2022 14:03:41 +0000 (15:03 +0100)
committerPablo Neira Ayuso <pablo@netfilter.org>
Fri, 8 Apr 2022 12:36:06 +0000 (14:36 +0200)
With netdev and bridge nfprotos, loggers may see arbitrary ethernet
frames. Print at least basic info like interfaces and MAC header data.

Signed-off-by: Phil Sutter <phil@nwl.cc>
Signed-off-by: Pablo Neira Ayuso <pablo@netfilter.org>
net/netfilter/nf_log_syslog.c

index d1dcf36545d79578d08e79630f526d91a6d01577..a7ff6fdbafc94f83224d0ae9575511da5c75511c 100644 (file)
@@ -894,6 +894,33 @@ static struct nf_logger nf_ip6_logger __read_mostly = {
        .me             = THIS_MODULE,
 };
 
+static void nf_log_unknown_packet(struct net *net, u_int8_t pf,
+                                 unsigned int hooknum,
+                                 const struct sk_buff *skb,
+                                 const struct net_device *in,
+                                 const struct net_device *out,
+                                 const struct nf_loginfo *loginfo,
+                                 const char *prefix)
+{
+       struct nf_log_buf *m;
+
+       /* FIXME: Disabled from containers until syslog ns is supported */
+       if (!net_eq(net, &init_net) && !sysctl_nf_log_all_netns)
+               return;
+
+       m = nf_log_buf_open();
+
+       if (!loginfo)
+               loginfo = &default_loginfo;
+
+       nf_log_dump_packet_common(m, pf, hooknum, skb, in, out, loginfo,
+                                 prefix);
+
+       dump_mac_header(m, loginfo, skb);
+
+       nf_log_buf_close(m);
+}
+
 static void nf_log_netdev_packet(struct net *net, u_int8_t pf,
                                 unsigned int hooknum,
                                 const struct sk_buff *skb,
@@ -913,6 +940,10 @@ static void nf_log_netdev_packet(struct net *net, u_int8_t pf,
        case htons(ETH_P_RARP):
                nf_log_arp_packet(net, pf, hooknum, skb, in, out, loginfo, prefix);
                break;
+       default:
+               nf_log_unknown_packet(net, pf, hooknum, skb,
+                                     in, out, loginfo, prefix);
+               break;
        }
 }