verify bpf expr in telemetry

2 files changed, 1 insertions, 25 deletions

diff --git a/include/job_ctx.h b/include/job_ctx.h
index 9d6410a..95febb7 100644
--- a/include/job_ctx.h
+++ b/include/job_ctx.h
@@ -19,5 +19,4 @@ void telemetry_job_add_cb(const char * table_name, const char * key, const char
 void telemetry_job_del_cb(const char * table_name, void ** ad, long argl, void * argp);
 void telemetry_job_uuid_get(job_bitmap_t job_id, unsigned char * uu);
 
-void job_desc_dump(const struct dp_trace_job_desc * desc);
-int is_valid_bpf_expression(const char * expr);
\ No newline at end of file
+void job_desc_dump(const struct dp_trace_job_desc * desc);
\ No newline at end of file
diff --git a/src/job_ctx.c b/src/job_ctx.c
index 51c3c7f..45e1354 100644
--- a/src/job_ctx.c
+++ b/src/job_ctx.c
@@ -6,7 +6,6 @@
 
 #include <MESA/maat.h>
 #include <cjson/cJSON.h>
-#include <pcap.h>
 #include <stdlib.h>
 #include <string.h>
 
@@ -86,13 +85,6 @@ void telemetry_job_add_cb(const char * table_name, const char * key, const char
         return;
     }
 
-    int valid_bpf_expr = is_valid_bpf_expression(job_desc->bpf_expr);
-    if (valid_bpf_expr < 0)
-    {
-        dzlog_info("not a valid bpf expr: %s", job_desc->bpf_expr);
-        return;
-    }
-
     int index = telemetry_unused_job_index_get();
     if (index < 0)
     {
@@ -201,19 +193,4 @@ void job_desc_dump(const struct dp_trace_job_desc * desc)
     }
     len += snprintf(traffic_link_id_str + len, sizeof(traffic_link_id_str) - len, "%s", "]");
     dzlog_info("    traffic_link_ids:%s", traffic_link_id_str);
-}
-
-int is_valid_bpf_expression(const char * expr)
-{
-    int ret = 0;
-    pcap_t * pcap_handle = NULL;
-    struct bpf_program fp = {};
-
-    pcap_handle = pcap_open_dead(DLT_EN10MB, 65535);
-    if (pcap_compile(pcap_handle, &fp, expr, 0, PCAP_NETMASK_UNKNOWN) < 0)
-    {
-        dzlog_info("pcap_compile execution failed: %s", pcap_geterr(pcap_handle));
-        ret = -1;
-    }
-    return ret;
 }
\ No newline at end of file