attempt to fix garbage memory access in packet_t struct

pouriyajamshidi · May 3, 2024 · 9f12e84 · 9f12e84
1 parent c6f34ae
commit 9f12e84
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 0 deletions.
diff --git a/bpf/flat.c b/bpf/flat.c
@@ -2,6 +2,7 @@
 #include <stdint.h>
 #include <stdlib.h>
 
+#include <linux/string.h>
 #include <linux/bpf.h>
 #include <linux/bpf_common.h>
 
@@ -149,6 +150,10 @@ int flat(struct __sk_buff* skb) {
         return TC_ACT_OK;
     }
 
+    // Zero-initialize the memory region of pkt
+    // so that we do not access garbage
+    memset(pkt, 0, sizeof(struct packet_t));
+
     uint32_t offset = 0;
 
     if (handle_ip_packet(head, tail, &offset, pkt) == TC_ACT_OK) {

diff --git a/internal/probe/probe_bpfeb.o b/internal/probe/probe_bpfeb.o
diff --git a/internal/probe/probe_bpfel.o b/internal/probe/probe_bpfel.o