Fixed icmp next hop mtu is unsigned short so refactored ringbuff even…

…t to repurpose event->sport
r-caamano · Jan 9, 2024 · 60995e5 · 60995e5
1 parent 1aaf41a
commit 60995e5
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/src/zfw.c b/src/zfw.c
@@ -1782,8 +1782,9 @@ static int process_events(void *ctx, void *data, size_t len){
                 __u8 inner_ttl = evt->dest[0];
                 __u8 outer_ttl = evt->source[0];
                 if(code == 4){
+                    /*evt->sport is use repurposed store next hop mtu*/
                     sprintf(message, "%s : %s : %s : %s :%s --> reported next hop mtu:%d > FRAGMENTATION NEEDED IN PATH TO:%s:%d\n", ts, ifname,
-                    (evt->direction == INGRESS) ? "INGRESS" : "EGRESS", protocol,saddr, ntohs(evt->source[1]), daddr, ntohs(evt->dport));
+                    (evt->direction == INGRESS) ? "INGRESS" : "EGRESS", protocol,saddr, ntohs(evt->sport), daddr, ntohs(evt->dport));
                     if(logging){
                         res = write_log(log_file_name, message);
                     }else{

diff --git a/src/zfw_tc_ingress.c b/src/zfw_tc_ingress.c
@@ -756,8 +756,7 @@ int bpf_sk_splice(struct __sk_buff *skb){
                                 event.daddr = o_session->ipv4.daddr;
                                 event.tracking_code = icmph->code;
                                 if(icmph->code == 4){
-                                    event.source[1] = icmph->un.frag.mtu;
-                                }
+                                    event.sport = icmph->un.frag.mtu;
                                 event.dport = o_session->ipv4.dport;
                                 send_event(&event);
                                 bpf_sk_release(sk);