Changeset 6327ce6 for lib

Timestamp:

12/19/18 14:47:21 (21 months ago)

Author:

Jacob Van Walraven <jcv9@…>

Branches:

develop

Children:

8df87c4

Parents:

1668208

Message:

Fix memory leak when configuring bpf filters issue #87

Location:

lib

Files:

• format_linux_common.c (modified) (5 diffs)
• format_linux_ring.c (modified) (1 diff)

lib/format_linux_common.c

@@ -66 +66 @@
         unsigned int arphrd;
         libtrace_dlt_t dlt;
-        libtrace_filter_t *f;
         int sock;
         pcap_t *pcap;
-
-        /* Take a copy of the filter object as it was passed in */
-        f = (libtrace_filter_t *) malloc(sizeof(libtrace_filter_t));
-        memcpy(f, filter, sizeof(libtrace_filter_t));
 
         /* If we are passed a filter with "flag" set to zero, then we must
@@ -83 +78 @@
          * anything (we've just copied it above).
          */
-        if (f->flag == 0) {
+        if (filter->flag == 0) {
                 sock = socket(PF_INET, SOCK_STREAM, 0);
                 memset(&ifr, 0, sizeof(struct ifreq));
@@ -99 +94 @@
                                 FORMAT_DATA->snaplen);
 
-                if (pcap_compile(pcap, &f->filter, f->filterstring, 0, 0) == -1) {
+                if (pcap_compile(pcap, &filter->filter, filter->filterstring, 0, 0) == -1) {
                         /* Filter didn't compile, set flag to 0 so we can
                          * detect this when trace_start() is called and
                          * produce a useful error
                          */
-                        f->flag = 0;
+                        filter->flag = 0;
                         trace_set_err(libtrace, TRACE_ERR_INIT_FAILED,
                                       "Failed to compile BPF filter (%s): %s",
-                                      f->filterstring, pcap_geterr(pcap));
+                                      filter->filterstring, pcap_geterr(pcap));
                 } else {
                         /* Set the "flag" to indicate that the filterstring
                          * has been compiled
                          */
-                        f->flag = 1;
+                        filter->flag = 1;
                 }
 
@@ -119 +114 @@
         }
 
-        if (FORMAT_DATA->filter != NULL)
-                free(FORMAT_DATA->filter);
-
-        FORMAT_DATA->filter = f;
+        FORMAT_DATA->filter = filter;
 
         return 0;
@@ -497 +489 @@
 {
         if (libtrace->format_data) {
-                if (FORMAT_DATA->filter != NULL)
-                        free(FORMAT_DATA->filter);
 
                 if (FORMAT_DATA->per_stream)

lib/format_linux_ring.c

@@ -307 +307 @@
                         }
                 }
-
-                if (FORMAT_DATA->filter != NULL)
-                        free(FORMAT_DATA->filter);
 
                 if (FORMAT_DATA->per_stream)