Embed .lua files into falco executable

Instead of having .lua files external to the program responsible for
loading rules, embed the contents of those files into the executable
and load them as strings instead of as files:

Add a cmake custom command below userspace/engine/lua that calls a
bash script lua-to-cpp.sh to generate falco_engine_lua_files.{cpp,hh}
that are compiled into the falco engine library.

The script creates a .cpp file that has const char * symbols for each
file, as well as lists of files that should be loaded when the falco
engine is loaded. There are actually two lists:

- lua_module_strings: these are loaded and also added to the lua
  runtime package.preload table, so they are available when lua code
  require()s them.

- lua_code_strings: these are loaded *and* evaluated, so the functions
  in them are availble to be called from C++.

This simplifies some of the falco_common methods, as there's no need
to keep track of a "main" lua file to load or paths from which the lua
loader should find files for modules, and there's no need to keep
track of an "alternate" lua directory that occurs for debug builds.

Also, there's no need to include any .lua files in the installed
packages, as they're built into the falco binary.

Signed-off-by: Mark Stemm <mark.stemm@gmail.com>

.circleci/config.yml

@@ -13,7 +13,7 @@ jobs:
           command: apk update
       - run:
           name: Install build dependencies
-          command: apk add g++ gcc cmake make git bash perl linux-headers autoconf automake m4 libtool elfutils-dev libelf-static patch binutils
+          command: apk add g++ gcc cmake cmake make git bash perl linux-headers autoconf automake m4 libtool elfutils-dev libelf-static patch binutils
       - run:
           name: Prepare project
           command: |
@@ -176,6 +176,38 @@ jobs:
             pushd build
             make tests
             popd
+  # Build using CentOS 8
+  # This build is static, dependencies are bundled in the Falco binary
+  "build/centos8":
+    docker:
+      - image: centos:8
+    steps:
+      - checkout
+      - run:
+          name: Update base image
+          command: dnf update -y
+      - run:
+          name: Install dependencies
+          command: dnf install gcc gcc-c++ git make cmake autoconf automake pkg-config patch libtool elfutils-libelf-devel diffutils kernel-devel kernel-headers kernel-core clang llvm which -y
+      - run:
+          name: Prepare project
+          command: |
+            mkdir build
+            pushd build
+            cmake -DBUILD_BPF=On -DUSE_BUNDLED_DEPS=On ..
+            popd
+      - run:
+          name: Build
+          command: |
+            pushd build
+            KERNELDIR=/lib/modules/$(ls /lib/modules)/build make -j4 all
+            popd
+      - run:
+          name: Run unit tests
+          command: |
+            pushd build
+            make tests
+            popd
   # Build using our own builder base image using centos 7
   # This build is static, dependencies are bundled in the Falco binary
   "build/centos7":
@@ -359,15 +391,9 @@ jobs:
             /source/falco/scripts/publish-rpm -f /build/release/falco-${FALCO_VERSION}-x86_64.rpm -r rpm-dev
       - run:
           name: Publish bin-dev
-          command: |
-            FALCO_VERSION=$(cat /build/release/userspace/falco/config_falco.h | grep 'FALCO_VERSION ' | cut -d' ' -f3 | sed -e 's/^"//' -e 's/"$//')
-            /source/falco/scripts/publish-bin -f /build/release/falco-${FALCO_VERSION}-x86_64.tar.gz -r bin-dev -a x86_64
-      - run:
-          name: Publish bin-static-dev
           command: |
             FALCO_VERSION=$(cat /build-static/release/userspace/falco/config_falco.h | grep 'FALCO_VERSION ' | cut -d' ' -f3 | sed -e 's/^"//' -e 's/"$//')
-            cp -f /build-static/release/falco-${FALCO_VERSION}-x86_64.tar.gz /build-static/release/falco-${FALCO_VERSION}-static-x86_64.tar.gz
-            /source/falco/scripts/publish-bin -f /build-static/release/falco-${FALCO_VERSION}-static-x86_64.tar.gz -r bin-dev -a x86_64
+            /source/falco/scripts/publish-bin -f /build-static/release/falco-${FALCO_VERSION}-x86_64.tar.gz -r bin-dev -a x86_64
   "publish/packages-deb-dev":
     docker:
       - image: docker.io/debian:stable
@@ -479,15 +505,9 @@ jobs:
             /source/falco/scripts/publish-rpm -f /build/release/falco-${FALCO_VERSION}-x86_64.rpm -r rpm
       - run:
           name: Publish bin
-          command: |
-            FALCO_VERSION=$(cat /build/release/userspace/falco/config_falco.h | grep 'FALCO_VERSION ' | cut -d' ' -f3 | sed -e 's/^"//' -e 's/"$//')
-            /source/falco/scripts/publish-bin -f /build/release/falco-${FALCO_VERSION}-x86_64.tar.gz -r bin -a x86_64
-      - run:
-          name: Publish bin-static
           command: |
             FALCO_VERSION=$(cat /build-static/release/userspace/falco/config_falco.h | grep 'FALCO_VERSION ' | cut -d' ' -f3 | sed -e 's/^"//' -e 's/"$//')
-            cp -f /build-static/release/falco-${FALCO_VERSION}-x86_64.tar.gz /build-static/release/falco-${FALCO_VERSION}-static-x86_64.tar.gz
-            /source/falco/scripts/publish-bin -f /build-static/release/falco-${FALCO_VERSION}-static-x86_64.tar.gz -r bin -a x86_64
+            /source/falco/scripts/publish-bin -f /build-static/release/falco-${FALCO_VERSION}-x86_64.tar.gz -r bin -a x86_64
   "publish/packages-deb":
     docker:
       - image: docker.io/debian:stable
@@ -598,6 +618,7 @@ workflows:
       - "build/ubuntu-focal"
       - "build/ubuntu-focal-debug"
       - "build/ubuntu-bionic"
+      - "build/centos8"
       - "build/centos7"
       - "build/centos7-debug"
       - "tests/integration":
@@ -661,7 +682,7 @@ workflows:
               only: master
           requires:
             - publish/docker-dev
-      # - "quality/static-analysis" # This is temporarily disabled: https://github.com/falcosecurity/falco/issues/1526
+      # - "quality/static-analysis" # This is temporarly disabled: https://github.com/falcosecurity/falco/issues/1526
   release:
     jobs:
       - "build/musl":

.cmake-format

@@ -7,7 +7,7 @@ line_width = 120
 # How many spaces to tab for indent
 tab_size = 2
 
-# If arg lists are longer than this, break them always
+# If arglists are longer than this, break them always
 max_subargs_per_line = 3
 
 # If true, separate flow control names from their parentheses with a space
@@ -21,7 +21,7 @@ separate_fn_name_with_space = False
 dangle_parens = False
 
 # If the statement spelling length (including space and parenthesis is larger
-# than the tab width by more than this among, then force reject un-nested
+# than the tab width by more than this amoung, then force reject un-nested
 # layouts.
 max_prefix_chars = 2
 
@@ -54,7 +54,7 @@ always_wrap = []
 algorithm_order = [0, 1, 2, 3, 4]
 
 # If true, the argument lists which are known to be sortable will be sorted
-# lexicographically
+# lexicographicall
 enable_sort = True
 
 # If true, the parsers may infer whether or not an argument list is sortable

.gitignore

@@ -10,8 +10,11 @@ test/.phoronix-test-suite
 test/results*.json.*
 test/build
 
+userspace/engine/lua/lyaml
+userspace/engine/lua/lyaml.lua
+
 .vscode/*
 
 .luacheckcache
 
-*.idea*
+*.idea*

ADOPTERS.md

@@ -24,8 +24,6 @@ This is a list of production adopters of Falco (in alphabetical order):
 
 * [Coveo](https://www.coveo.com/) - Coveo stitches together content and data, learning from every interaction, to tailor every experience using AI to drive growth, satisfy customers and develop employee proficiency. All Falco events are centralized in our SIEM for analysis. Understanding what is running on production servers, and the context around why things are running is even more tricky now that we have further abstractions with containers and orchestration systems. Falco is giving us a good visibility inside containers and complement other Host and Network Intrusion Detection Systems. In a near future, we expect to deploy serverless functions to take action when Falco identifies patterns worth taking action for.
 
-* [Fairwinds](https://fairwinds.com/) - [Fairwinds Insights](https://fairwinds.com/insights), Kubernetes governance software, integrates Falco to offer a single pane of glass view into potential security incidents. Insights adds out-of-the-box integrations and rules filter to reduce alert fatigue and improve security response. The platform adds security prevention, detection, and response capabilities to your existing Kubernetes infrastructure. Security and DevOps teams benefit from a centralized view of container security vulnerability scanning and runtime container security.
-
 * [Frame.io](https://frame.io/) - Frame.io is a cloud-based (SaaS) video review and collaboration platform that enables users to securely upload source media, work-in-progress edits, dailies, and more into private workspaces where they can invite their team and clients to collaborate on projects. Understanding what is running on production servers, and the context around why things are running is even more tricky now that we have further abstractions like Docker and Kubernetes. To get this needed visibility into our system, we rely on Falco. Falco's ability to collect raw system calls such as open, connect, exec, along with their arguments offer key insights on what is happening on the production system and became the foundation of our intrusion detection and alerting system.
 
 * [Giant Swarm](https://www.giantswarm.io/) - Giant Swarm manages Kubernetes clusters and infrastructure for enterprises across multiple cloud providers as well as several flavors of on-premises data centers. Our platform provisions and monitors pure "vanilla" Kubernetes clusters which can be augmented with managed solutions to many common Kubernetes challenges, including security. We use Falco for anomaly detection as part of our collection of entirely open-source tools for securing our own clusters, and offer the same capabilities to our customers as part of our [managed security offering](https://docs.giantswarm.io/app-platform/apps/security/).
@@ -41,10 +39,10 @@ This is a list of production adopters of Falco (in alphabetical order):
 
 * [Pocteo](https://pocteo.co) - Pocteo helps with Kubernetes adoption in enterprises by providing a variety of services such as training, consulting, auditing and mentoring. We build CI/CD pipelines the GitOps way, as well as design and run k8s clusters. Pocteo uses Falco as a runtime monitoring system to secure clients' workloads against suspicious behavior and ensure k8s pods immutability. We also use Falco to collect, process and act on security events through a response engine and serverless functions.
 
-* [Preferral](https://www.preferral.com) - Preferral is a HIPAA-compliant platform for Referral Management and Online Referral Forms. Preferral streamlines the referral process for patients, specialists and their referral partners. By automating the referral process, referring practices spend less time on the phone, manual efforts are eliminated, and patients get the right care from the right specialist. Preferral leverages Falco to provide a Host Intrusion Detection System to meet their HIPAA compliance requirements.
+* [Preferral](https://www.preferral.com) - Preferral is a HIPAA-compliant platform for Referral Management and Online Referral Forms. Preferral streamlines the referral process for patients, specialists and their referral partners. By automating the referral process, referring practices spend less time on the phone, manual efforts are eliminated, and patients get the right care from the right specialist. Preferral leverages Falco to provide a Host Intrusion Detection System to meet their HIPPA compliance requirements.
   * https://hipaa.preferral.com/01-preferral_hipaa_compliance/
 
-* [Qonto](https://qonto.com) - Qonto is a modern banking for SMEs and freelancers. Qonto provides a fully featured business account with a simplified accounting flow. Falco is used by our SecOps team to detect suspicious behaviors in our clusters.
+* [Qonto](https://qonto.com) - Qonto is a modern banking for SMEs and freelancers. Qonto provides a fully featured business account with a simplified accounting flow. Falco is used by our SecOps team to detect suspicous behaviors in our clusters.
 
 * [Raft](https://goraft.tech) - Raft is a government contractor that offers cloud-native solutions across many different agencies including DoD (Department of Defense), HHS (Health and Human Services), as well as within CFPB (Consumer Finance Protection Bureau). Raft leverages Falco to detect threats in our client's Kubernetes clusters and as a Host Intrusion Detection System. Raft proudly recommends Falco across all our different projects.
 
@@ -66,7 +64,7 @@ This is a list of production adopters of Falco (in alphabetical order):
 
 * [Yahoo! JAPAN](https://www.yahoo.co.jp/) Yahoo! JAPAN is a leading company of internet in Japan. We build an AI Platform in our private cloud and provide it to scientists in our company. AI Platform is a multi-tenant Kubernetes environment and more flexible, faster, more efficient Machine Learning environment. Falco is used to detect unauthorized commands and malicious access and our AI Platform is monitored and alerted by Falco.
 
-* [Sysdig](https://www.sysdig.com/) Sysdig originally created Falco in 2016 to detect unexpected or suspicious activity using a rules engine on top of the data that comes from the sysdig kernel system call driver. Sysdig provides tooling to help with vulnerability management, compliance, detection, incident response and forensics in Cloud-native environments. Sysdig Secure has extended Falco to include: a rule library, the ability to update macros, lists & rules via the user interface and API, automated tuning of rules, and rule creation based on profiling known system behavior. On top of the basic Falco rules, Sysdig Secure implements the concept of a "Security policy" that can comprise several rules which are evaluated for a user-defined infrastructure scope like Kubernetes namespaces, OpenShift clusters, deployment workload, cloud regions etc.
+* [Sysdig](https://www.sysdig.com/) Sysdig originally created Falco in 2016 to detect unexpected or suspicious activity using a rules engine on top of the data that comes from the sysdig kernel system call probe. Sysdig provides tooling to help with vulnerability management, compliance, detection, incident response and forensics in Cloud-native environments. Sysdig Secure has extended Falco to include: a rule library, the ability to update macros, lists & rules via the user interface and API, automated tuning of rules, and rule creation based on profiling known system behavior. On top of the basic Falco rules, Sysdig Secure implements the concept of a "Security policy" that can comprise several rules which are evaluated for a user-defined infrastructure scope like Kubernetes namespaces, OpenShift clusters, deployment workload, cloud regions etc.
 
 ## Projects that use Falco libs
 

CHANGELOG.md

@@ -1,153 +1,5 @@
 # Change Log
 
-## v0.31.1
-
-Released on 2022-03-09
-
-### Major Changes
-
-
-* new: add a new drop category `n_drops_scratch_map` [[#1916](https://github.com/falcosecurity/falco/pull/1916)] - [@Andreagit97](https://github.com/Andreagit97)
-* new: allow to specify multiple --cri options [[#1893](https://github.com/falcosecurity/falco/pull/1893)] - [@FedeDP](https://github.com/FedeDP)
-
-
-### Minor Changes
-
-* refactor(userspace/falco): replace direct getopt_long() cmdline option parsing with third-party cxxopts library. [[#1886](https://github.com/falcosecurity/falco/pull/1886)] - [@mstemm](https://github.com/mstemm)
-* update: driver version is b7eb0dd [[#1923](https://github.com/falcosecurity/falco/pull/1923)] - [@LucaGuerra](https://github.com/LucaGuerra)
-
-
-### Bug Fixes
-
-* fix(userspace/falco): correct plugins init config conversion from YAML to JSON [[#1907](https://github.com/falcosecurity/falco/pull/1907)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* fix(userspace/engine): for rules at the informational level being loaded at the notice level [[#1885](https://github.com/falcosecurity/falco/pull/1885)] - [@mike-stewart](https://github.com/mike-stewart)
-* chore(userspace/falco): fixes truncated -b option description. [[#1915](https://github.com/falcosecurity/falco/pull/1915)] - [@andreabonanno](https://github.com/andreabonanno)
-* update(falco): updates usage description for -o, --option [[#1903](https://github.com/falcosecurity/falco/pull/1903)] - [@andreabonanno](https://github.com/andreabonanno)
-
-### Security Fixes
-
-* Fix for a TOCTOU issue that could lead to rule bypass (CVE-2022-26316). For more information, see the [advisory](https://github.com/falcosecurity/falco/security/advisories/GHSA-6v9j-2vm2-ghf7).
-
-### Rule Changes
-
-* rule(Detect outbound connections to common miner pool ports): fix url in rule output [[#1918](https://github.com/falcosecurity/falco/pull/1918)] - [@jsoref](https://github.com/jsoref)
-* rule(macro somebody_becoming_themself): renaming macro to somebody_becoming_themselves [[#1918](https://github.com/falcosecurity/falco/pull/1918)] - [@jsoref](https://github.com/jsoref)
-* rule(list package_mgmt_binaries): `npm` added [[#1866](https://github.com/falcosecurity/falco/pull/1866)] - [@rileydakota](https://github.com/rileydakota)
-* rule(Launch Package Management Process in Container): support for detecting `npm` usage [[#1866](https://github.com/falcosecurity/falco/pull/1866)] - [@rileydakota](https://github.com/rileydakota)
-* rule(Polkit Local Privilege Escalation Vulnerability): new rule created to detect CVE-2021-4034 [[#1877](https://github.com/falcosecurity/falco/pull/1877)] - [@darryk10](https://github.com/darryk10)
-* rule(macro: modify_shell_history): avoid false-positive alerts triggered by modifications to .zsh_history.new and .zsh_history.LOCK files [[#1832](https://github.com/falcosecurity/falco/pull/1832)] - [@m4wh6k](https://github.com/m4wh6k)
-* rule(macro: truncate_shell_history): avoid false-positive alerts triggered by modifications to .zsh_history.new and .zsh_history.LOCK files [[#1832](https://github.com/falcosecurity/falco/pull/1832)] - [@m4wh6k](https://github.com/m4wh6k)
-* rule(macro sssd_writing_krb): fixed a false-positive alert that was being generated when SSSD updates /etc/krb5.keytab [[#1825](https://github.com/falcosecurity/falco/pull/1825)] - [@mac-chaffee](https://github.com/mac-chaffee)
-* rule(macro write_etc_common): fixed a false-positive alert that was being generated when SSSD updates /etc/krb5.keytab [[#1825](https://github.com/falcosecurity/falco/pull/1825)] - [@mac-chaffee](https://github.com/mac-chaffee)
-* upgrade macro(keepalived_writing_conf) [[#1742](https://github.com/falcosecurity/falco/pull/1742)] - [@pabloopez](https://github.com/pabloopez)
-* rule_output(Delete Bucket Public Access Block) typo [[#1888](https://github.com/falcosecurity/falco/pull/1888)] - [@pabloopez](https://github.com/pabloopez)
-
-
-### Non user-facing changes
-
-* fix(build): fix civetweb linking in cmake module [[#1919](https://github.com/falcosecurity/falco/pull/1919)] - [@LucaGuerra](https://github.com/LucaGuerra)
-* chore(userspace/engine): remove unused lua functions and state vars [[#1908](https://github.com/falcosecurity/falco/pull/1908)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* fix(userspace/falco): applies FALCO_INSTALL_CONF_FILE as the default … [[#1900](https://github.com/falcosecurity/falco/pull/1900)] - [@andreabonanno](https://github.com/andreabonanno)
-* fix(scripts): correct typo in `falco-driver-loader` help message [[#1899](https://github.com/falcosecurity/falco/pull/1899)] - [@leogr](https://github.com/leogr)
-* update(build)!: replaced various `PROBE` with `DRIVER` where necessary. [[#1887](https://github.com/falcosecurity/falco/pull/1887)] - [@FedeDP](https://github.com/FedeDP)
-* Add [Fairwinds](https://fairwinds.com) to the adopters list [[#1917](https://github.com/falcosecurity/falco/pull/1917)] - [@sudermanjr](https://github.com/sudermanjr)
-* build(cmake): several cmake changes to speed up/simplify builds for external projects and copying files from source-to-build directories [[#1905](https://github.com/falcosecurity/falco/pull/1905)] - [@mstemm](https://github.com/mstemm)
-
-
-## v0.31.0
-
-Released on 2022-01-31
-
-### Major Changes
-
-
-* new: add support for plugins to extend Falco functionality to new event sources and custom fields [[#1753](https://github.com/falcosecurity/falco/pull/1753)] - [@mstemm](https://github.com/mstemm)
-* new: add ability to set User-Agent http header when sending http output. Provide default value of 'falcosecurity/falco'. [[#1850](https://github.com/falcosecurity/falco/pull/1850)] - [@yoshi314](https://github.com/yoshi314)
-* new(configuration): support defining plugin init config as a YAML [[#1852](https://github.com/falcosecurity/falco/pull/1852)] - [@jasondellaluce](https://github.com/jasondellaluce)
-
-
-### Minor Changes
-
-* rules: add the official Falco ECR repository to rules [[#1817](https://github.com/falcosecurity/falco/pull/1817)] - [@calvinbui](https://github.com/calvinbui)
-* build: update CircleCI machine image for eBPF tests to a newer version of ubuntu [[#1764](https://github.com/falcosecurity/falco/pull/1764)] - [@mstemm](https://github.com/mstemm)
-* update(engine): refactor Falco engine to be agnostic to specific event sources [[#1715](https://github.com/falcosecurity/falco/pull/1715)] - [@mstemm](https://github.com/mstemm)
-* build: upgrade civetweb to v1.15 [[#1782](https://github.com/falcosecurity/falco/pull/1782)] - [@FedeDP](https://github.com/FedeDP)
-* update: driver version is 319368f1ad778691164d33d59945e00c5752cd27 now [[#1861](https://github.com/falcosecurity/falco/pull/1861)] - [@FedeDP](https://github.com/FedeDP)
-* build: allow using local libs source dir by setting `FALCOSECURITY_LIBS_SOURCE_DIR` in cmake [[#1791](https://github.com/falcosecurity/falco/pull/1791)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* build: the statically linked binary package is now published with the `-static` suffix [[#1873](https://github.com/falcosecurity/falco/pull/1873)] - [@LucaGuerra](https://github.com/LucaGuerra)
-* update!: removed "--alternate-lua-dir" cmdline option as lua scripts are now embedded in Falco executable. [[#1872](https://github.com/falcosecurity/falco/pull/1872)] - [@FedeDP](https://github.com/FedeDP)
-* build: switch to dynamic build for the binary package (`.tar.gz`) [[#1853](https://github.com/falcosecurity/falco/pull/1853)] - [@LucaGuerra](https://github.com/LucaGuerra)
-* update: simpleconsumer filtering is now being done at kernel level [[#1846](https://github.com/falcosecurity/falco/pull/1846)] - [@FedeDP](https://github.com/FedeDP)
-* update(scripts/falco-driver-loader): first try to load the latest kmod version, then fallback to an already installed if any [[#1863](https://github.com/falcosecurity/falco/pull/1863)] - [@leogr](https://github.com/leogr)
-* refactor: clean up --list output with better formatting and no duplicate sections across event sources. [[#1816](https://github.com/falcosecurity/falco/pull/1816)] - [@mstemm](https://github.com/mstemm)
-* update: embed .lua files used to load/compile rules into the main falco executable, for simplicity and to avoid tampering. [[#1843](https://github.com/falcosecurity/falco/pull/1843)] - [@mstemm](https://github.com/mstemm)
-* update: support non-enumerable event sources in gRPC outputs service [[#1840](https://github.com/falcosecurity/falco/pull/1840)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* docs: add jasondellaluce to OWNERS [[#1818](https://github.com/falcosecurity/falco/pull/1818)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* chore: --list option can be used to selectively list fields related to new sources that are introduced by plugins [[#1839](https://github.com/falcosecurity/falco/pull/1839)] - [@loresuso](https://github.com/loresuso)
-* update(userspace/falco): support arbitrary-depth nested values in YAML configuration [[#1792](https://github.com/falcosecurity/falco/pull/1792)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* build: bump FakeIt version to 2.0.9 [[#1797](https://github.com/falcosecurity/falco/pull/1797)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* update: allow append of new exceptions to rules [[#1780](https://github.com/falcosecurity/falco/pull/1780)] - [@sai-arigeli](https://github.com/sai-arigeli)
-* update: Linux packages are now signed with SHA256 [[#1758](https://github.com/falcosecurity/falco/pull/1758)] - [@twa16](https://github.com/twa16)
-
-
-### Bug Fixes
-
-* fix(scripts/falco-driver-loader): fix for SELinux insmod denials [[#1756](https://github.com/falcosecurity/falco/pull/1756)] - [@dwindsor](https://github.com/dwindsor)
-* fix(scripts/falco-driver-loader): correctly clean loaded drivers when using `--clean` [[#1795](https://github.com/falcosecurity/falco/pull/1795)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* fix(userspace/falco): in case output_file cannot be opened, throw a falco exception [[#1773](https://github.com/falcosecurity/falco/pull/1773)] - [@FedeDP](https://github.com/FedeDP)
-* fix(userspace/engine): support jsonpointer escaping in rule parser [[#1777](https://github.com/falcosecurity/falco/pull/1777)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* fix(scripts/falco-driver-loader): support kernel object files in `.zst` and `.gz` compression formats [[#1863](https://github.com/falcosecurity/falco/pull/1863)] - [@leogr](https://github.com/leogr)
-* fix(engine): correctly format json output in json_event [[#1847](https://github.com/falcosecurity/falco/pull/1847)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* fix: set http output content type to text/plain when json output is disabled [[#1829](https://github.com/falcosecurity/falco/pull/1829)] - [@FedeDP](https://github.com/FedeDP)
-* fix(userspace/falco): accept 'Content-Type' header that contains "application/json", but it is not strictly equal to it [[#1800](https://github.com/falcosecurity/falco/pull/1800)] - [@FedeDP](https://github.com/FedeDP)
-* fix(userspace/engine): supporting enabled-only overwritten rules [[#1775](https://github.com/falcosecurity/falco/pull/1775)] - [@jasondellaluce](https://github.com/jasondellaluce)
-
-
-### Rule Changes
-
-* rule(Create Symlink Over Sensitive File): corrected typo in rule output [[#1820](https://github.com/falcosecurity/falco/pull/1820)] - [@deepskyblue86](https://github.com/deepskyblue86)
-* rule(macro open_write): add support to openat2 [[#1796](https://github.com/falcosecurity/falco/pull/1796)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* rule(macro open_read): add support to openat2 [[#1796](https://github.com/falcosecurity/falco/pull/1796)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* rule(macro open_directory): add support to openat2 [[#1796](https://github.com/falcosecurity/falco/pull/1796)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* rule(Create files below dev): add support to openat2 [[#1796](https://github.com/falcosecurity/falco/pull/1796)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* rule(Container Drift Detected (open+create)): add support to openat2 [[#1796](https://github.com/falcosecurity/falco/pull/1796)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* rule(macro sensitive_mount): add containerd socket [[#1815](https://github.com/falcosecurity/falco/pull/1815)] - [@loresuso](https://github.com/loresuso)
-* rule(macro spawned_process): monitor also processes spawned by `execveat` [[#1868](https://github.com/falcosecurity/falco/pull/1868)] - [@Andreagit97](https://github.com/Andreagit97)
-* rule(Create Hardlink Over Sensitive Files): new rule to detect hard links created over sensitive files [[#1810](https://github.com/falcosecurity/falco/pull/1810)] - [@sberkovich](https://github.com/sberkovich)
-* rule(Detect crypto miners using the Stratum protocol): add `stratum2+tcp` and `stratum+ssl` protocols detection [[#1810](https://github.com/falcosecurity/falco/pull/1810)] - [@sberkovich](https://github.com/sberkovich)
-* rule(Sudo Potential Privilege Escalation): correct special case for the CVE-2021-3156 exploit [[#1810](https://github.com/falcosecurity/falco/pull/1810)] - [@sberkovich](https://github.com/sberkovich)
-* rule(list falco_hostnetwork_images): moved to k8s_audit_rules.yaml to avoid a warning when usng falco_rules.yaml only [[#1681](https://github.com/falcosecurity/falco/pull/1681)] - [@leodido](https://github.com/leodido)
-* rule(list deb_binaries): remove `apt-config` [[#1860](https://github.com/falcosecurity/falco/pull/1860)] - [@Andreagit97](https://github.com/Andreagit97)
-* rule(Launch Remote File Copy Tools in Container): add additional binaries: curl and wget. [[#1771](https://github.com/falcosecurity/falco/pull/1771)] - [@ec4n6](https://github.com/ec4n6)
-* rule(list known_sa_list): add coredns, coredns-autoscaler, endpointslicemirroring-controller, horizontal-pod-autoscaler, job-controller, node-controller (nodelifecycle), persistent-volume-binder, pv-protection-controller, pvc-protection-controller, root-ca-cert-publisher and service-account-controller as allowed service accounts in the kube-system namespace [[#1760](https://github.com/falcosecurity/falco/pull/1760)] - [@sboschman](https://github.com/sboschman)
-
-
-### Non user-facing changes
-
-* fix: force-set evt.type for plugin source events [[#1878](https://github.com/falcosecurity/falco/pull/1878)] - [@FedeDP](https://github.com/FedeDP)
-* fix: updated some warning strings; properly refresh lua files embedded in falco [[#1864](https://github.com/falcosecurity/falco/pull/1864)] - [@FedeDP](https://github.com/FedeDP)
-* style(userspace/engine): avoid creating multiple versions of methods only to assume default ruleset. Use a default argument instead. [[#1754](https://github.com/falcosecurity/falco/pull/1754)] - [@FedeDP](https://github.com/FedeDP)
-* add raft in the adopters list [[#1776](https://github.com/falcosecurity/falco/pull/1776)] - [@teshsharma](https://github.com/teshsharma)
-* build: always populate partial version variables [[#1778](https://github.com/falcosecurity/falco/pull/1778)] - [@dnwe](https://github.com/dnwe)
-* build: updated cloudtrail plugin to latest version [[#1865](https://github.com/falcosecurity/falco/pull/1865)] - [@FedeDP](https://github.com/FedeDP)
-* replace ".." concatenation with table.concat [[#1834](https://github.com/falcosecurity/falco/pull/1834)] - [@VadimZy](https://github.com/VadimZy)
-* fix(userspace/engine): actually make m_filter_all_event_types useful by properly using it as fallback when no filter event types is provided [[#1875](https://github.com/falcosecurity/falco/pull/1875)] - [@FedeDP](https://github.com/FedeDP)
-* fix(build): do not show plugin options in musl optimized builds [[#1871](https://github.com/falcosecurity/falco/pull/1871)] - [@LucaGuerra](https://github.com/LucaGuerra)
-* fix(aws_cloudtrail_rules.yaml): correct required plugin versions [[#1867](https://github.com/falcosecurity/falco/pull/1867)] - [@FedeDP](https://github.com/FedeDP)
-* docs: fix priority level "info" to "informational" [[#1858](https://github.com/falcosecurity/falco/pull/1858)] - [@Andreagit97](https://github.com/Andreagit97)
-* Field properties changes [[#1838](https://github.com/falcosecurity/falco/pull/1838)] - [@mstemm](https://github.com/mstemm)
-* update(build): updated libs to latest master version; updated plugins versions [[#1856](https://github.com/falcosecurity/falco/pull/1856)] - [@FedeDP](https://github.com/FedeDP)
-* Add Giant Swarm to Adopters list [[#1842](https://github.com/falcosecurity/falco/pull/1842)] - [@stone-z](https://github.com/stone-z)
-* update(tests): remove `token_bucket` unit tests [[#1798](https://github.com/falcosecurity/falco/pull/1798)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* fix(build): use consistent 7-character build abbrev sha [[#1830](https://github.com/falcosecurity/falco/pull/1830)] - [@LucaGuerra](https://github.com/LucaGuerra)
-* add Phoenix to adopters list [[#1806](https://github.com/falcosecurity/falco/pull/1806)] - [@kaldyka](https://github.com/kaldyka)
-* remove unused files in test directory [[#1801](https://github.com/falcosecurity/falco/pull/1801)] - [@jasondellaluce](https://github.com/jasondellaluce)
-* drop Falco luajit module, use the one provided by libs [[#1788](https://github.com/falcosecurity/falco/pull/1788)] - [@FedeDP](https://github.com/FedeDP)
-* chore(build): update libs version to 7906f7e [[#1790](https://github.com/falcosecurity/falco/pull/1790)] - [@LucaGuerra](https://github.com/LucaGuerra)
-* Add SysFlow to list of libs adopters [[#1747](https://github.com/falcosecurity/falco/pull/1747)] - [@araujof](https://github.com/araujof)
-* build: dropped centos8 circleci build because it is useless [[#1882](https://github.com/falcosecurity/falco/pull/1882)] - [@FedeDP](https://github.com/FedeDP)
-
-
 ## v0.30.0
 
 Released on 2021-10-01
@@ -399,8 +251,8 @@ Released on 2021-01-18
 * docs(proposals): Exceptions handling proposal [[#1376](https://github.com/falcosecurity/falco/pull/1376)] - [@mstemm](https://github.com/mstemm)
 * docs: fix a broken link of README [[#1516](https://github.com/falcosecurity/falco/pull/1516)] - [@oke-py](https://github.com/oke-py)
 * docs: adding the kubernetes privileged use case to use cases [[#1484](https://github.com/falcosecurity/falco/pull/1484)] - [@fntlnz](https://github.com/fntlnz)
-* rules(Mkdir binary dirs): Adds exe_running_docker_save as an exception as this rules can be triggered when a container is created. [[#1386](https://github.com/falcosecurity/falco/pull/1386)] - [@jhwbarlow](https://github.com/jhwbarlow)
-* rules(Create Hidden Files): Adds exe_running_docker_save as an exception as this rules can be triggered when a container is created. [[#1386](https://github.com/falcosecurity/falco/pull/1386)] - [@jhwbarlow](https://github.com/jhwbarlow)
+* rules(Mkdir binary dirs): Adds exe_running_docker_save as an exception as this rules can be triggerred when a container is created. [[#1386](https://github.com/falcosecurity/falco/pull/1386)] - [@jhwbarlow](https://github.com/jhwbarlow)
+* rules(Create Hidden Files): Adds exe_running_docker_save as an exception as this rules can be triggerred when a container is created. [[#1386](https://github.com/falcosecurity/falco/pull/1386)] - [@jhwbarlow](https://github.com/jhwbarlow)
 * docs(.circleci): welcome Jonah (Amazon) as a new Falco CI maintainer [[#1518](https://github.com/falcosecurity/falco/pull/1518)] - [@leodido](https://github.com/leodido)
 * build: falcosecurity/falco:master also available on the AWS ECR Public registry [[#1512](https://github.com/falcosecurity/falco/pull/1512)] - [@leodido](https://github.com/leodido)
 * build: falcosecurity/falco:latest also available on the AWS ECR Public registry [[#1512](https://github.com/falcosecurity/falco/pull/1512)] - [@leodido](https://github.com/leodido)
@@ -849,7 +701,7 @@ Released on 2020-01-23
 ### Bug Fixes
 
 * fix: providing clang into docker-builder [[#972](https://github.com/falcosecurity/falco/pull/972)]
-* fix: prevent throwing json type error c++ exceptions outside of the falco engine when processing k8s audit events. [[#928](https://github.com/falcosecurity/falco/pull/928)]
+* fix: prevent throwing json type error c++ exceptions outside of the falco engine when procesing k8s audit events. [[#928](https://github.com/falcosecurity/falco/pull/928)]
 * fix(docker/kernel/linuxkit): correct from for falco minimal image [[#913](https://github.com/falcosecurity/falco/pull/913)]
 
 ### Rule Changes
@@ -964,7 +816,7 @@ Released 2019-07-31
 
 * Fix a problem that would cause prevent container metadata lookups when falco was daemonized [[#731](https://github.com/falcosecurity/falco/pull/731)]
 
-* Allow rule priorities to be expressed as lowercase and a mix of lower/uppercase [[#737](https://github.com/falcosecurity/falco/pull/737)]
+* Allow rule priorites to be expressed as lowercase and a mix of lower/uppercase [[#737](https://github.com/falcosecurity/falco/pull/737)]
 
 ### Rule Changes
 
@@ -1159,7 +1011,7 @@ Released 2019-05-13
 
 * Docker-based builder/tester: You can now build Falco using the [falco-builder](https://falco.org/docs/source/#build-using-falco-builder-container) docker image, and run regression tests using the [falco-tester](https://falco.org/docs/source/#test-using-falco-tester-container) docker image. [[#522](https://github.com/falcosecurity/falco/pull/522)] [[#584](https://github.com/falcosecurity/falco/pull/584)]
 
-* Several small docs changes to improve clarity and readability [[#524](https://github.com/falcosecurity/falco/pull/524)] [[#540](https://github.com/falcosecurity/falco/pull/540)] [[#541](https://github.com/falcosecurity/falco/pull/541)] [[#542](https://github.com/falcosecurity/falco/pull/542)]
+* Several small docs changes to improve clarity and readibility [[#524](https://github.com/falcosecurity/falco/pull/524)] [[#540](https://github.com/falcosecurity/falco/pull/540)] [[#541](https://github.com/falcosecurity/falco/pull/541)] [[#542](https://github.com/falcosecurity/falco/pull/542)]
 
 * Add instructions on how to enable K8s Audit Logging for kops [[#535](https://github.com/falcosecurity/falco/pull/535)]
 
@@ -1274,7 +1126,7 @@ Released 2019-01-16
 
 * Fix FPs related to `apt-config`/`apt-cache`, `apk` [[#490](https://github.com/falcosecurity/falco/pull/490)]
 
-* New rules `Launch Package Management Process in Container`, `Netcat Remote Code Execution in Container`, `Launch Suspicious Network Tool in Container` look for host-level network tools like `netcat`, package management tools like `apt-get`, or network tool binaries being run in a container. [[#490](https://github.com/falcosecurity/falco/pull/490)]
+* New rules `Launch Package Management Process in Container`, `Netcat Remote Code Execution in Container`, `Lauch Suspicious Network Tool in Container` look for host-level network tools like `netcat`, package management tools like `apt-get`, or network tool binaries being run in a container. [[#490](https://github.com/falcosecurity/falco/pull/490)]
 
 * Fix the `inbound` and `outbound` macros so they work with sendto/recvfrom/sendmsg/recvmsg. [[#470](https://github.com/falcosecurity/falco/pull/470)]
 
@@ -1307,7 +1159,7 @@ Released 2018-11-09
 
 * Better coverage (e.g. reduced FPs) for critical stack, hids systems, ufw, cloud-init, etc. [[#445](https://github.com/falcosecurity/falco/pull/445)]
 
-* New rules `Launch Package Management Process in Container`, `Netcat Remote Code Execution in Container`, and `Launch Suspicious Network Tool in Container` look for running various suspicious programs in a container. [[#461](https://github.com/falcosecurity/falco/pull/461)]
+* New rules `Launch Package Management Process in Container`, `Netcat Remote Code Execution in Container`, and `Lauch Suspicious Network Tool in Container` look for running various suspicious programs in a container. [[#461](https://github.com/falcosecurity/falco/pull/461)]
 
 * Misc changes to address false positives in GKE, Istio, etc. [[#455](https://github.com/falcosecurity/falco/pull/455)] [[#439](https://github.com/falcosecurity/falco/issues/439)]
 
@@ -1362,7 +1214,7 @@ Released 2018-07-24
 
 ### Minor Changes
 
-* Rules may now have an `skip-if-unknown-filter` property. If set to true, a rule will be skipped if its condition/output property refers to a filtercheck (e.g. `fd.some-new-attribute`) that is not present in the current falco version. [[#364](https://github.com/draios/falco/pull/364)] [[#345](https://github.com/draios/falco/issues/345)]
+* Rules may now have an `skip-if-unknown-filter` property. If set to true, a rule will be skipped if its condition/output property refers to a filtercheck (e.g. `fd.some-new-attibute`) that is not present in the current falco version. [[#364](https://github.com/draios/falco/pull/364)] [[#345](https://github.com/draios/falco/issues/345)]
 * Small changes to Falco `COPYING` file so github automatically recognizes license [[#380](https://github.com/draios/falco/pull/380)]
 * New example integration showing how to connect Falco with Anchore to dynamically create falco rules based on negative scan results [[#390](https://github.com/draios/falco/pull/390)]
 * New example integration showing how to connect Falco, [nats](https://nats.io/), and K8s to run flexible "playbooks" based on Falco events [[#389](https://github.com/draios/falco/pull/389)]
@@ -1463,7 +1315,7 @@ Released 2017-10-10
 
 Released 2017-10-10
 
-**Important**: the location for falco's configuration file has moved from `/etc/falco.yaml` to `/etc/falco/falco.yaml`. The default rules file has moved from `/etc/falco_rules.yaml` to `/etc/falco/falco_rules.yaml`. In addition, 0.8.0 has added a _local_ rules file to `/etc/falco/falco_rules.local.yaml`. See [the documentation](https://github.com/draios/falco/wiki/Falco-Default-and-Local-Rules-Files) for more details.
+**Important**: the location for falco's configuration file has moved from `/etc/falco.yaml` to `/etc/falco/falco.yaml`. The default rules file has moved from `/etc/falco_rules.yaml` to `/etc/falco/falco_rules.yaml`. In addition, 0.8.0 has added a _local_ ruls file to `/etc/falco/falco_rules.local.yaml`. See [the documentation](https://github.com/draios/falco/wiki/Falco-Default-and-Local-Rules-Files) for more details.
 
 ### Major Changes
 

CMakeLists.txt

@@ -58,7 +58,6 @@ if(CMAKE_BUILD_TYPE STREQUAL "debug")
 else()
   set(CMAKE_BUILD_TYPE "release")
   set(KBUILD_FLAGS "${DRAIOS_FEATURE_FLAGS}")
-  add_definitions(-DBUILD_TYPE_RELEASE)
 endif()
 message(STATUS "Build type: ${CMAKE_BUILD_TYPE}")
 
@@ -68,7 +67,6 @@ endif()
 
 if(MUSL_OPTIMIZED_BUILD)
   set(MUSL_FLAGS "-static -Os -fPIE -pie")
-  add_definitions(-DMUSL_OPTIMIZED)
 endif()
 
 # explicitly set hardening flags
@@ -99,8 +97,8 @@ set(CMAKE_CXX_FLAGS_RELEASE "-O3 -fno-strict-aliasing -DNDEBUG")
 include(GetFalcoVersion)
 
 set(PACKAGE_NAME "falco")
-set(DRIVER_NAME "falco")
-set(DRIVER_DEVICE_NAME "falco")
+set(PROBE_NAME "falco")
+set(PROBE_DEVICE_NAME "falco")
 set(DRIVERS_REPO "https://download.falco.org/driver")
 if(CMAKE_INSTALL_PREFIX_INITIALIZED_TO_DEFAULT)
   set(CMAKE_INSTALL_PREFIX
@@ -150,8 +148,6 @@ if(NOT MINIMAL_BUILD)
   include(civetweb)
 endif()
 
-include(cxxopts)
-
 # Lpeg
 include(lpeg)
 
@@ -210,9 +206,7 @@ add_subdirectory(userspace/engine)
 add_subdirectory(userspace/falco)
 add_subdirectory(tests)
 
-if(NOT MUSL_OPTIMIZED_BUILD)
-  include(plugins)
-endif()
+include(plugins)
 
 # Packages configuration
 include(CPackConfig)

README.md

@@ -5,7 +5,7 @@
 
 [![Build Status](https://img.shields.io/circleci/build/github/falcosecurity/falco/master?style=for-the-badge)](https://circleci.com/gh/falcosecurity/falco) [![CII Best Practices Summary](https://img.shields.io/cii/summary/2317?label=CCI%20Best%20Practices&style=for-the-badge)](https://bestpractices.coreinfrastructure.org/projects/2317) [![GitHub](https://img.shields.io/github/license/falcosecurity/falco?style=for-the-badge)](COPYING)
 
-Want to talk? Join us on the [#falco](https://kubernetes.slack.com/messages/falco) channel in the [Kubernetes Slack](https://slack.k8s.io).
+Want to talk? Join us on the [#falco](https://kubernetes.slack.com/archives/CMWH3EH32) channel in the [Kubernetes Slack](https://slack.k8s.io).
 
 ### Latest releases
 
@@ -58,25 +58,10 @@ Notes:
 ---
 
 The Falco Project, originally created by [Sysdig](https://sysdig.com), is an incubating [CNCF](https://cncf.io) open source cloud native runtime security tool.
-Falco makes it easy to consume kernel events, and enrich those events with information from Kubernetes and the rest of the cloud native stack. 
-Falco can also be extended to other data sources by using plugins.
+Falco makes it easy to consume kernel events, and enrich those events with information from Kubernetes and the rest of the cloud native stack.
 Falco has a rich set of security rules specifically built for Kubernetes, Linux, and cloud-native.
 If a rule is violated in a system, Falco will send an alert notifying the user of the violation and its severity.
 
-### What can Falco detect?
-
-Falco can detect and alert on any behavior that involves making Linux system calls.
-Falco alerts can be triggered by the use of specific system calls, their arguments, and by properties of the calling process.
-For example, Falco can easily detect incidents including but not limited to:
-
-- A shell is running inside a container or pod in Kubernetes.
-- A container is running in privileged mode, or is mounting a sensitive path, such as `/proc`, from the host.
-- A server process is spawning a child process of an unexpected type.
-- Unexpected read of a sensitive file, such as `/etc/shadow`.
-- A non-device file is written to `/dev`.
-- A standard system binary, such as `ls`, is making an outbound network connection.
-- A privileged pod is started in a Kubernetes cluster.
-
 ### Installing Falco
 
 If you would like to run Falco in **production** please adhere to the [official installation guide](https://falco.org/docs/getting-started/installation/).
@@ -105,22 +90,20 @@ The Falco Project supports various SDKs for this endpoint.
 | Rust     | [client-rs](https://github.com/falcosecurity/client-rs) |
 | Python   | [client-py](https://github.com/falcosecurity/client-py) |
 
-### Plugins
 
-Falco comes with a [plugin framework](https://falco.org/docs/plugins/) that extends it to potentially any cloud detection scenario. Plugins are shared libraries that conform to a documented API and allow for:
+### What can Falco detect?
 
-- Adding new event sources that can be used in rules;
-- Adding the ability to define new fields and extract information from events.
-
-The Falco Project maintains [various plugins](https://github.com/falcosecurity/plugins) and provides SDKs for plugin development.
-
-
-##### SDKs
-
-| Language | Repository                                                                    |
-|----------|-------------------------------------------------------------------------------|
-| Go       | [falcosecurity/plugin-sdk-go](https://github.com/falcosecurity/plugin-sdk-go) |
+Falco can detect and alert on any behavior that involves making Linux system calls.
+Falco alerts can be triggered by the use of specific system calls, their arguments, and by properties of the calling process.
+For example, Falco can easily detect incidents including but not limited to:
 
+- A shell is running inside a container or pod in Kubernetes.
+- A container is running in privileged mode, or is mounting a sensitive path, such as `/proc`, from the host.
+- A server process is spawning a child process of an unexpected type.
+- Unexpected read of a sensitive file, such as `/etc/shadow`.
+- A non-device file is written to `/dev`.
+- A standard system binary, such as `ls`, is making an outbound network connection.
+- A privileged pod is started in a Kubernetes cluster.
 
 ### Documentation
 
@@ -132,7 +115,7 @@ To get involved with The Falco Project please visit [the community repository](h
 
 How to reach out?
 
- - Join the [#falco](https://kubernetes.slack.com/messages/falco) channel on the [Kubernetes Slack](https://slack.k8s.io)
+ - Join the #falco channel on the [Kubernetes Slack](https://slack.k8s.io)
  - [Join the Falco mailing list](https://lists.cncf.io/g/cncf-falco-dev)
  - [Read the Falco documentation](https://falco.org/docs/)
 

RELEASE.md

@@ -95,7 +95,7 @@ Now assume `x.y.z` is the new version.
 
     #### Release Manager <github handle>
 
-    <!-- Substitute GitHub handle with the release manager's one -->
+    <!-- Substitute Github handle with the release manager's one -->
     ```
 
 - Finally, publish the release!

brand/README.md

@@ -56,7 +56,7 @@ If a rule has been violated, Falco triggers an alert.
 ### How does Falco work?
 
 Falco traces kernel events and reports information about the system calls being executed at runtime.
-Falco leverages the extended berkeley packet filter (eBPF) which is a kernel feature implemented for dynamic crash-resilient and secure code execution in the kernel. 
+Falco leverages the extended berkley packet filter (eBPF) which is a kernel feature implemented for dynamic crash-resilient and secure code execution in the kernel. 
 Falco enriches these kernel events with information about containers running on the system.
 Falco also can consume signals from other input streams such as the containerd socket, the Kubernetes API server and the Kubernetes audit log.
 At runtime, Falco will reason about these events and assert them against configured security rules.
@@ -113,7 +113,7 @@ Falco ultimately is a security engine. It reasons about signals coming from a sy
 
 ##### Anomaly detection
 
-This refers to an event that occurs with something unusual, concerning, or odd occurs.
+This refers to an event that occurs with something unsual, concerning, or odd occurs.
 We can associate anomalies with unwanted behavior, and alert in their presence.
 
 ##### Detection tooling
@@ -143,10 +143,6 @@ Sometimes this word is incorrectly used to refer to a `probe`.
 
 The global term for the software that sends events from the kernel. Such as the eBPF `probe` or the `kernel module`.
 
-#### Plugin
-
-Used to describe a dynamic shared library (`.so` files in Unix, `.dll` files in Windows) that conforms to a documented API and allows to extend Falco's capabilities.
-
 #### Falco
 
 The name of the project, and also the name of [the main engine](https://github.com/falcosecurity/falco) that the rest of the project is built on.

cmake/cpack/CMakeCPackOptions.cmake

@@ -10,4 +10,5 @@ endif()
 
 if(CPACK_GENERATOR MATCHES "TGZ")
 	set(CPACK_SET_DESTDIR "ON")
+	set(CPACK_STRIP_FILES "OFF")
 endif()

cmake/modules/CatchAddTests.cmake

@@ -45,7 +45,7 @@ string(REPLACE "\n" ";" output "${output}")
 # Parse output
 foreach(line ${output})
   set(test ${line})
-  # use escape commas to handle properly test cases with commands inside the name
+  # use escape commas to handle properly test cases with commans inside the name
   string(REPLACE "," "\\," test_name ${test})
   # ...and add to script
   add_command(add_test "${prefix}${test}${suffix}" ${TEST_EXECUTOR} "${TEST_EXECUTABLE}" "${test_name}" ${extra_args})

cmake/modules/DownloadStringViewLite.cmake

@@ -24,7 +24,6 @@ ExternalProject_Add(
   GIT_TAG "v1.4.0"
   CONFIGURE_COMMAND ""
   BUILD_COMMAND ""
-  UPDATE_COMMAND ""
   INSTALL_COMMAND
     ${CMAKE_COMMAND} -E copy ${STRING_VIEW_LITE_PREFIX}/src/string-view-lite/include/nonstd/string_view.hpp
     ${STRING_VIEW_LITE_INCLUDE}/nonstd/string_view.hpp)

cmake/modules/civetweb.cmake

@@ -12,8 +12,8 @@
 #
 
 set(CIVETWEB_SRC "${PROJECT_BINARY_DIR}/civetweb-prefix/src/civetweb/")
-set(CIVETWEB_LIB "${CIVETWEB_SRC}/install/lib/libcivetweb.a")
-SET(CIVETWEB_CPP_LIB "${CIVETWEB_SRC}/install/lib/libcivetweb-cpp.a")
+set(CIVETWEB_LIB "${CIVETWEB_SRC}/install/${CMAKE_INSTALL_LIBDIR}/libcivetweb.a")
+SET(CIVETWEB_CPP_LIB "${CIVETWEB_SRC}/install/${CMAKE_INSTALL_LIBDIR}/libcivetweb-cpp.a")
 set(CIVETWEB_INCLUDE_DIR "${CIVETWEB_SRC}/install/include")
 message(STATUS "Using bundled civetweb in '${CIVETWEB_SRC}'")
 if (USE_BUNDLED_OPENSSL)
@@ -25,7 +25,6 @@ if (USE_BUNDLED_OPENSSL)
             INSTALL_DIR ${CIVETWEB_SRC}/install
             CMAKE_ARGS
             -DBUILD_TESTING=off
-            -DCMAKE_INSTALL_LIBDIR=lib
             -DCIVETWEB_BUILD_TESTING=off
             -DCIVETWEB_ENABLE_CXX=on
             -DCIVETWEB_ENABLE_SERVER_EXECUTABLE=off

cmake/modules/copy_files_to_build_dir.cmake

@@ -1,30 +0,0 @@
-#
-# Copyright (C) 2022 The Falco Authors.
-#
-# Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with
-# the License. You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the
-# specific language governing permissions and limitations under the License.
-#
-
-function(copy_files_to_build_dir source_files targetsuffix)
-
-  set(build_files)
-
-  foreach(file_path ${source_files})
-	  get_filename_component(trace_file ${file_path} NAME)
-	  list(APPEND build_files ${CMAKE_CURRENT_BINARY_DIR}/${trace_file})
-  endforeach()
-
-  add_custom_target(copy-files-${targetsuffix} ALL
-	  DEPENDS ${build_files})
-
-  add_custom_command(OUTPUT ${build_files}
-	  COMMAND ${CMAKE_COMMAND} -E copy_if_different ${source_files} ${CMAKE_CURRENT_BINARY_DIR}
-	  DEPENDS ${source_files})
-
-endfunction()

cmake/modules/cxxopts.cmake

@@ -1,23 +0,0 @@
-#
-# Copyright (C) 2022 The Falco Authors.
-#
-# Licensed under the Apache License, Version 2.0 (the "License"); you may not use this file except in compliance with
-# the License. You may obtain a copy of the License at
-#
-# http://www.apache.org/licenses/LICENSE-2.0
-#
-# Unless required by applicable law or agreed to in writing, software distributed under the License is distributed on an
-# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the
-# specific language governing permissions and limitations under the License.
-#
-
-set(CXXOPTS_SRC "${PROJECT_BINARY_DIR}/cxxopts-prefix/src/cxxopts/")
-set(CXXOPTS_INCLUDE_DIR "${CXXOPTS_SRC}/include")
-
-ExternalProject_Add(
-        cxxopts
-        URL "https://github.com/jarro2783/cxxopts/archive/refs/tags/v3.0.0.tar.gz"
-        URL_HASH "SHA256=36f41fa2a46b3c1466613b63f3fa73dc24d912bc90d667147f1e43215a8c6d00"
-	CONFIGURE_COMMAND ""
-	BUILD_COMMAND ""
-	INSTALL_COMMAND "")

cmake/modules/falcosecurity-libs.cmake

@@ -24,8 +24,8 @@ else()
   # default below In case you want to test against another falcosecurity/libs version just pass the variable - ie., `cmake
   # -DFALCOSECURITY_LIBS_VERSION=dev ..`
   if(NOT FALCOSECURITY_LIBS_VERSION)
-    set(FALCOSECURITY_LIBS_VERSION "b7eb0dd65226a8dc254d228c8d950d07bf3521d2")
-    set(FALCOSECURITY_LIBS_CHECKSUM "SHA256=0f6dcdc3b94243c91294698ee343806539af81c5b33c60c6acf83fc1aa455e85")
+    set(FALCOSECURITY_LIBS_VERSION "bb9bee8e522fc953c2a79093d688d3d82b925e8b")
+    set(FALCOSECURITY_LIBS_CHECKSUM "SHA256=ab2f18ff9c8d92dd06088ccfa73d4230fce3617613229f5afd839a37c13b0459")
   endif()
 
   # cd /path/to/build && cmake /path/to/source
@@ -47,8 +47,8 @@ if(MUSL_OPTIMIZED_BUILD)
   add_definitions(-DMUSL_OPTIMIZED)
 endif()
 
-set(DRIVER_VERSION "${FALCOSECURITY_LIBS_VERSION}")
-set(DRIVER_NAME "falco")
+set(PROBE_VERSION "${FALCOSECURITY_LIBS_VERSION}")
+set(PROBE_NAME "falco")
 set(DRIVER_PACKAGE_NAME "falco")
 set(SCAP_BPF_PROBE_ENV_VAR_NAME "FALCO_BPF_PROBE")
 set(SCAP_HOST_ROOT_ENV_VAR_NAME "HOST_ROOT")

cmake/modules/plugins.cmake

@@ -13,12 +13,12 @@
 
 include(ExternalProject)
 
-string(TOLOWER ${CMAKE_HOST_SYSTEM_NAME} PLUGINS_SYSTEM_NAME)
+set(PLUGINS_VERSION "0.1.0-rc1-28-g019437e")
 
 ExternalProject_Add(
   cloudtrail-plugin
-  URL "https://download.falco.org/plugins/stable/cloudtrail-0.2.3-${PLUGINS_SYSTEM_NAME}-${CMAKE_HOST_SYSTEM_PROCESSOR}.tar.gz"
-  URL_HASH "SHA256=3dfce36f37a4f834b6078c6b78776414472a6ee775e8f262535313cc4031d0b7"
+  URL "https://download.falco.org/plugins/dev/cloudtrail-${PLUGINS_VERSION}-${CMAKE_HOST_SYSTEM_PROCESSOR}.tar.gz"
+  URL_HASH "SHA256=ad9692957c5435238e07d1625e1b247eabe98b85f54de9218367fdd73a6f3f0b"
   CONFIGURE_COMMAND ""
   BUILD_COMMAND ""
   INSTALL_COMMAND "")
@@ -27,8 +27,8 @@ install(FILES "${PROJECT_BINARY_DIR}/cloudtrail-plugin-prefix/src/cloudtrail-plu
 
 ExternalProject_Add(
   json-plugin
-  URL "https://download.falco.org/plugins/stable/json-0.2.2-${PLUGINS_SYSTEM_NAME}-${CMAKE_HOST_SYSTEM_PROCESSOR}.tar.gz"
-  URL_HASH "SHA256=83eb411c9f2125695875b229c6e7974e6a4cc7f028be146b79d26db30372af5e"
+  URL "https://download.falco.org/plugins/dev/json-${PLUGINS_VERSION}-${CMAKE_HOST_SYSTEM_PROCESSOR}.tar.gz"
+  URL_HASH "SHA256=721ea5226b0f623915d0d5c34870589ad33a8ff795b0daa1af72f21a67430077"
   CONFIGURE_COMMAND ""
   BUILD_COMMAND ""
   INSTALL_COMMAND "")

docker/local/rules/CMakeLists.txt

@@ -1,7 +1,13 @@
-include(copy_files_to_build_dir)
-
 # Note: list of rules is created at cmake time, not build time
 file(GLOB test_rule_files
 	"${CMAKE_CURRENT_SOURCE_DIR}/../../../test/rules/*.yaml")
 
-copy_files_to_build_dir("${test_rule_files}" docker-local-rules)
+foreach(rule_file_path ${test_rule_files})
+	get_filename_component(rule_file ${rule_file_path} NAME)
+	add_custom_target(docker-local-rule-${rule_file} ALL
+		DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/${rule_file})
+	add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/${rule_file}
+		COMMAND ${CMAKE_COMMAND} -E copy ${rule_file_path} ${CMAKE_CURRENT_BINARY_DIR}/${rule_file}
+		DEPENDS ${rule_file_path})
+endforeach()
+

docker/local/traces/CMakeLists.txt

@@ -1,7 +1,13 @@
-include(copy_files_to_build_dir)
-
 # Note: list of traces is created at cmake time, not build time
 file(GLOB test_trace_files
 	"${CMAKE_CURRENT_SOURCE_DIR}/../../../test/trace_files/*.scap")
 
-copy_files_to_build_dir("${test_trace_files}" docker-local-traces)
+foreach(trace_file_path ${test_trace_files})
+	get_filename_component(trace_file ${trace_file_path} NAME)
+	add_custom_target(docker-local-trace-${trace_file} ALL
+		DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/${trace_file})
+	add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		COMMAND ${CMAKE_COMMAND} -E copy ${trace_file_path} ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		DEPENDS ${trace_file_path})
+endforeach()
+

docker/no-driver/Dockerfile

@@ -20,7 +20,7 @@ RUN curl -L -o falco.tar.gz \
 RUN sed -e 's/time_format_iso_8601: false/time_format_iso_8601: true/' < /falco/etc/falco/falco.yaml > /falco/etc/falco/falco.yaml.new \
     && mv /falco/etc/falco/falco.yaml.new /falco/etc/falco/falco.yaml
 
-FROM debian:11-slim
+FROM scratch
 
 LABEL maintainer="cncf-falco-dev@lists.cncf.io"
 

falco.yaml

@@ -92,7 +92,7 @@ log_level: info
 # Minimum rule priority level to load and run. All rules having a
 # priority more severe than this level will be loaded/run.  Can be one
 # of "emergency", "alert", "critical", "error", "warning", "notice",
-# "informational", "debug".
+# "info", "debug".
 priority: debug
 
 # Whether or not output to any of the output channels below is
@@ -246,7 +246,6 @@ program_output:
 http_output:
   enabled: false
   url: http://some.url
-  user_agent: "falcosecurity/falco"
 
 # Falco supports running a gRPC server with two main binding types
 # 1. Over the network with mandatory mutual TLS authentication (mTLS)

proposals/20190909-psp-rules-support.md

@@ -27,7 +27,7 @@ That's where Falco comes in. We want to make it possible for Falco to perform a
 
 Transparently read a candidate PSP into an equivalent set of Falco rules that can look for the conditions in the PSP.
 
-The PSP is converted into a set of Falco rules which can be either saved as a file for later use/inspection, or loaded directly so that they can monitor system calls and k8s audit activity.
+The PSP is converted into a set of Falco rules which can be either saved as a file for later use/inspection, or loaded directly so they they can monitor system calls and k8s audit activity.
 
 ### Non-Goals
 
@@ -51,6 +51,6 @@ No diagrams yet.
 
 * We'll use [inja](https://github.com/pantor/inja) as the templating engine.
 
-* For the most part, we can rely on the existing framework of rules, filter expressions, and output expressions that already exist in Falco. One significant change will be that filter fields can extract more than one "value" per event, and we'll need to define new operators to perform set comparisons between values in an event and values in the comparison right-hand-side.
+* For the most part, we can rely on the existing framework of rules, filter expressions, and output expressions that already exist in Falco. One significant change will be that filter fields can extract more than one "value" per event, and we'll need to define new operators to perform set comparisions betweeen values in an event and values in the comparison right-hand-side.
 
 * This will rely heavily on existing support for [K8s Audit Events](https://falco.org/docs/event-sources/kubernetes-audit/) in Falco.

proposals/20191030-api.md

@@ -6,13 +6,13 @@ This is a proposal to better structure the Falco API.
 
 The Falco API is a set of contracts describing how users can interacts with Falco.
 
-By defining a set of interfaces the Falco Authors intend to decouple Falco from other softwares and data (eg., from the input sources) and, at the same time, make it more extensible.
+By definiing a set of interfaces the Falco Authors intend to decouple Falco from other softwares and data (eg., from the input sources) and, at the same time, make it more extensible.
 
-Thus, this document intent is to propose a list of services that constitute the Falco API (targeting the first stable version of Falco, v1.0.0).
+Thus, this document intent is to propose a list of services that contistute the Falco API (targeting the first stable version of Falco, v1.0.0).
 
 ## Motivation
 
-We want to enable users to use third-party clients to interface with Falco outputs, inputs, rules, and configurations.
+We want to enable users to use thirdy-party clients to interface with Falco outputs, inputs, rules, and configurations.
 
 Such ability would enable the community to create a whole set of OSS tools, built on top of Falco.
 
@@ -94,7 +94,7 @@ This translates in having the following set of `proto` files.
     }
     ```
 
-- one or more `.proto` containing the command models - ie., the already existing `schema.proto` containing source enum, etc.
+- one or more `.proto` containing the commond models - ie., the already existing `schema.proto` containing source enum, etc.
 
     ```proto3
     # schema.proto

proposals/20191217-rules-naming-convention.md

@@ -36,7 +36,7 @@ There will be no intention to cover Falco rule syntax in this proposal.
 
 ### Use cases
 
-When new PRs are created in the area of rules, reviewers need to examine whether there are new rules, macros or lists are introduced. If yes, check whether follow the naming convention.
+When new PRs are created in the area of rules, reviewers need to examine whether there are new rules, macros or lists are introduced. If yes, check wether follow the naming convention.
 
 ### Diagrams
 

proposals/20200506-artifacts-scope-part-1.md

@@ -82,7 +82,7 @@ This is done as needed, and can best be measured by the need to cut a release an
 
 ### official support
 
-As the need for a project grows, it can ultimately achieve the highest and most coveted status within The Falco Project. "_Official support_."
+As the need for a project grows, it can ultimately achieve the highest and most coveted status within The Falco Project. "_Offical support_."
 
 The artifacts listed above are part of the official Falco release process. These artifact will be refined and amended by the [Part 2](./20200506-artifacts-scope-part-2.md).
 
@@ -111,4 +111,4 @@ Update documentation in [falco-website#184](https://github.com/falcosecurity/fal
 ### Adjusting projects
 
  - YAML manifest documentation to be moved to `contrib`
- - Minikube, Kind, Puppet, Ansible, etc documentation to be moved to `contrib`
+ - Minkube, Kind, Puppet, Ansible, etc documentation to be moved to `contrib`

proposals/20200828-structured-exception-handling.md

@@ -196,13 +196,13 @@ Exception values will most commonly be defined in rules with append: true. Here'
 
 A rule exception applies if for a given event, the fields in a rule.exception match all of the values in some exception.item. For example, if a program `apk` writes to a file below `/usr/lib/alpine`, the rule will not trigger, even if the condition is met.
 
-Notice that an item in a values list can be a list. This allows building exceptions with operators like "in", "pmatch", etc. that work on a list of items. The item can also be a name of an existing list. If not present surrounding parentheses will be added.
+Notice that an item in a values list can be a list. This allows building exceptions with operators like "in", "pmatch", etc. that work on a list of items. The item can also be a name of an existing list. If not present surrounding parantheses will be added.
 
 Finally, note that the structure of the values property differs between the items where fields is a list of fields (proc_writer/container_writer/proc_filenames) and when it is a single field (procs_only). This changes how the condition snippet is constructed.
 
 ### Implementation
 
-For exception items where the fields property is a list of field names, each exception can be thought of as an implicit "and not (field1 cmp1 val1 and field2 cmp2 val2 and...)" appended to the rule's condition. For exception items where the fields property is a single field name, the exception can be thought of as an implicit "and not field cmp (val1, val2, ...)". In practice, that's how exceptions will be implemented.
+For exception items where the fields property is a list of field names, each exception can be thought of as an implicit "and not (field1 cmp1 val1 and field2 cmp2 val2 and...)" appended to the rule's condition. For exception items where the fields property is a single field name, the exception can be thought of as an implict "and not field cmp (val1, val2, ...)". In practice, that's how exceptions will be implemented.
 
 When a rule is parsed, the original condition will be wrapped in an extra layer of parentheses and all exception values will be appended to the condition. For example, using the example above, the resulting condition will be:
 
@@ -214,7 +214,7 @@ When a rule is parsed, the original condition will be wrapped in an extra layer
 	(fd.filename in (python, go))))
 ```
 
-The exceptions are effectively syntactic sugar that allows expressing sets of exceptions in a concise way.
+The exceptions are effectively syntatic sugar that allows expressing sets of exceptions in a concise way.
 
 ### Advantages
 

proposals/20200901-artifacts-cleanup.md

@@ -2,7 +2,7 @@
 
 This document reflects when and how we clean up the Falco artifacts from their storage location.
 
-**Superseded by**: [drivers-storage-s3 proposal](https://github.com/falcosecurity/falco/blob/master/proposals/20201025-drivers-storage-s3.md).
+**Superseeded by**: [drivers-storage-s3 proposal](https://github.com/falcosecurity/falco/blob/master/proposals/20201025-drivers-storage-s3.md).
 
 ## Motivation
 
@@ -90,7 +90,7 @@ This way, assuming the number of prebuilt drivers does not skyrocket, we can rea
 
 Notice that, in case a Falco stable release will not depend on a new driver version, this means the last two driver versions will, in this case, cover more than the two Falco stable releases.
 
-### Archiving
+### Archivation
 
 Since the process of building drivers is time and resource consuming, this document also proposes to move the driver versions in other storage facilities.
 

proposals/20210501-plugin-system.md

@@ -66,7 +66,7 @@ Source plugins also provide an "id", which is globally unique and is used in cap
 
 An extractor plugin focuses only on field extraction from events generated by other plugins, or by the core libraries. It does *not* provide an event source, but can extract fields from other event sources. An example is json field extraction, where a plugin might be able to extract fields from arbitrary json payloads.
 
-An extractor plugin provides an optional set of event sources. When the framework receives an event with an event source in the plugin's set of event sources, fields in expressions/Falco outputs will be extracted from events using the plugin. An extractor plugin can also *not* name a set of event sources. In this case, fields will be extracted from *all* events, regardless of source. In this case, the extractor plugin must detect the format of arbitrary payloads and be able to return NULL/no value when the payload is not supported.
+An extractor plugin provides an optional set of event sources. When the framework receives an event with an event source in the plugin's set of event sources, fields in expressions/Falco outputs will be extracted from events using the plugin. An extractor plugin can also *not* name a set of event sources. In this case, fields will be extracted from *all* events, regardless of source. In this case, the exctractor plugin must detect the format of arbitrary payloads and be able to return NULL/no value when the payload is not supported.
 
 ### Support for Plugin Events in Capture Files.
 
@@ -91,7 +91,7 @@ The libraries will do everything possible to validate the data coming from the p
 
 ### Plugin/Event Source registries
 
-Every source plugin requires its own, unique plugin ID to interoperate with Falco and the other plugins. The plugin ID will be used by the libs to properly process incoming events (for example, when saving events to file and loading them back), and by plugins to unambiguously recognize their dependencies.
+Every source plugin requires its own, unique plugin ID to interoperate with Falco and the other plugins. The plugin ID will be used by the libs to properly process incoming events (for example, when saving events to file and loading them back), and by plugins to unuambiguosly recognize their dependencies.
 
 To facilitate the allocation and distribution of plugin IDs, we will require that plugin developers request IDs for their plugins to the Falco organization. The mechanism used for plugin allocation is not determined yet and will be discussed in the future.
 
@@ -474,7 +474,7 @@ typedef struct
 	// Return value: a json array of strings containing event
 	//   sources returned by a source plugin's get_event_source()
 	//   function.
-	// This function is optional--if NULL then the extractor
+	// This function is optional--if NULL then the exctractor
 	// plugin will receive every event.
 	//
 	char* (*get_extract_event_sources)();
@@ -526,7 +526,7 @@ We will also make a change to compile rules/macros/lists selectively based on th
 
 ### Handling Duplicate/Overlapping Fields in Plugins/Libraries Core
 
-At an initial glance, adding plugins introduces the possibility of tens/hundreds of new filtercheck fields that could potentially overlap/conflict. For example, what happens if a plugin defines a "proc.name" field? However, the notion of "event source" makes these potential conflicts manageable.
+At an initial glance, adding plugins introduces the possibility of tens/hundreds of new filtercheck fields that could potentially overlap/conflict. For example, what happens if a plugin defines a "proc.name" field? However, the notion of "event source" makes these potential conflicts managable.
 
 Remember that field extraction is always done in the context of an event, and each event can be mapped back to an event source. So we only need to ensure that filtercheck fields are distinct for a given event source. For example, it's perfectly valid for an AWS Cloudtrail plugin to define a proc.name field, as the events generated by that plugin are wholly separate from syscall events. For syscall events, the AWS Cloudtrail plugin is not involved and the core libraries extract the process name for the tid performing a syscall. For AWS Cloudtrail events, the core libraries are not involved in field extraction and is performed by the AWS Cloudtrail plugin instead.
 

rules/aws_cloudtrail_rules.yaml

@@ -1,5 +1,5 @@
 #
-# Copyright (C) 2022 The Falco Authors.
+# Copyright (C) 2019 The Falco Authors.
 #
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -22,9 +22,7 @@
 # anything semver-compatible.
 - required_plugin_versions:
   - name: cloudtrail
-    version: 0.2.3
-  - name: json
-    version: 0.2.2
+    version: 0.1.0
 
 # Note that this rule is disabled by default. It's useful only to
 # verify that the cloudtrail plugin is sending events properly.  The
@@ -341,7 +339,7 @@
       json.value[/requestParameters/PublicAccessBlockConfiguration/BlockPublicAcls]=false or
       json.value[/requestParameters/PublicAccessBlockConfiguration/IgnorePublicAcls]=false)
   output:
-    A public access block for a bucket has been deleted
+    A pulic access block for a bucket has been deleted
     (requesting user=%ct.user,
      requesting IP=%ct.srcip,
      AWS region=%ct.region,

rules/falco_rules.yaml

@@ -1,5 +1,5 @@
 #
-# Copyright (C) 2022 The Falco Authors.
+# Copyright (C) 2020 The Falco Authors.
 #
 #
 # Licensed under the Apache License, Version 2.0 (the "License");
@@ -63,14 +63,11 @@
   condition: rename or remove
 
 - macro: spawned_process
-  condition: evt.type in (execve, execveat) and evt.dir=<
+  condition: evt.type = execve and evt.dir=<
 
 - macro: create_symlink
   condition: evt.type in (symlink, symlinkat) and evt.dir=<
 
-- macro: create_hardlink
-  condition: evt.type in (link, linkat) and evt.dir=<
-
 - macro: chmod
   condition: (evt.type in (chmod, fchmod, fchmodat) and evt.dir=<)
 
@@ -219,13 +216,13 @@
 - list: deb_binaries
   items: [dpkg, dpkg-preconfigu, dpkg-reconfigur, dpkg-divert, apt, apt-get, aptitude,
     frontend, preinst, add-apt-reposit, apt-auto-remova, apt-key,
-    apt-listchanges, unattended-upgr, apt-add-reposit, apt-cache, apt.systemd.dai
+    apt-listchanges, unattended-upgr, apt-add-reposit, apt-config, apt-cache, apt.systemd.dai
     ]
 
 # The truncated dpkg-preconfigu is intentional, process names are
 # truncated at the falcosecurity-libs level.
 - list: package_mgmt_binaries
-  items: [rpm_binaries, deb_binaries, update-alternat, gem, npm, pip, pip3, sane-utils.post, alternatives, chef-client, apk, snapd]
+  items: [rpm_binaries, deb_binaries, update-alternat, gem, pip, pip3, sane-utils.post, alternatives, chef-client, apk, snapd]
 
 - macro: package_mgmt_procs
   condition: proc.name in (package_mgmt_binaries)
@@ -443,9 +440,6 @@
 - list: shell_config_directories
   items: [/etc/zsh]
 
-- macro: user_known_shell_config_modifiers
-  condition: (never_true)
-
 - rule: Modify Shell Configuration File
   desc: Detect attempt to modify shell configuration files
   condition: >
@@ -455,7 +449,6 @@
      fd.directory in (shell_config_directories))
     and not proc.name in (shell_binaries)
     and not exe_running_docker_save
-    and not user_known_shell_config_modifiers
   output: >
     a shell configuration file has been modified (user=%user.name user_loginuid=%user.loginuid command=%proc.cmdline pcmdline=%proc.pcmdline file=%fd.name container_id=%container.id image=%container.image.repository)
   priority:
@@ -568,7 +561,7 @@
 
 # Qualys seems to run a variety of shell subprocesses, at various
 # levels. This checks at a few levels without the cost of a full
-# proc.aname, which traverses the full parent hierarchy.
+# proc.aname, which traverses the full parent heirarchy.
 - macro: run_by_qualys
   condition: >
     (proc.pname=qualys-cloud-ag or
@@ -779,9 +772,6 @@
 - macro: centrify_writing_krb
   condition: (proc.name in (adjoin,addns) and fd.name startswith /etc/krb5)
 
-- macro: sssd_writing_krb
-  condition: (proc.name=adcli and proc.aname[2]=sssd and fd.name startswith /etc/krb5)
-
 - macro: cockpit_writing_conf
   condition: >
     ((proc.pname=cockpit-kube-la or proc.aname[2]=cockpit-kube-la)
@@ -1110,7 +1100,7 @@
   condition: (proc.name=oc and fd.name startswith /etc/origin/node)
 
 - macro: keepalived_writing_conf
-  condition: (proc.name in (keepalived, kube-keepalived) and fd.name=/etc/keepalived/keepalived.conf)
+  condition: (proc.name=keepalived and fd.name=/etc/keepalived/keepalived.conf)
 
 - macro: etcd_manager_updating_dns
   condition: (container and proc.name=etcd-manager and fd.name=/etc/hosts)
@@ -1225,7 +1215,6 @@
     and not nginx_writing_certs
     and not chef_client_writing_conf
     and not centrify_writing_krb
-    and not sssd_writing_krb
     and not cockpit_writing_conf
     and not ipsec_writing_conf
     and not httpd_writing_ssl_conf
@@ -1882,6 +1871,19 @@
               container.image.repository in (falco_sensitive_mount_images) or
               container.image.repository startswith quay.io/sysdig/)
 
+# These container images are allowed to run with hostnetwork=true
+- list: falco_hostnetwork_images
+  items: [
+    gcr.io/google-containers/prometheus-to-sd,
+    gcr.io/projectcalico-org/typha,
+    gcr.io/projectcalico-org/node,
+    gke.gcr.io/gke-metadata-server,
+    gke.gcr.io/kube-proxy,
+    gke.gcr.io/netd-amd64,
+    k8s.gcr.io/ip-masq-agent-amd64
+    k8s.gcr.io/prometheus-to-sd,
+    ]
+
 # Add conditions to this macro (probably in a separate file,
 # overwriting this macro) to specify additional containers that are
 # allowed to perform sensitive mounts.
@@ -2197,7 +2199,7 @@
 #   output: "sshd sent error message to syslog (error=%evt.buffer)"
 #   priority: WARNING
 
-- macro: somebody_becoming_themselves
+- macro: somebody_becoming_themself
   condition: ((user.name=nobody and evt.arg.uid=nobody) or
               (user.name=www-data and evt.arg.uid=www-data) or
               (user.name=_apt and evt.arg.uid=_apt) or
@@ -2235,7 +2237,7 @@
     evt.type=setuid and evt.dir=>
     and (known_user_in_container or not container)
     and not (user.name=root or user.uid=0)
-    and not somebody_becoming_themselves
+    and not somebody_becoming_themself
     and not proc.name in (known_setuid_binaries, userexec_binaries, mail_binaries, docker_binaries,
                           nomachine_binaries)
     and not proc.name startswith "runc:"
@@ -2259,7 +2261,7 @@
     activity by any programs that can manage users, passwords, or permissions. sudo and su are excluded.
     Activity in containers is also excluded--some containers create custom users on top
     of a base linux distribution at startup.
-    Some innocuous command lines that don't actually change anything are excluded.
+    Some innocuous commandlines that don't actually change anything are excluded.
   condition: >
     spawned_process and proc.name in (user_mgmt_binaries) and
     not proc.name in (su, sudo, lastlog, nologin, unix_chkpwd) and not container and
@@ -2309,7 +2311,7 @@
 # explicitly enumerate the container images that you want to allow
 # access to EC2 metadata. In this main falco rules file, there isn't
 # any way to know all the containers that should have access, so any
-# container is allowed, by repeating the "container" macro. In the
+# container is alllowed, by repeating the "container" macro. In the
 # overridden macro, the condition would look something like
 # (container.image.repository = vendor/container-1 or
 # container.image.repository = vendor/container-2 or ...)
@@ -2572,15 +2574,15 @@
   condition: >
     (modify and (
       evt.arg.name contains "bash_history" or
-      evt.arg.name endswith "zsh_history" or
+      evt.arg.name contains "zsh_history" or
       evt.arg.name contains "fish_read_history" or
       evt.arg.name endswith "fish_history" or
       evt.arg.oldpath contains "bash_history" or
-      evt.arg.oldpath endswith "zsh_history" or
+      evt.arg.oldpath contains "zsh_history" or
       evt.arg.oldpath contains "fish_read_history" or
       evt.arg.oldpath endswith "fish_history" or
       evt.arg.path contains "bash_history" or
-      evt.arg.path endswith "zsh_history" or
+      evt.arg.path contains "zsh_history" or
       evt.arg.path contains "fish_read_history" or
       evt.arg.path endswith "fish_history"))
 
@@ -2588,7 +2590,7 @@
   condition: >
     (open_write and (
       fd.name contains "bash_history" or
-      fd.name endswith "zsh_history" or
+      fd.name contains "zsh_history" or
       fd.name contains "fish_read_history" or
       fd.name endswith "fish_history") and evt.arg.flags contains "O_TRUNC")
 
@@ -2627,7 +2629,7 @@
   items: [hyperkube, kubelet, k3s-agent]
 
 # This macro should be overridden in user rules as needed. This is useful if a given application
-# should not be ignored altogether with the user_known_chmod_applications list, but only in
+# should not be ignored alltogether with the user_known_chmod_applications list, but only in
 # specific conditions.
 - macro: user_known_set_setuid_or_setgid_bit_conditions
   condition: (never_true)
@@ -2707,17 +2709,7 @@
     (evt.arg.target in (sensitive_file_names) or evt.arg.target in (sensitive_directory_names))
   output: >
     Symlinks created over sensitive files (user=%user.name user_loginuid=%user.loginuid command=%proc.cmdline target=%evt.arg.target linkpath=%evt.arg.linkpath parent_process=%proc.pname)
-  priority: WARNING
-  tags: [file, mitre_exfiltration]
-
-- rule: Create Hardlink Over Sensitive Files
-  desc: Detect hardlink created over sensitive files
-  condition: >
-    create_hardlink and
-    (evt.arg.oldpath in (sensitive_file_names))
-  output: >
-    Hardlinks created over sensitive files (user=%user.name user_loginuid=%user.loginuid command=%proc.cmdline target=%evt.arg.oldpath linkpath=%evt.arg.newpath parent_process=%proc.pname)
-  priority: WARNING
+  priority: NOTICE
   tags: [file, mitre_exfiltration]
 
 - list: miner_ports
@@ -2822,13 +2814,13 @@
   desc: Miners typically connect to miner pools on common ports.
   condition: net_miner_pool and not trusted_images_query_miner_domain_dns
   enabled: false
-  output: Outbound connection to IP/Port flagged by https://cryptoioc.ch (command=%proc.cmdline port=%fd.rport ip=%fd.rip container=%container.info image=%container.image.repository)
+  output: Outbound connection to IP/Port flagged by cryptoioc.ch (command=%proc.cmdline port=%fd.rport ip=%fd.rip container=%container.info image=%container.image.repository)
   priority: CRITICAL
   tags: [network, mitre_execution]
 
 - rule: Detect crypto miners using the Stratum protocol
   desc: Miners typically specify the mining pool to connect to with a URI that begins with 'stratum+tcp'
-  condition: spawned_process and (proc.cmdline contains "stratum+tcp" or proc.cmdline contains "stratum2+tcp" or proc.cmdline contains "stratum+ssl" or proc.cmdline contains "stratum2+ssl")
+  condition: spawned_process and proc.cmdline contains "stratum+tcp"
   output: Possible miner running (command=%proc.cmdline container=%container.info image=%container.image.repository)
   priority: CRITICAL
   tags: [process, mitre_execution]
@@ -3047,7 +3039,7 @@
 # A privilege escalation to root through heap-based buffer overflow
 - rule: Sudo Potential Privilege Escalation
   desc: Privilege escalation vulnerability affecting sudo (<= 1.9.5p2). Executing sudo using sudoedit -s or sudoedit -i command with command-line argument that ends with a single backslash character from an unprivileged user it's possible to elevate the user privileges to root.
-  condition: spawned_process and user.uid != 0 and (proc.name=sudoedit or proc.name = sudo) and (proc.args contains -s or proc.args contains -i or proc.args contains --login) and (proc.args contains "\ " or proc.args endswith \)
+  condition: spawned_process and user.uid != 0 and proc.name=sudoedit and (proc.args contains -s or proc.args contains -i) and (proc.args contains "\ " or proc.args endswith \)
   output: "Detect Sudo Privilege Escalation Exploit (CVE-2021-3156) (user=%user.name parent=%proc.pname cmdline=%proc.cmdline %container.info)"
   priority: CRITICAL
   tags: [filesystem, mitre_privilege_escalation]
@@ -3065,17 +3057,13 @@
 - macro: mount_info
   condition: (proc.args="" or proc.args intersects ("-V", "-l", "-h"))
 
-- macro: user_known_mount_in_privileged_containers
-  condition: (never_true)
-
 - rule: Mount Launched in Privileged Container
-  desc: Detect file system mount happened inside a privileged container which might lead to container escape.
+  desc: Detect file system mount happened inside a privilegd container which might lead to container escape.
   condition: >
     spawned_process and container
     and container.privileged=true
     and proc.name=mount
     and not mount_info
-    and not user_known_mount_in_privileged_containers
   output: Mount was executed inside a privileged container (user=%user.name user_loginuid=%user.loginuid command=%proc.cmdline %container.info image=%container.image.repository:%container.image.tag)
   priority: WARNING
   tags: [container, cis, mitre_lateral_movement]
@@ -3132,17 +3120,7 @@
   priority: NOTICE
   tags: [network, process, mitre_command_and_control]
 
-# This rule helps detect CVE-2021-4034:
-# A privilege escalation to root through memory corruption
-- rule: Polkit Local Privilege Escalation Vulnerability (CVE-2021-4034)
-  desc: "This rule detects an attempt to exploit a privilege escalation vulnerability in Polkit's pkexec. By running specially crafted code, a local user can leverage this flaw to gain root privileges on a compromised system"
-  condition:
-    spawned_process and user.uid != 0 and proc.name=pkexec and proc.args = ''
-  output:
-    "Detect Polkit pkexec Local Privilege Escalation Exploit (CVE-2021-4034) (user=%user.loginname uid=%user.loginuid command=%proc.cmdline args=%proc.args)"
-  priority: CRITICAL
-  tags: [process, mitre_privilege_escalation]
-
 # Application rules have moved to application_rules.yaml. Please look
 # there if you want to enable them by adding to
 # falco_rules.local.yaml.
+

rules/k8s_audit_rules.yaml

@@ -152,19 +152,6 @@
   source: k8s_audit
   tags: [k8s]
 
-# These container images are allowed to run with hostnetwork=true
-- list: falco_hostnetwork_images
-  items: [
-    gcr.io/google-containers/prometheus-to-sd,
-    gcr.io/projectcalico-org/typha,
-    gcr.io/projectcalico-org/node,
-    gke.gcr.io/gke-metadata-server,
-    gke.gcr.io/kube-proxy,
-    gke.gcr.io/netd-amd64,
-    k8s.gcr.io/ip-masq-agent-amd64
-    k8s.gcr.io/prometheus-to-sd,
-    ]
-
 # Corresponds to K8s CIS Benchmark 1.7.4
 - rule: Create HostNetwork Pod
   desc: Detect an attempt to start a pod using the host network.
@@ -366,7 +353,7 @@
   tags: [k8s]
 
 # Detect any attempt to create a ClusterRoleBinding to the cluster-admin user
-# (expand this to any built-in cluster role that does "sensitive" things)
+# (exapand this to any built-in cluster role that does "sensitive" things)
 - rule: Attach to cluster-admin Role
   desc: Detect any attempt to create a ClusterRoleBinding to the cluster-admin user
   condition: kevt and clusterrolebinding and kcreate and ka.req.binding.role=cluster-admin
@@ -572,7 +559,7 @@
 # cluster creation. This may signify a permission setting too broader.
 # As we can't check for role of the user on a general ka.* event, this
 # may or may not be an administrator. Customize the full_admin_k8s_users
-# list to your needs, and activate at your discretion.
+# list to your needs, and activate at your discrection.
 
 # # How to test:
 # # Execute any kubectl command connected using default cluster user, as:

scripts/debian/postinst.in

@@ -18,7 +18,7 @@
 set -e
 
 DKMS_PACKAGE_NAME="@PACKAGE_NAME@"
-DKMS_VERSION="@DRIVER_VERSION@"
+DKMS_VERSION="@PROBE_VERSION@"
 NAME="@PACKAGE_NAME@"
 
 postinst_found=0

scripts/debian/prerm.in

@@ -18,7 +18,7 @@
 set -e
 
 DKMS_PACKAGE_NAME="@PACKAGE_NAME@"
-DKMS_VERSION="@DRIVER_VERSION@"
+DKMS_VERSION="@PROBE_VERSION@"
 
 case "$1" in
 	remove|upgrade|deconfigure)

scripts/falco-driver-loader

@@ -160,26 +160,15 @@ load_kernel_module_compile() {
 		echo "make CC=${CURRENT_GCC} \$@" >> /tmp/falco-dkms-make
 		chmod +x /tmp/falco-dkms-make
 		if dkms install --directive="MAKE='/tmp/falco-dkms-make'" -m "${DRIVER_NAME}" -v "${DRIVER_VERSION}" -k "${KERNEL_RELEASE}" 2>/dev/null; then
-			echo "* ${DRIVER_NAME} module installed in dkms"
-			KO_FILE="/var/lib/dkms/${DRIVER_NAME}/${DRIVER_VERSION}/${KERNEL_RELEASE}/${ARCH}/module/${DRIVER_NAME}"
-			if [ -f "$KO_FILE.ko" ]; then
-				KO_FILE="$KO_FILE.ko"
-			elif [ -f "$KO_FILE.ko.gz" ]; then
-				KO_FILE="$KO_FILE.ko.gz"
-			elif [ -f "$KO_FILE.ko.xz" ]; then
-				KO_FILE="$KO_FILE.ko.xz"
-			elif [ -f "$KO_FILE.ko.zst" ]; then
-				KO_FILE="$KO_FILE.ko.zst"
-			else
-				>&2 echo "${DRIVER_NAME} module file not found"
-				return
-			fi
-			echo "* ${DRIVER_NAME} module found: ${KO_FILE}"
-			echo "* Trying insmod"
-			chcon -t modules_object_t "$KO_FILE" > /dev/null 2>&1 || true
-			if insmod "$KO_FILE" > /dev/null 2>&1; then
+			echo "* ${DRIVER_NAME} module installed in dkms, trying to insmod"
+			chcon -t modules_object_t "/var/lib/dkms/${DRIVER_NAME}/${DRIVER_VERSION}/${KERNEL_RELEASE}/${ARCH}/module/${DRIVER_NAME}.ko" > /dev/null 2>&1 || true
+			chcon -t modules_object_t "/var/lib/dkms/${DRIVER_NAME}/${DRIVER_VERSION}/${KERNEL_RELEASE}/${ARCH}/module/${DRIVER_NAME}.ko.xz" > /dev/null 2>&1 || true
+			if insmod "/var/lib/dkms/${DRIVER_NAME}/${DRIVER_VERSION}/${KERNEL_RELEASE}/${ARCH}/module/${DRIVER_NAME}.ko" > /dev/null 2>&1; then
 				echo "* Success: ${DRIVER_NAME} module found and loaded in dkms"
 				exit 0
+			elif insmod "/var/lib/dkms/${DRIVER_NAME}/${DRIVER_VERSION}/${KERNEL_RELEASE}/${ARCH}/module/${DRIVER_NAME}.ko.xz" > /dev/null 2>&1; then
+				echo "* Success: ${DRIVER_NAME} module found and loaded in dkms (xz)"
+				exit 0
 			else
 				echo "* Unable to insmod ${DRIVER_NAME} module"
 			fi
@@ -207,12 +196,8 @@ load_kernel_module_download() {
 	if curl -L --create-dirs "${FALCO_DRIVER_CURL_OPTIONS}" -o "${HOME}/.falco/${FALCO_KERNEL_MODULE_FILENAME}" "${URL}"; then
 		echo "* Download succeeded"
 		chcon -t modules_object_t "${HOME}/.falco/${FALCO_KERNEL_MODULE_FILENAME}" > /dev/null 2>&1 || true
-		if insmod "${HOME}/.falco/${FALCO_KERNEL_MODULE_FILENAME}"; then
-			echo "* Success: ${DRIVER_NAME} module found and inserted"
-			exit 0
-		else
-			>&2 echo "Unable to insmod the prebuilt ${DRIVER_NAME} module"
-		fi	
+		insmod "${HOME}/.falco/${FALCO_KERNEL_MODULE_FILENAME}" && echo "* Success: ${DRIVER_NAME} module found and inserted"
+		exit $?
 	else
 		>&2 echo "Unable to find a prebuilt ${DRIVER_NAME} module"
 		return
@@ -256,6 +241,11 @@ load_kernel_module() {
 		exit 0
 	fi
 
+	echo "* Trying to load a system ${DRIVER_NAME} module, if present"
+	if modprobe "${DRIVER_NAME}" > /dev/null 2>&1; then
+		echo "* Success: ${DRIVER_NAME} module found and loaded with modprobe"
+		exit 0
+	fi
 
 	echo "* Looking for a ${DRIVER_NAME} module locally (kernel ${KERNEL_RELEASE})"
 
@@ -278,13 +268,6 @@ load_kernel_module() {
 		load_kernel_module_compile
 	fi
 
-	# Last try (might load a previous driver version)
-	echo "* Trying to load a system ${DRIVER_NAME} module, if present"
-	if modprobe "${DRIVER_NAME}" > /dev/null 2>&1; then
-		echo "* Success: ${DRIVER_NAME} module found and loaded with modprobe"
-		exit 0
-	fi
-
 	# Not able to download a prebuilt module nor to compile one on-the-fly
 	>&2 echo "Consider compiling your own ${DRIVER_NAME} driver and loading it or getting in touch with the Falco community"
 	exit 1
@@ -514,7 +497,7 @@ print_usage() {
 	echo "  --source-only  skip execution and allow sourcing in another script"
 	echo ""
 	echo "Environment variables:"
-	echo "  DRIVERS_REPO             specify a different URL where to look for prebuilt Falco drivers"
+	echo "  DRIVER_REPO              specify a different URL where to look for prebuilt Falco drivers"
 	echo "  DRIVER_NAME              specify a different name for the driver"
 	echo "  DRIVER_INSECURE_DOWNLOAD whether you want to allow insecure downloads or not"
 	echo ""
@@ -547,8 +530,8 @@ if [[ -z "$MAX_RMMOD_WAIT" ]]; then
 	MAX_RMMOD_WAIT=60
 fi
 
-DRIVER_VERSION="@DRIVER_VERSION@"
-DRIVER_NAME=${DRIVER_NAME:-"@DRIVER_NAME@"}
+DRIVER_VERSION="@PROBE_VERSION@"
+DRIVER_NAME=${DRIVER_NAME:-"@PROBE_NAME@"}
 FALCO_VERSION="@FALCO_VERSION@"
 
 DRIVER="module"

scripts/rpm/postinstall.in

@@ -15,7 +15,7 @@
 # limitations under the License.
 #
 
-mod_version="@DRIVER_VERSION@"
+mod_version="@PROBE_VERSION@"
 dkms add -m falco -v $mod_version --rpm_safe_upgrade
 if [ `uname -r | grep -c "BOOT"` -eq 0 ] && [ -e /lib/modules/`uname -r`/build/include ]; then
 	dkms build -m falco -v $mod_version

scripts/rpm/preuninstall.in

@@ -15,5 +15,5 @@
 # limitations under the License.
 #
 
-mod_version="@DRIVER_VERSION@"
+mod_version="@PROBE_VERSION@"
 dkms remove -m falco -v $mod_version --all --rpm_safe_upgrade

test/CMakeLists.txt

@@ -1,4 +1,7 @@
 add_subdirectory(trace_files)
 
+add_custom_target(test-trace-files ALL)
+add_dependencies(test-trace-files trace-files-base-scap trace-files-psp trace-files-k8s-audit trace-files-plugins)
+
 add_subdirectory(plugins)
 add_subdirectory(confs/plugins)

test/README.md

@@ -71,7 +71,7 @@ The `falco_traces.yaml` test suite gets generated through the `falco_traces.yaml
 
 ### falco_tests_package
 
-The `falco_tests_package.yaml` test suite requires some additional setup steps to be successfully run on your local machine.
+The `falco_tests_package.yaml` test suite requires some additional setup steps to be succesfully run on your local machine.
 
 In particular, it requires some runners (ie., docker images) to be already built and present into your local machine.
 

test/falco_k8s_audit_tests.yaml

@@ -49,7 +49,6 @@ trace_files: !mux
     detect: False
     rules_file:
       - ../rules/falco_rules.yaml
-      - ../rules/k8s_audit_rules.yaml
       - ./rules/k8s_audit/engine_v4_k8s_audit_rules.yaml
       - ./rules/k8s_audit/trust_nginx_container.yaml
     trace_file: trace_files/k8s_audit/create_nginx_pod_privileged.json
@@ -75,7 +74,6 @@ trace_files: !mux
     detect: False
     rules_file:
       - ../rules/falco_rules.yaml
-      - ../rules/k8s_audit_rules.yaml
       - ./rules/k8s_audit/engine_v4_k8s_audit_rules.yaml
       - ./rules/k8s_audit/trust_nginx_container.yaml
     trace_file: trace_files/k8s_audit/create_nginx_pod_hostnetwork.json

test/falco_test.py

@@ -524,10 +524,10 @@ class FalcoTest(Test):
                 ["md5sum", "/boot/config-{}".format(kernel_release)]).rstrip()
             config_hash = md5_output.split(" ")[0]
 
-            driver_filename = "falco-{}-{}-{}-{}.ko".format(
+            probe_filename = "falco-{}-{}-{}-{}.ko".format(
                 falco_version, arch, kernel_release, config_hash)
             driver_path = os.path.join(self.falcodir, "driver", "falco.ko")
-            module_path = os.path.join(self.module_dir, driver_filename)
+            module_path = os.path.join(self.module_dir, probe_filename)
             self.log.debug("Copying {} to {}".format(driver_path, module_path))
             shutil.copyfile(driver_path, module_path)
 

test/falco_tests.yaml

@@ -44,8 +44,8 @@ trace_files: !mux
       - not_equals_after_evttype: [execve]
       - not_after_evttype: [execve]
       - leading_trailing_evttypes: [execve,open]
-      - leading_multitrailing_evttypes: [connect,execve,open]
-      - leading_multitrailing_evttypes_using_in: [connect,execve,open]
+      - leading_multtrailing_evttypes: [connect,execve,open]
+      - leading_multtrailing_evttypes_using_in: [connect,execve,open]
       - not_equals_at_end: [all]
       - not_at_end: [all]
       - not_before_trailing_evttype: [all]

test/falco_tests_plugins.yaml

@@ -54,21 +54,21 @@ trace_files: !mux
 
   multiple_source_plugins:
     exit_status: 1
-    stderr_contains: "Can not load multiple source plugins. cloudtrail already loaded."
+    stderr_contains: "Runtime error: Can not load multiple source plugins. cloudtrail already loaded. Exiting."
     conf_file: BUILD_DIR/test/confs/plugins/multiple_source_plugins.yaml
     rules_file:
       - rules/plugins/cloudtrail_create_instances.yaml
 
   incompatible_extract_sources:
     exit_status: 1
-    stderr_contains: "Extractor plugin not compatible with event source aws_cloudtrail."
+    stderr_contains: "Runtime error: Extractor plugin not compatible with event source aws_cloudtrail. Exiting."
     conf_file: BUILD_DIR/test/confs/plugins/incompatible_extract_sources.yaml
     rules_file:
       - rules/plugins/cloudtrail_create_instances.yaml
 
   overlap_extract_sources:
     exit_status: 1
-    stderr_contains: "Extractor plugins have overlapping compatible event source test_source."
+    stderr_contains: "Runtime error: Extractor plugins have overlapping compatible event source test_source. Exiting."
     conf_file: BUILD_DIR/test/confs/plugins/overlap_extract_sources.yaml
     rules_file:
       - rules/plugins/cloudtrail_create_instances.yaml
@@ -82,7 +82,7 @@ trace_files: !mux
 
   incompat_plugin_rules_version:
     exit_status: 1
-    stderr_contains: "Plugin cloudtrail version .* not compatible with required plugin version 100000.0.0."
+    stderr_contains: "Runtime error: Plugin cloudtrail version 0.1.0 not compatible with required plugin version 100000.0.0. Exiting."
     conf_file: BUILD_DIR/test/confs/plugins/cloudtrail_json_create_instances.yaml
     rules_file:
       - rules/plugins/cloudtrail_incompat_plugin_version.yaml
@@ -103,20 +103,4 @@ trace_files: !mux
       - Cloudtrail Create Instance
     stderr_contains: "Rule Cloudtrail Create Instance: warning .unknown-source.: unknown source aws_cloudtrail, skipping"
 
-  no_plugins_unknown_source_macro:
-    detect: False
-    rules_file:
-      - rules/plugins/cloudtrail_macro.yaml
-    trace_file: trace_files/empty.scap
-    stderr_contains: "Macro Some Cloudtrail Macro: warning .unknown-source.: unknown source aws_cloudtrail, skipping"
-
-  no_plugins_unknown_source_rule_exception:
-    detect: False
-    rules_file:
-      - rules/plugins/cloudtrail_create_instances_exceptions.yaml
-    trace_file: trace_files/empty.scap
-    rules_warning:
-      - Cloudtrail Create Instance
-    stderr_contains: "Rule Cloudtrail Create Instance: warning .unknown-source.: unknown source aws_cloudtrail, skipping"
-
 

test/rules/falco_rules_warnings.yaml

@@ -56,13 +56,13 @@
   output: "None"
   priority: WARNING
 
-- rule: leading_multitrailing_evttypes
+- rule: leading_multtrailing_evttypes
   desc: one evttype at beginning, multiple at end
   condition: evt.type=execve and proc.name=foo or evt.type=open or evt.type=connect
   output: "None"
   priority: WARNING
 
-- rule: leading_multitrailing_evttypes_using_in
+- rule: leading_multtrailing_evttypes_using_in
   desc: one evttype at beginning, multiple at end, using in
   condition: evt.type=execve and proc.name=foo or evt.type in (open, connect)
   output: "None"

test/rules/k8s_audit/engine_v4_k8s_audit_rules.yaml

@@ -246,7 +246,7 @@
   tags: [k8s]
 
 # Detect any attempt to create a ClusterRoleBinding to the cluster-admin user
-# (expand this to any built-in cluster role that does "sensitive" things)
+# (exapand this to any built-in cluster role that does "sensitive" things)
 - rule: Attach to cluster-admin Role
   desc: Detect any attempt to create a ClusterRoleBinding to the cluster-admin user
   condition: kevt and clusterrolebinding and kcreate and ka.req.binding.role=cluster-admin

test/rules/plugins/cloudtrail_create_instances_exceptions.yaml

@@ -1,9 +0,0 @@
-- rule: Cloudtrail Create Instance
-  desc: Detect Creating an EC2 Instance
-  condition: evt.num > 0 and ct.name="StartInstances"
-  output: EC2 Instance Created (evtnum=%evt.num info=%evt.plugininfo id=%ct.id user name=%json.value[/userIdentity/userName])
-  exceptions:
-  - name: user_secreid
-    fields: [aws.user, aws.region]
-  priority: INFO
-  source: aws_cloudtrail

test/rules/plugins/cloudtrail_macro.yaml

@@ -1,4 +0,0 @@
-- macro: Some Cloudtrail Macro
-  condition: aws.user=bob
-  source: aws_cloudtrail
-

test/trace_files/CMakeLists.txt

@@ -2,11 +2,19 @@ add_subdirectory(k8s_audit)
 add_subdirectory(psp)
 add_subdirectory(plugins)
 
-include(copy_files_to_build_dir)
-
 # Note: list of traces is created at cmake time, not build time
 file(GLOB test_trace_files
 	"${CMAKE_CURRENT_SOURCE_DIR}/*.scap")
 
-copy_files_to_build_dir("${test_trace_files}" base-scap)
+foreach(trace_file_path ${test_trace_files})
+	get_filename_component(trace_file ${trace_file_path} NAME)
+	add_custom_target(test-trace-${trace_file} ALL
+		DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/${trace_file})
+	add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		COMMAND ${CMAKE_COMMAND} -E copy ${trace_file_path} ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		DEPENDS ${trace_file_path})
+	list(APPEND BASE_SCAP_TRACE_FILES_TARGETS test-trace-${trace_file})
+endforeach()
 
+add_custom_target(trace-files-base-scap ALL)
+add_dependencies(trace-files-base-scap ${BASE_SCAP_TRACE_FILES_TARGETS})

test/trace_files/k8s_audit/CMakeLists.txt

@@ -1,7 +1,16 @@
-include(copy_files_to_build_dir)
-
 # Note: list of traces is created at cmake time, not build time
 file(GLOB test_trace_files
 	"${CMAKE_CURRENT_SOURCE_DIR}/*.json")
 
-copy_files_to_build_dir("${test_trace_files}" k8s-audit)
+foreach(trace_file_path ${test_trace_files})
+	get_filename_component(trace_file ${trace_file_path} NAME)
+	add_custom_target(test-trace-${trace_file} ALL
+		DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/${trace_file})
+	add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		COMMAND ${CMAKE_COMMAND} -E copy ${trace_file_path} ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		DEPENDS ${trace_file_path})
+	list(APPEND K8S_AUDIT_TRACE_FILES_TARGETS test-trace-${trace_file})
+endforeach()
+
+add_custom_target(trace-files-k8s-audit ALL)
+add_dependencies(trace-files-k8s-audit ${K8S_AUDIT_TRACE_FILES_TARGETS})

test/trace_files/plugins/CMakeLists.txt

@@ -1,7 +1,16 @@
-include(copy_files_to_build_dir)
-
 # Note: list of traces is created at cmake time, not build time
 file(GLOB test_trace_files
 	"${CMAKE_CURRENT_SOURCE_DIR}/*.json")
 
-copy_files_to_build_dir("${test_trace_files}" plugins)
+foreach(trace_file_path ${test_trace_files})
+	get_filename_component(trace_file ${trace_file_path} NAME)
+	add_custom_target(test-trace-${trace_file} ALL
+		DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/${trace_file})
+	add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		COMMAND ${CMAKE_COMMAND} -E copy ${trace_file_path} ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		DEPENDS ${trace_file_path})
+	list(APPEND PLUGINS_TRACE_FILES_TARGETS test-trace-${trace_file})
+endforeach()
+
+add_custom_target(trace-files-plugins ALL)
+add_dependencies(trace-files-plugins ${PLUGINS_TRACE_FILES_TARGETS})

test/trace_files/psp/CMakeLists.txt

@@ -1,8 +1,17 @@
-include(copy_files_to_build_dir)
-
 # Note: list of traces is created at cmake time, not build time
 file(GLOB test_trace_files
 	"${CMAKE_CURRENT_SOURCE_DIR}/*.json"
 	"${CMAKE_CURRENT_SOURCE_DIR}/*.scap")
 
-copy_files_to_build_dir("${test_trace_files}" psp)
+foreach(trace_file_path ${test_trace_files})
+	get_filename_component(trace_file ${trace_file_path} NAME)
+	add_custom_target(test-trace-${trace_file} ALL
+		DEPENDS ${CMAKE_CURRENT_BINARY_DIR}/${trace_file})
+	add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		COMMAND ${CMAKE_COMMAND} -E copy ${trace_file_path} ${CMAKE_CURRENT_BINARY_DIR}/${trace_file}
+		DEPENDS ${trace_file_path})
+	list(APPEND PSP_TRACE_FILES_TARGETS test-trace-${trace_file})
+endforeach()
+
+add_custom_target(trace-files-psp ALL)
+add_dependencies(trace-files-psp ${PSP_TRACE_FILES_TARGETS})

tests/CMakeLists.txt

@@ -17,25 +17,17 @@
 if(MINIMAL_BUILD)
   set(
     FALCO_TESTS_SOURCES
-    ${PROJECT_SOURCE_DIR}/userspace/falco/app_runnable_action.cpp
-    ${PROJECT_SOURCE_DIR}/userspace/falco/app_action_manager.cpp
-    ${PROJECT_SOURCE_DIR}/userspace/falco/logger.cpp
     test_base.cpp
     engine/test_rulesets.cpp
     engine/test_falco_utils.cpp
-    falco/test_actions.cpp
     falco/test_configuration.cpp
   )
 else()
   set(
     FALCO_TESTS_SOURCES
-    ${PROJECT_SOURCE_DIR}/userspace/falco/app_runnable_action.cpp
-    ${PROJECT_SOURCE_DIR}/userspace/falco/app_action_manager.cpp
-    ${PROJECT_SOURCE_DIR}/userspace/falco/logger.cpp
     test_base.cpp
     engine/test_rulesets.cpp
     engine/test_falco_utils.cpp
-    falco/test_actions.cpp
     falco/test_configuration.cpp
     falco/test_webserver.cpp
   )

tests/engine/test_rulesets.cpp

@@ -42,9 +42,8 @@ TEST_CASE("Should enable/disable for exact match w/ default ruleset", "[rulesets
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable("one_rule", exact_match, enabled);
 	REQUIRE(r.num_rules_for_ruleset(default_ruleset) == 1);
@@ -58,9 +57,8 @@ TEST_CASE("Should enable/disable for exact match w/ specific ruleset", "[ruleset
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable("one_rule", exact_match, enabled, non_default_ruleset);
 	REQUIRE(r.num_rules_for_ruleset(non_default_ruleset) == 1);
@@ -78,9 +76,8 @@ TEST_CASE("Should not enable for exact match different rule name", "[rulesets]")
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable("some_other_rule", exact_match, enabled);
 	REQUIRE(r.num_rules_for_ruleset(default_ruleset) == 0);
@@ -91,9 +88,8 @@ TEST_CASE("Should enable/disable for exact match w/ substring and default rulese
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable("one_rule", substring_match, enabled);
 	REQUIRE(r.num_rules_for_ruleset(default_ruleset) == 1);
@@ -107,9 +103,8 @@ TEST_CASE("Should not enable for substring w/ exact_match", "[rulesets]")
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable("one_", exact_match, enabled);
 	REQUIRE(r.num_rules_for_ruleset(default_ruleset) == 0);
@@ -120,9 +115,8 @@ TEST_CASE("Should enable/disable for prefix match w/ default ruleset", "[ruleset
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable("one_", substring_match, enabled);
 	REQUIRE(r.num_rules_for_ruleset(default_ruleset) == 1);
@@ -136,9 +130,8 @@ TEST_CASE("Should enable/disable for suffix match w/ default ruleset", "[ruleset
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable("_rule", substring_match, enabled);
 	REQUIRE(r.num_rules_for_ruleset(default_ruleset) == 1);
@@ -152,9 +145,8 @@ TEST_CASE("Should enable/disable for substring match w/ default ruleset", "[rule
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable("ne_ru", substring_match, enabled);
 	REQUIRE(r.num_rules_for_ruleset(default_ruleset) == 1);
@@ -168,9 +160,8 @@ TEST_CASE("Should enable/disable for substring match w/ specific ruleset", "[rul
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable("ne_ru", substring_match, enabled, non_default_ruleset);
 	REQUIRE(r.num_rules_for_ruleset(non_default_ruleset) == 1);
@@ -188,10 +179,9 @@ TEST_CASE("Should enable/disable for tags w/ default ruleset", "[rulesets]")
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 	std::set<std::string> want_tags = {"some_tag"};
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable_tags(want_tags, enabled);
 	REQUIRE(r.num_rules_for_ruleset(default_ruleset) == 1);
@@ -205,10 +195,9 @@ TEST_CASE("Should enable/disable for tags w/ specific ruleset", "[rulesets]")
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 	std::set<std::string> want_tags = {"some_tag"};
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable_tags(want_tags, enabled, non_default_ruleset);
 	REQUIRE(r.num_rules_for_ruleset(non_default_ruleset) == 1);
@@ -226,10 +215,9 @@ TEST_CASE("Should not enable for different tags", "[rulesets]")
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 	std::set<std::string> want_tags = {"some_different_tag"};
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable_tags(want_tags, enabled);
 	REQUIRE(r.num_rules_for_ruleset(non_default_ruleset) == 0);
@@ -240,10 +228,9 @@ TEST_CASE("Should enable/disable for overlapping tags", "[rulesets]")
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> filter = create_filter();
 	string rule_name = "one_rule";
-	string source = "syscall";
 	std::set<std::string> want_tags = {"some_tag", "some_different_tag"};
 
-	r.add(source, rule_name, tags, filter);
+	r.add(rule_name, tags, filter);
 
 	r.enable_tags(want_tags, enabled);
 	REQUIRE(r.num_rules_for_ruleset(default_ruleset) == 1);
@@ -254,17 +241,16 @@ TEST_CASE("Should enable/disable for overlapping tags", "[rulesets]")
 
 TEST_CASE("Should enable/disable for incremental adding tags", "[rulesets]")
 {
-	string source = "syscall";
 	falco_ruleset r;
 	std::shared_ptr<gen_event_filter> rule1_filter = create_filter();
 	string rule1_name = "one_rule";
 	std::set<std::string> rule1_tags = {"rule1_tag"};
-	r.add(source, rule1_name, rule1_tags, rule1_filter);
+	r.add(rule1_name, rule1_tags, rule1_filter);
 
 	std::shared_ptr<gen_event_filter> rule2_filter = create_filter();
 	string rule2_name = "two_rule";
 	std::set<std::string> rule2_tags = {"rule2_tag"};
-	r.add(source, rule2_name, rule2_tags, rule2_filter);
+	r.add(rule2_name, rule2_tags, rule2_filter);
 
 	std::set<std::string> want_tags;
 

tests/falco/test_actions.cpp

@@ -1,387 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "app_action_manager.h"
-
-#include <algorithm>
-#include <list>
-#include <memory>
-#include <string>
-#include <vector>
-
-#include <catch.hpp>
-
-// Test actions just record the order they were run (or skipped)
-class test_action : public falco::app::runnable_action {
-public:
-
-	static std::vector<std::string> s_actions_run;
-
-	test_action(const std::string &name,
-		    const std::string &group,
-		    const std::list<std::string> &prerequsites,
-		    run_result res)
-		: m_name(name),
-		  m_group(group),
-		  m_prerequsites(prerequsites),
-		  m_res(res)
-		{
-		}
-
-	~test_action()
-		{
-		}
-
-	const std::string &name()
-	{
-		return m_name;
-	}
-
-	const std::string &group()
-	{
-		return m_group;
-	}
-
-	const std::list<std::string> &prerequsites()
-	{
-		return m_prerequsites;
-	}
-
-	run_result run()
-	{
-		s_actions_run.push_back(m_name);
-		return m_res;
-	}
-
-private:
-	std::string m_name;
-	std::string m_group;
-	std::list<std::string> m_prerequsites;
-	run_result m_res;
-};
-
-std::vector<std::string> test_action::s_actions_run;
-
-static std::list<std::string> empty;
-static std::list<std::string> prereq_a = {"a"};
-static std::list<std::string> prereq_aa = {"aa"};
-static std::list<std::string> prereq_ab = {"ab"};
-static std::list<std::string> prereq_m = {"m"};
-static std::list<std::string> prereq_n = {"n"};
-
-// The action names denote the dependency order e.g. "a", "b", "c" are
-// all independent, "aa" and "ab" depend on a but are independent of
-// each other, "aaa" "aab" depend on "aa" but are independent, etc.
-
-static falco::app::runnable_action::run_result success_proceed{true, "", true};
-
-static falco::app::runnable_action::run_result success_noproceed{true, "", false};
-
-
-static std::shared_ptr<test_action> a = std::make_shared<test_action>(std::string("a"),
-								      std::string("init"),
-								      empty,
-								      success_proceed);
-
-static std::shared_ptr<test_action> a_noproceed = std::make_shared<test_action>(std::string("a"),
-										std::string("init"),
-										empty,
-										success_noproceed);
-
-static std::shared_ptr<test_action> b = std::make_shared<test_action>(std::string("b"),
-								      std::string("init"),
-								      empty,
-								      success_proceed);
-
-static std::shared_ptr<test_action> c = std::make_shared<test_action>(std::string("c"),
-								      std::string("init"),
-								      empty,
-								      success_proceed);
-
-static std::shared_ptr<test_action> d = std::make_shared<test_action>(std::string("d"),
-								      std::string("init"),
-								      empty,
-								      success_proceed);
-
-std::shared_ptr<test_action> aa = std::make_shared<test_action>(std::string("aa"),
-								std::string("init"),
-								prereq_a,
-								success_proceed);
-
-std::shared_ptr<test_action> ab = std::make_shared<test_action>(std::string("ab"),
-								std::string("init"),
-								prereq_a,
-								success_proceed);
-
-std::shared_ptr<test_action> aa_noproceed = std::make_shared<test_action>(std::string("aa"),
-									  std::string("init"),
-									  prereq_a,
-									  success_noproceed);
-
-std::shared_ptr<test_action> aaa = std::make_shared<test_action>(std::string("aaa"),
-								 std::string("init"),
-								 prereq_aa,
-								 success_proceed);
-
-std::shared_ptr<test_action> aab = std::make_shared<test_action>(std::string("aab"),
-								 std::string("init"),
-								 prereq_aa,
-								 success_proceed);
-
-std::shared_ptr<test_action> aba = std::make_shared<test_action>(std::string("aba"),
-								 std::string("init"),
-								 prereq_ab,
-								 success_proceed);
-
-static std::shared_ptr<test_action> m = std::make_shared<test_action>(std::string("m"),
-								      std::string("run"),
-								      empty,
-								      success_proceed);
-
-static std::shared_ptr<test_action> ma = std::make_shared<test_action>(std::string("ma"),
-								       std::string("run"),
-								       prereq_m,
-								       success_proceed);
-
-static std::shared_ptr<test_action> n = std::make_shared<test_action>(std::string("n"),
-								      std::string("run"),
-								      empty,
-								      success_proceed);
-
-static std::shared_ptr<test_action> na = std::make_shared<test_action>(std::string("na"),
-								       std::string("run"),
-								       prereq_n,
-								       success_proceed);
-
-static std::vector<std::string>::iterator find_action(const std::string &name,
-						      std::vector<std::string>::iterator begin = test_action::s_actions_run.begin())
-{
-	return std::find(begin,
-			 test_action::s_actions_run.end(),
-			 name);
-}
-
-static bool action_is_found(const std::string &name,
-			    std::vector<std::string>::iterator begin = test_action::s_actions_run.begin())
-{
-	auto it = find_action(name, begin);
-
-	return (it != test_action::s_actions_run.end());
-}
-
-TEST_CASE("action manager can add and run actions", "[actions]")
-{
-	std::list<std::string> groups = {"init", "run"};
-
-	SECTION("Two independent")
-	{
-		falco::app::action_manager amgr;
-		amgr.set_groups(groups);
-
-		test_action::s_actions_run.clear();
-
-		amgr.add(a);
-		amgr.add(b);
-
-		amgr.run();
-
-		// Can't compare to any direct vector as order is not guaranteed
-		REQUIRE(action_is_found(a->name()) == true);
-		REQUIRE(action_is_found(b->name()) == true);
-	}
-
-	SECTION("Two dependent")
-	{
-		falco::app::action_manager amgr;
-		amgr.set_groups(groups);
-
-		test_action::s_actions_run.clear();
-
-		amgr.add(a);
-		amgr.add(aa);
-
-		amgr.run();
-
-		std::vector<std::string> exp_actions_run = {"a", "aa"};
-		REQUIRE(test_action::s_actions_run == exp_actions_run);
-	}
-
-	SECTION("One independent, two dependent")
-	{
-		falco::app::action_manager amgr;
-		amgr.set_groups(groups);
-
-		test_action::s_actions_run.clear();
-
-		amgr.add(a);
-		amgr.add(aa);
-		amgr.add(b);
-
-		amgr.run();
-
-		// Can't compare to any direct vector as order is not guaranteed
-		REQUIRE(action_is_found(a->name()) == true);
-		REQUIRE(action_is_found(aa->name()) == true);
-		REQUIRE(action_is_found(b->name()) == true);
-
-		// Ensure that aa appears after a
-		auto it = find_action(a->name());
-		REQUIRE(action_is_found(aa->name(), it) == true);
-	}
-
-	SECTION("Two dependent, first does not proceed")
-	{
-		falco::app::action_manager amgr;
-		amgr.set_groups(groups);
-
-		test_action::s_actions_run.clear();
-
-		amgr.add(a_noproceed);
-		amgr.add(aa);
-
-		amgr.run();
-
-		std::vector<std::string> exp_actions_run = {"a"};
-		REQUIRE(test_action::s_actions_run == exp_actions_run);
-	}
-
-	SECTION("Two dependent, second does not proceed")
-	{
-		falco::app::action_manager amgr;
-		amgr.set_groups(groups);
-
-		test_action::s_actions_run.clear();
-
-		amgr.add(a);
-		amgr.add(aa_noproceed);
-
-		amgr.run();
-
-		std::vector<std::string> exp_actions_run = {"a", "aa"};
-		REQUIRE(test_action::s_actions_run == exp_actions_run);
-	}
-
-	SECTION("Three dependent, first does not proceed")
-	{
-		falco::app::action_manager amgr;
-		amgr.set_groups(groups);
-
-		test_action::s_actions_run.clear();
-
-		amgr.add(a_noproceed);
-		amgr.add(aa);
-		amgr.add(aaa);
-
-		amgr.run();
-
-		std::vector<std::string> exp_actions_run = {"a"};
-		REQUIRE(test_action::s_actions_run == exp_actions_run);
-	}
-
-	SECTION("Three dependent, second does not proceed")
-	{
-		falco::app::action_manager amgr;
-		amgr.set_groups(groups);
-
-		test_action::s_actions_run.clear();
-
-		amgr.add(a);
-		amgr.add(aa_noproceed);
-		amgr.add(aaa);
-
-		amgr.run();
-
-		std::vector<std::string> exp_actions_run = {"a", "aa"};
-		REQUIRE(test_action::s_actions_run == exp_actions_run);
-	}
-
-	SECTION("Groups")
-	{
-		falco::app::action_manager amgr;
-		amgr.set_groups(groups);
-
-		test_action::s_actions_run.clear();
-
-		amgr.add(ma);
-		amgr.add(m);
-		amgr.add(aa);
-		amgr.add(a);
-
-		amgr.run();
-
-		std::vector<std::string> exp_actions_run = {"a", "aa", "m", "ma"};
-		REQUIRE(test_action::s_actions_run == exp_actions_run);
-	}
-
-	SECTION("Complex")
-	{
-		falco::app::action_manager amgr;
-		amgr.set_groups(groups);
-
-		test_action::s_actions_run.clear();
-
-		amgr.add(a);
-		amgr.add(b);
-		amgr.add(c);
-		amgr.add(d);
-		amgr.add(aa);
-		amgr.add(ab);
-		amgr.add(aaa);
-		amgr.add(aab);
-		amgr.add(aba);
-		amgr.add(m);
-		amgr.add(ma);
-		amgr.add(n);
-		amgr.add(na);
-
-		amgr.run();
-
-		// a, b, c, d must be found. Order not specified.
-		REQUIRE(action_is_found(a->name()) == true);
-		REQUIRE(action_is_found(b->name()) == true);
-		REQUIRE(action_is_found(c->name()) == true);
-		REQUIRE(action_is_found(d->name()) == true);
-
-		// aa, ab must be after a.
-		auto it = find_action(a->name());
-		REQUIRE(action_is_found(aa->name(), it) == true);
-		REQUIRE(action_is_found(ab->name(), it) == true);
-
-		// aaa, aab must be after aa
-		it = find_action(aa->name());
-		REQUIRE(action_is_found(aaa->name(), it) == true);
-		REQUIRE(action_is_found(aab->name(), it) == true);
-
-		// aba must be after ab
-		it = find_action(ab->name());
-		REQUIRE(action_is_found(aba->name(), it) == true);
-
-		// The run actions must be the last four
-		std::vector<std::string>::iterator last_four = test_action::s_actions_run.end() - 4;
-		REQUIRE(action_is_found(m->name(), last_four) == true);
-		REQUIRE(action_is_found(ma->name(), last_four) == true);
-		REQUIRE(action_is_found(n->name(), last_four) == true);
-		REQUIRE(action_is_found(na->name(), last_four) == true);
-
-		// ma must be after m
-		it = find_action(m->name());
-		REQUIRE(action_is_found(ma->name(), it) == true);
-
-		// na must be after n
-		it = find_action(n->name());
-		REQUIRE(action_is_found(na->name(), it) == true);
-	}
-}

userspace/engine/CMakeLists.txt

@@ -56,3 +56,6 @@ else()
 endif()
 
 target_link_libraries(falco_engine "${FALCO_SINSP_LIBRARY}" "${LPEG_LIB}" "${LYAML_LIB}" "${LIBYAML_LIB}" luafiles)
+
+configure_file(config_falco_engine.h.in config_falco_engine.h)
+

userspace/engine/config_falco_engine.h.in

@@ -1,5 +1,5 @@
 /*
-Copyright (C) 2022 The Falco Authors.
+Copyright (C) 2019 The Falco Authors.
 
 Licensed under the Apache License, Version 2.0 (the "License");
 you may not use this file except in compliance with the License.
@@ -14,22 +14,7 @@ See the License for the specific language governing permissions and
 limitations under the License.
 */
 
-#include "app_runnable_action.h"
+#pragma once
 
-namespace falco {
-namespace app {
-
-runnable_action::runnable_action()
-{
-}
-
-runnable_action::~runnable_action()
-{
-}
-
-void runnable_action::deinit()
-{
-}
-
-}; // namespace application
-}; // namespace falco
+#define FALCO_ENGINE_LUA_DIR "${FALCO_ABSOLUTE_SHARE_DIR}/lua/"
+#define FALCO_ENGINE_SOURCE_LUA_DIR "${PROJECT_SOURCE_DIR}/userspace/engine/lua/"

userspace/engine/falco_common.cpp

@@ -16,6 +16,7 @@ limitations under the License.
 
 #include <fstream>
 
+#include "config_falco_engine.h"
 #include "falco_common.h"
 #include "banned.h" // This raises a compilation error when certain functions are used
 #include "falco_engine_lua_files.hh"

userspace/engine/falco_engine.cpp

@@ -25,6 +25,7 @@ limitations under the License.
 #include "falco_engine.h"
 #include "falco_utils.h"
 #include "falco_engine_version.h"
+#include "config_falco_engine.h"
 
 #include "formats.h"
 
@@ -43,7 +44,7 @@ const std::string falco_engine::s_default_ruleset = "falco-default-ruleset";
 
 using namespace std;
 
-falco_engine::falco_engine(bool seed_rng)
+falco_engine::falco_engine(bool seed_rng, const std::string& alternate_lua_dir)
 	: m_next_ruleset_id(0),
 	  m_min_priority(falco_common::PRIORITY_DEBUG),
 	  m_sampling_ratio(1), m_sampling_multiplier(0),
@@ -81,7 +82,7 @@ static std::string fieldclass_key(const gen_event_filter_factory::filter_fieldcl
 	return fld_info.name + fld_info.shortdesc;
 }
 
-void falco_engine::list_fields(std::string &source, bool verbose, bool names_only, bool markdown)
+void falco_engine::list_fields(std::string &source, bool verbose, bool names_only)
 {
 	// Maps from field class name + short desc to list of event
 	// sources for which this field class can be used.
@@ -126,29 +127,18 @@ void falco_engine::list_fields(std::string &source, bool verbose, bool names_onl
 
 			seen_fieldclasses.insert(key);
 
-			if(names_only)
-			{
-				for(auto &field : fld_class.fields)
-				{
-					// Skip fields with the EPF_TABLE_ONLY flag.
-					if(field.tags.find("EPF_TABLE_ONLY") != field.tags.end())
-					{
-						continue;
-					}
-
-					printf("%s\n", field.name.c_str());
-				}
-			}
-			else if (markdown)
-			{
-				printf("%s\n", fld_class.as_markdown(
-									fieldclass_event_sources[fieldclass_key(fld_class)]).c_str());
-			}
-			else
+			if(!names_only)
 			{
 				printf("%s\n", fld_class.as_string(verbose,
 								   fieldclass_event_sources[fieldclass_key(fld_class)]).c_str());
 			}
+			else
+			{
+				for(auto &field : fld_class.fields)
+				{
+					printf("%s\n", field.name.c_str());
+				}
+			}
 		}
 	}
 }
@@ -392,7 +382,7 @@ void falco_engine::describe_rule(string *rule)
 	return m_rules->describe_rule(rule);
 }
 
-// Print statistics on the rules that triggered
+// Print statistics on the the rules that triggered
 void falco_engine::print_stats()
 {
 	lua_getglobal(m_ls, lua_print_stats.c_str());
@@ -425,7 +415,7 @@ void falco_engine::add_filter(std::shared_ptr<gen_event_filter> filter,
 		throw falco_exception(err);
 	}
 
-	it->second->add(source, rule, tags, filter);
+	it->second->add(rule, tags, filter);
 }
 
 bool falco_engine::is_source_valid(const std::string &source)
@@ -437,7 +427,7 @@ bool falco_engine::is_plugin_compatible(const std::string &name,
 					const std::string &version,
 					std::string &required_version)
 {
-	sinsp_plugin::version plugin_version(version);
+	sinsp_plugin::version plugin_version(version.c_str());
 
 	if(!plugin_version.m_valid)
 	{
@@ -452,12 +442,13 @@ bool falco_engine::is_plugin_compatible(const std::string &name,
 
 	for(auto &rversion : m_required_plugin_versions[name])
 	{
-		sinsp_plugin::version req_version(rversion);
-		if (!plugin_version.check(req_version))
+		sinsp_plugin::version req_version(rversion.c_str());
+		if(req_version.m_version_major > plugin_version.m_version_major)
 		{
 			required_version = rversion;
 			return false;
 		}
+
 	}
 
 	return true;

userspace/engine/falco_engine.h

@@ -32,6 +32,7 @@ limitations under the License.
 #include "rules.h"
 #include "ruleset.h"
 
+#include "config_falco_engine.h"
 #include "falco_common.h"
 
 //
@@ -43,7 +44,7 @@ limitations under the License.
 class falco_engine : public falco_common
 {
 public:
-	falco_engine(bool seed_rng=true);
+	falco_engine(bool seed_rng=true, const std::string& alternate_lua_dir=FALCO_ENGINE_SOURCE_LUA_DIR);
 	virtual ~falco_engine();
 
 	// A given engine has a version which identifies the fields
@@ -54,7 +55,7 @@ public:
 
 	// Print to stdout (using printf) a description of each field supported by this engine.
 	// If source is non-empty, only fields for the provided source are printed.
-	void list_fields(std::string &source, bool verbose, bool names_only, bool markdown);
+	void list_fields(std::string &source, bool verbose, bool names_only);
 
 	//
 	// Load rules either directly or from a filename.

userspace/engine/falco_engine_version.h

@@ -22,3 +22,4 @@ limitations under the License.
 // represents the fields supported by this version of Falco. It's used
 // at build time to detect a changed set of fields.
 #define FALCO_FIELDS_CHECKSUM "4de812495f8529ac20bda2b9774462b15911a51df293d59fe9ccb6b922fdeb9d"
+

userspace/engine/formats.cpp

@@ -20,7 +20,7 @@ limitations under the License.
 #include "falco_engine.h"
 #include "banned.h" // This raises a compilation error when certain functions are used
 
-falco_formats::falco_formats(std::shared_ptr<falco_engine> engine,
+falco_formats::falco_formats(falco_engine *engine,
 			     bool json_include_output_property,
 			     bool json_include_tags_property)
 	: m_falco_engine(engine),

userspace/engine/formats.h

@@ -33,7 +33,7 @@ extern "C"
 class falco_formats
 {
 public:
-	falco_formats(std::shared_ptr<falco_engine> engine,
+	falco_formats(falco_engine *engine,
 		      bool json_include_output_property,
 		      bool json_include_tags_property);
 	virtual ~falco_formats();
@@ -45,7 +45,7 @@ public:
 					     const std::string &format);
 
 protected:
-	std::shared_ptr<falco_engine> m_falco_engine;
+	falco_engine *m_falco_engine;
 	bool m_json_include_output_property;
 	bool m_json_include_tags_property;
 };

userspace/engine/json_evt.cpp

@@ -1529,9 +1529,12 @@ void json_event_formatter::set_format(output_format of, const std::string &forma
 bool json_event_formatter::tostring_withformat(gen_event *gevt, std::string &output, gen_event_formatter::output_format of)
 {
 	json_event *ev = static_cast<json_event *>(gevt);
+
+	std::string ret;
+
 	if(of == OF_JSON)
 	{
-		output = tojson(ev);
+		ret = tojson(ev);
 		return true;
 	}
 	else

userspace/engine/json_evt.h

@@ -278,7 +278,7 @@ private:
 	// events. See alias struct for usage.
 	std::list<nlohmann::json::json_pointer> m_jptrs;
 
-	// The extraction function to use. May not be defined, in which
+	// Theextraction function to use. May not be defined, in which
 	// case the default function is used.
 	extract_t m_extract;
 

userspace/engine/lua/CMakeLists.txt

@@ -10,7 +10,7 @@
 # "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the License for the
 # specific language governing permissions and limitations under the License.
 
-file(GLOB_RECURSE lua_files ${CMAKE_CURRENT_SOURCE_DIR} *.lua)
+file(GLOB_RECURSE lua_module_files ${CMAKE_CURRENT_SOURCE_DIR} *.lua)
 
 add_custom_command(OUTPUT ${CMAKE_CURRENT_BINARY_DIR}/falco_engine_lua_files.cpp
 	COMMAND bash ${CMAKE_CURRENT_SOURCE_DIR}/lua-to-cpp.sh ${CMAKE_CURRENT_SOURCE_DIR} ${LYAML_LUA_DIR} ${CMAKE_CURRENT_BINARY_DIR}

userspace/engine/lua/modules/compiler.lua

@@ -40,8 +40,8 @@ end
    The AST is changed in-place.
 
    The return value is a boolean which is true if any macro was
-   substituted. This allows a caller to re-traverse until no more macros are
-   found, a simple strategy for recursive resolutions (e.g. when a macro
+   substitued. This allows a caller to re-traverse until no more macros are
+   found, a simple strategy for recursive resoltuions (e.g. when a macro
    definition uses another macro).
 
 --]]
@@ -113,6 +113,31 @@ function expand_macros(ast, defs, changed)
    return true, changed
 end
 
+function get_macros(ast, set)
+   if (ast.type == "Macro") then
+      set[ast.value] = true
+      return set
+   end
+
+   if ast.type == "Filter" then
+      return get_macros(ast.value, set)
+   end
+
+   if ast.type == "BinaryBoolOp" then
+      local left = get_macros(ast.left, {})
+      local right = get_macros(ast.right, {})
+
+      for m, _ in pairs(left) do set[m] = true end
+      for m, _ in pairs(right) do set[m] = true end
+
+      return set
+   end
+   if ast.type == "UnaryBoolOp" then
+      return get_macros(ast.argument, set)
+   end
+   return set
+end
+
 function get_filters(ast)
 
    local filters = {}

userspace/engine/lua/modules/parser.lua

@@ -57,7 +57,7 @@ local function geterrorinfo()
       end
 end
 
--- creates an error message using the farthest failure position
+-- creates an errror message using the farthest failure position
 local function errormsg()
    return geterrorinfo() / function(t)
          local p = t.ffp or 1

userspace/engine/lua/rule_loader.lua

@@ -59,10 +59,10 @@ end
 
 -- Permissive for case and for common abbreviations.
 priorities = {
-   Emergency=0, Alert=1, Critical=2, Error=3, Warning=4, Notice=5, Informational=6, Debug=7,
-   emergency=0, alert=1, critical=2, error=3, warning=4, notice=5, informational=6, debug=7,
-   EMERGENCY=0, ALERT=1, CRITICAL=2, ERROR=3, WARNING=4, NOTICE=5, INFORMATIONAL=6, DEBUG=7,
-   INFO=6, info=6
+   Emergency=0, Alert=1, Critical=2, Error=3, Warning=4, Notice=5, Informational=5, Debug=7,
+   emergency=0, alert=1, critical=2, error=3, warning=4, notice=5, informational=5, debug=7,
+   EMERGENCY=0, ALERT=1, CRITICAL=2, ERROR=3, WARNING=4, NOTICE=5, INFORMATIONAL=5, DEBUG=7,
+   INFO=5, info=5
 }
 
 --[[
@@ -157,6 +157,17 @@ local function create_filter_obj(node, lua_parser, parent_bool_op)
    return nil
 end
 
+function set_output(output_format, state)
+
+   if(output_ast.type == "OutputFormat") then
+
+      local format
+
+   else
+      error ("Unexpected type in set_output: ".. output_ast.type)
+   end
+end
+
 -- This should be keep in sync with parser.lua
 defined_comp_operators = {
    ["="]=1,
@@ -186,7 +197,7 @@ defined_list_comp_operators = {
 -- object. The by_name index is used for things like describing rules,
 -- and the by_idx index is used to map the relational node index back
 -- to a rule.
-local state = {macros={}, lists={}, rules_by_name={},
+local state = {macros={}, lists={}, filter_ast=nil, rules_by_name={},
 	       skipped_rules_by_name={}, macros_by_name={}, lists_by_name={},
 	       n_rules=0, rules_by_idx={}, ordered_rule_names={}, ordered_macro_names={}, ordered_list_names={}}
 
@@ -282,20 +293,19 @@ function split_lines(rules_content)
 end
 
 function get_orig_yaml_obj(rules_lines, row)
-    idx = row
-    local t = {}
-    while (idx <= #rules_lines) do
-        t[#t + 1] = rules_lines[idx]
-        idx = idx + 1
+   local ret = ""
 
-        if idx > #rules_lines or rules_lines[idx] == "" or string.sub(rules_lines[idx], 1, 1) == '-' then
-            break
-        end
-    end
-    t[#t + 1] = ""
-    local ret = ""
-    ret = table.concat(t, "\n")
-    return ret
+   idx = row
+   while (idx <= #rules_lines) do
+      ret = ret..rules_lines[idx].."\n"
+      idx = idx + 1
+
+      if idx > #rules_lines or rules_lines[idx] == "" or string.sub(rules_lines[idx], 1, 1) == '-' then
+	 break
+      end
+   end
+
+   return ret
 end
 
 function get_lines(rules_lines, row, num_lines)
@@ -460,14 +470,6 @@ function load_rules_doc(rules_mgr, doc, load_state)
 	    v['source'] = "syscall"
 	 end
 
-	 valid = falco_rules.is_source_valid(rules_mgr, v['source'])
-
-	 if valid == false then
-	    msg = "Macro "..v['macro']..": warning (unknown-source): unknown source "..v['source']..", skipping"
-	    warnings[#warnings + 1] = msg
-	    goto next_object
-	 end
-
 	 if state.macros_by_name[v['macro']] == nil then
 	    state.ordered_macro_names[#state.ordered_macro_names+1] = v['macro']
 	 end
@@ -550,14 +552,6 @@ function load_rules_doc(rules_mgr, doc, load_state)
 	    v['source'] = "syscall"
 	 end
 
-	 valid = falco_rules.is_source_valid(rules_mgr, v['source'])
-
-	 if valid == false then
-	    msg = "Rule "..v['rule']..": warning (unknown-source): unknown source "..v['source']..", skipping"
-	    warnings[#warnings + 1] = msg
-	    goto next_object
-	 end
-
 	 -- Add an empty exceptions property to the rule if not defined
 	 if v['exceptions'] == nil then
 	    v['exceptions'] = {}
@@ -625,7 +619,7 @@ function load_rules_doc(rules_mgr, doc, load_state)
 			return false, build_error_with_context(v['context'], "Rule exception item must have name property"), warnings
 		     end
 
-		     -- Separate case when a exception name is not found
+		     -- Seperate case when a exception name is not found
 		     -- This means that a new exception is being appended
 
 		     local new_exception = true
@@ -751,8 +745,6 @@ function load_rules_doc(rules_mgr, doc, load_state)
 	 arr = build_error_with_context(context, "Unknown top level object: "..table.tostring(v))
 	 warnings[#warnings + 1] = arr[1]
       end
-
-      ::next_object::
    end
 
    return true, {}, warnings
@@ -762,69 +754,65 @@ end
 -- Populates exfields with all fields used
 function build_exception_condition_string_multi_fields(eitem, exfields)
 
-    local fields = eitem['fields']
-    local comps = eitem['comps']
+   local fields = eitem['fields']
+   local comps = eitem['comps']
 
-    local icond = {}
+   local icond = "("
 
-    icond[#icond + 1] = "("
+   for i, values in ipairs(eitem['values']) do
 
-    local lcount = 0
-    for i, values in ipairs(eitem['values']) do
-        if #fields ~= #values then
-            return nil, "Exception item " .. eitem['name'] .. ": fields and values lists must have equal length"
-        end
+      if #fields ~= #values then
+	 return nil, "Exception item "..eitem['name']..": fields and values lists must have equal length"
+      end
 
-        if lcount ~= 0 then
-            icond[#icond + 1] = " or "
-        end
-        lcount = lcount + 1
+      if icond ~= "(" then
+	 icond=icond.." or "
+      end
 
-        icond[#icond + 1] = "("
+      icond=icond.."("
 
-        for k = 1, #fields do
-            if k > 1 then
-                icond[#icond + 1] = " and "
-            end
-            local ival = values[k]
-            local istr = ""
+      for k=1,#fields do
+	 if k > 1 then
+	    icond=icond.." and "
+	 end
+	 local ival = values[k]
+	 local istr = ""
 
-            -- If ival is a table, express it as (titem1, titem2, etc)
-            if type(ival) == "table" then
-                istr = "("
-                for _, item in ipairs(ival) do
-                    if istr ~= "(" then
-                        istr = istr .. ", "
-                    end
-                    istr = istr .. quote_item(item)
-                end
-                istr = istr .. ")"
-            else
-                -- If the corresponding operator is one that works on lists, possibly add surrounding parentheses.
-                if defined_list_comp_operators[comps[k]] then
-                    istr = paren_item(ival)
-                else
-                    -- Quote the value if not already quoted
-                    istr = quote_item(ival)
-                end
-            end
+	 -- If ival is a table, express it as (titem1, titem2, etc)
+	 if type(ival) == "table" then
+	    istr = "("
+	    for _, item in ipairs(ival) do
+	       if istr ~= "(" then
+		  istr = istr..", "
+	       end
+	       istr = istr..quote_item(item)
+	    end
+	    istr = istr..")"
+	 else
+	    -- If the corresponding operator is one that works on lists, possibly add surrounding parentheses.
+	    if defined_list_comp_operators[comps[k]] then
+	       istr = paren_item(ival)
+	    else
+	       -- Quote the value if not already quoted
+	       istr = quote_item(ival)
+	    end
+	 end
 
-            icond[#icond + 1] = fields[k] .. " " .. comps[k] .. " " .. istr
-            exfields[fields[k]] = true
-        end
+	 icond = icond..fields[k].." "..comps[k].." "..istr
+	 exfields[fields[k]] = true
+      end
 
-        icond[#icond + 1] = ")"
-    end
+      icond=icond..")"
+   end
 
-    icond[#icond + 1] = ")"
+   icond = icond..")"
 
-    -- Don't return a trivially empty condition string
-    local ret = table.concat(icond)
-    if ret == "()" then
-        return "", nil
-    end
+   -- Don't return a trivially empty condition string
+   if icond == "()" then
+      icond = ""
+   end
 
-    return ret, nil
+   return icond, nil
 
 end
 
@@ -1026,6 +1014,14 @@ function load_rules(rules_content,
 
       if (filter_ast.type == "Rule") then
 
+	 valid = falco_rules.is_source_valid(rules_mgr, v['source'])
+
+	 if valid == false then
+	    msg = "Rule "..v['rule']..": warning (unknown-source): unknown source "..v['source']..", skipping"
+	    warnings[#warnings + 1] = msg
+	    goto next_rule
+	 end
+
 	 state.n_rules = state.n_rules + 1
 
 	 state.rules_by_idx[state.n_rules] = v
@@ -1058,14 +1054,22 @@ function load_rules(rules_content,
 
 	 else
 	    num_evttypes = falco_rules.add_filter(rules_mgr, lua_parser, v['rule'], v['source'], v['tags'])
-	    if v['source'] == "syscall" and (num_evttypes == 0 or num_evttypes > 100) then
+	    if num_evttypes == 0 or num_evttypes > 100 then
 	       if warn_evttypes == true then
-		  msg = "Rule "..v['rule']..": warning (no-evttype):\n".."         matches too many evt.type values.\n".."         This has a significant performance penalty."
+		  msg = "Rule "..v['rule']..": warning (no-evttype):"
 		  warnings[#warnings + 1] = msg
 	       end
 	    end
 	 end
 
+	 -- Rule ASTs are merged together into one big AST, with "OR" between each
+	 -- rule.
+	 if (state.filter_ast == nil) then
+	    state.filter_ast = filter_ast.filter.value
+	 else
+	    state.filter_ast = { type = "BinaryBoolOp", operator = "or", left = state.filter_ast, right = filter_ast.filter.value }
+	 end
+
 	 -- Enable/disable the rule
 	 if (v['enabled'] == nil) then
 	    v['enabled'] = true

userspace/engine/rules.cpp

@@ -145,19 +145,11 @@ int falco_rules::add_filter(lua_State *ls)
 		lua_pop(ls, 1);
 	}
 
-	// todo(jasondellaluce,leogr,fededp): temp workaround, remove when fixed in libs
-	size_t num_evttypes = 1; // assume plugin
-	if(source == "syscall" || source == "k8s_audit")
-	{
-		num_evttypes = lp->filter()->evttypes().size();
-	}
+	size_t num_evttypes = lp->filter()->evttypes().size();
 
-	try
-	{
+	try {
 		rules->add_filter(lp->filter(), rule, source, tags);
-	}
-	catch (exception &e)
-	{
+	} catch (exception &e) {
 		std::string errstr = string("Could not add rule to falco engine: ") + e.what();
 		lua_pushstring(ls, errstr.c_str());
 		lua_error(ls);

userspace/engine/ruleset.cpp

@@ -66,7 +66,7 @@ void falco_ruleset::ruleset_filters::remove_wrapper_from_list(filter_wrapper_lis
 
 void falco_ruleset::ruleset_filters::add_filter(std::shared_ptr<filter_wrapper> wrap)
 {
-	std::set<uint16_t> fevttypes = wrap->evttypes();
+	std::set<uint16_t> fevttypes = wrap->filter->evttypes();
 
 	if(fevttypes.empty())
 	{
@@ -91,7 +91,7 @@ void falco_ruleset::ruleset_filters::add_filter(std::shared_ptr<filter_wrapper>
 
 void falco_ruleset::ruleset_filters::remove_filter(std::shared_ptr<filter_wrapper> wrap)
 {
-	std::set<uint16_t> fevttypes = wrap->evttypes();
+	std::set<uint16_t> fevttypes = wrap->filter->evttypes();
 
 	if(fevttypes.empty())
 	{
@@ -118,16 +118,18 @@ uint64_t falco_ruleset::ruleset_filters::num_filters()
 
 bool falco_ruleset::ruleset_filters::run(gen_event *evt)
 {
-    if(evt->get_type() < m_filter_by_event_type.size())
-    {
-        for(auto &wrap : m_filter_by_event_type[evt->get_type()])
-        {
-            if(wrap->filter->run(evt))
-            {
-                return true;
-            }
-        }
-    }
+	if(evt->get_type() >= m_filter_by_event_type.size())
+	{
+		return false;
+	}
+
+	for(auto &wrap : m_filter_by_event_type[evt->get_type()])
+	{
+		if(wrap->filter->run(evt))
+		{
+			return true;
+		}
+	}
 
 	// Finally, try filters that are not specific to an event type.
 	for(auto &wrap : m_filter_all_event_types)
@@ -147,18 +149,16 @@ void falco_ruleset::ruleset_filters::evttypes_for_ruleset(std::set<uint16_t> &ev
 
 	for(auto &wrap : m_filters)
 	{
-		auto fevttypes = wrap->evttypes();
+		auto fevttypes = wrap->filter->evttypes();
 		evttypes.insert(fevttypes.begin(), fevttypes.end());
 	}
 }
 
-void falco_ruleset::add(string &source,
-			string &name,
+void falco_ruleset::add(string &name,
 			set<string> &tags,
 			std::shared_ptr<gen_event_filter> filter)
 {
 	std::shared_ptr<filter_wrapper> wrap(new filter_wrapper());
-	wrap->source = source;
 	wrap->name = name;
 	wrap->tags = tags;
 	wrap->filter = filter;

userspace/engine/ruleset.h

@@ -34,8 +34,7 @@ public:
 	falco_ruleset();
 	virtual ~falco_ruleset();
 
-	void add(string &source,
-		 std::string &name,
+	void add(std::string &name,
 		 std::set<std::string> &tags,
 		 std::shared_ptr<gen_event_filter> filter);
 
@@ -74,21 +73,9 @@ private:
 
 	class filter_wrapper {
 	public:
-		std::string source;
 		std::string name;
 		std::set<std::string> tags;
 		std::shared_ptr<gen_event_filter> filter;
-		std::set<uint16_t> evttypes()
-		{
-			// todo(jasondellaluce,leogr): temp workaround, remove when fixed in libs
-			if(source == "syscall" || source == "k8s_audit")
-			{
-				return filter->evttypes();
-			}
-			// else assume plugins
-			return {ppm_event_type::PPME_PLUGINEVENT_E};
-			// workaround end
-		}
 	};
 
 	typedef std::list<std::shared_ptr<filter_wrapper>> filter_wrapper_list;

userspace/falco/CMakeLists.txt

@@ -15,33 +15,6 @@ configure_file(config_falco.h.in config_falco.h)
 
 set(
   FALCO_SOURCES
-  application.cpp
-  app_cmdline_options.cpp
-  app_action.cpp
-  app_runnable_action.cpp
-  app_actions/create_signal_handlers.cpp
-  app_actions/easyopts_action.cpp
-  app_actions/init_action.cpp
-  app_actions/run_action.cpp
-  app_actions/init_falco_engine.cpp
-  app_actions/init_inspector.cpp
-  app_actions/init_outputs.cpp
-  app_actions/list_fields.cpp
-  app_actions/list_plugins.cpp
-  app_actions/load_config.cpp
-  app_actions/load_plugins.cpp
-  app_actions/load_rules_files.cpp
-  app_actions/print_help.cpp
-  app_actions/print_ignored_events.cpp
-  app_actions/print_support.cpp
-  app_actions/print_version.cpp
-  app_actions/start_grpc_server.cpp
-  app_actions/start_webserver.cpp
-  app_actions/validate_rules_files.cpp
-  app_actions/daemonize.cpp
-  app_actions/open_inspector.cpp
-  app_actions/process_events.cpp
-  app_action_manager.cpp
   configuration.cpp
   logger.cpp
   falco_outputs.cpp
@@ -60,7 +33,6 @@ set(
   "${PROJECT_BINARY_DIR}/userspace/falco"
   "${PROJECT_BINARY_DIR}/driver/src"
   "${STRING_VIEW_LITE_INCLUDE}"
-  "${CXXOPTS_INCLUDE_DIR}"
   "${YAMLCPP_INCLUDE_DIR}"
   "${CMAKE_CURRENT_BINARY_DIR}"
   "${DRAIOS_DEPENDENCIES_DIR}/yaml-${DRAIOS_YAML_VERSION}/target/include"
@@ -74,7 +46,6 @@ set(
   luajit
   lpeg
   lyaml
-  cxxopts
 )
 
 set(
@@ -156,7 +127,6 @@ target_include_directories(
   falco
   PUBLIC
   ${FALCO_INCLUDE_DIRECTORIES}
-  ${CMAKE_CURRENT_SOURCE_DIR}
 )
 
 if(NOT MINIMAL_BUILD)

userspace/falco/app_action.cpp

@@ -1,48 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "application.h"
-#include "app_action.h"
-
-namespace falco {
-namespace app {
-
-action::action(application &app)
-	: m_app(app)
-{
-}
-
-action::~action()
-{
-}
-
-application &action::app()
-{
-	return m_app;
-}
-
-cmdline_options &action::options()
-{
-	return m_app.options();
-}
-
-application::action_state &action::state()
-{
-	return m_app.state();
-}
-
-}; // namespace application
-}; // namespace falco

userspace/falco/app_action.h

@@ -1,59 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#pragma once
-
-#include "application.h"
-#include "app_cmdline_options.h"
-#include "app_runnable_action.h"
-
-namespace falco {
-namespace app {
-
-// This class represents an "action" e.g. a chunk of code to execute
-// as a part of running the falco application. Examples of actions are:
-//   - initializing/configuring the inspector
-//   - loading/configuring plugins
-//   - reading events from a trace file or live event source
-//
-// Actions also include "one off" actions for things like --help
-// output, --list fields, etc.
-//
-// There's no attempt to distribute state (e.g. inspectors, lists of
-// plugins, etc) across actions or have ways to advertise what state
-// an action manages. The expectation is that all state that needs to
-// be used across actions is held in the provided application object
-// and actions know which state they should create and destroy.
-
-// The reason for a sublcass is to allow for building/running unit
-// tests for the action manager without bringing in all of the falco
-// application code (engine, outputs, grpc, etc).
-class action : public runnable_action {
-public:
-	action(application &app);
-	virtual ~action();
-
-	application &app();
-
-	cmdline_options &options();
-	application::action_state &state();
-
-private:
-	application &m_app;
-};
-
-}; // namespace application
-}; // namespace falco

userspace/falco/app_action_manager.cpp

@@ -1,163 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "app_action_manager.h"
-#include "logger.h"
-#include "falco_common.h"
-
-#include <algorithm>
-#include <list>
-#include <memory>
-#include <string>
-#include <utility>
-#include <vector>
-
-namespace falco {
-namespace app {
-
-action_manager::action_manager()
-{
-}
-
-action_manager::~action_manager()
-{
-}
-
-void action_manager::set_groups(std::list<std::string> &groups)
-{
-	m_groups = groups;
-}
-
-void action_manager::add(std::shared_ptr<runnable_action> act)
-{
-	m_actions[act->name()] = act;
-}
-
-runnable_action::run_result action_manager::run()
-{
-	runnable_action::run_result res;
-
-	sort_groups();
-	res = run_groups();
-	deinit_groups();
-
-	return res;
-}
-
-void action_manager::sort_groups()
-{
-	for(auto &group : m_groups)
-	{
-		std::vector<std::shared_ptr<runnable_action>> actions_ordered;
-
-		for(auto &pair : m_actions)
-		{
-			if(pair.second->group() == group)
-			{
-				actions_ordered.push_back(pair.second);
-			}
-		}
-
-		auto compare = [this](const std::shared_ptr<runnable_action> &a,
-				      const std::shared_ptr<runnable_action> &b) {
-			return this->compare_actions(a, b);
-		};
-
-		// Order the actions according to precedence
-		std::sort(actions_ordered.begin(), actions_ordered.end(), compare);
-
-		m_actions_ordered[group] = std::move(actions_ordered);
-	}
-}
-
-runnable_action::run_result action_manager::run_groups()
-{
-	runnable_action::run_result res;
-
-	for(auto &group : m_groups)
-	{
-		falco_logger::log(LOG_DEBUG, string("Running group ") + group);
-
-		for(auto &act : m_actions_ordered[group])
-		{
-			falco_logger::log(LOG_DEBUG, string("Running action ") + act->name());
-
-			res = act->run();
-
-			if(!res.success)
-			{
-				falco_logger::log(LOG_DEBUG, string("Could not complete ") + act->name() + " : " + res.errstr);
-			}
-
-			if(!res.proceed)
-			{
-				break;
-			}
-		}
-
-		if(!res.proceed)
-		{
-			break;
-		}
-	}
-
-	return res;
-}
-
-void action_manager::deinit_groups()
-{
-	for(auto &group : m_groups)
-	{
-		for(auto &act : m_actions_ordered[group])
-		{
-			falco_logger::log(LOG_DEBUG, string("Deinitializing action ") + act->name());
-
-			act->deinit();
-		}
-	}
-}
-
-bool action_manager::compare_actions(const std::shared_ptr<runnable_action> &a, const std::shared_ptr<runnable_action> &b)
-{
-	// Check b's prerequsites. If a is found return true.
-	for(auto &prereq_name : b->prerequsites())
-	{
-		if(prereq_name == a->name())
-		{
-			return true;
-		}
-	}
-
-	// Not a direct dependency. Check b's prerequsites recursively
-	for(auto &prereq_name : b->prerequsites())
-	{
-		auto it = m_actions.find(prereq_name);
-		if(it == m_actions.end())
-		{
-			throw falco_exception("No action with name " + prereq_name + " exists?");
-		}
-
-		if(compare_actions(a, it->second))
-		{
-			return true;
-		}
-	}
-
-	return false;
-}
-
-}; // namespace application
-}; // namespace falco

userspace/falco/app_action_manager.h

@@ -1,67 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#pragma once
-
-#include "app_runnable_action.h"
-
-#include <list>
-#include <memory>
-#include <map>
-#include <string>
-#include <vector>
-
-namespace falco {
-namespace app {
-
-// This class manages a set of actions, ensuring that they run in an
-// order that honors their dependencies, groups and their run results.
-
-class action_manager {
-public:
-	action_manager();
-	virtual ~action_manager();
-
-	// Actions are organized into groups. All actions from a
-	// given group are run before actions from another group.
-	//
-	// Example groups are "init", "run", etc.
-	//
-	// This specifies the order of groups.
-	void set_groups(std::list<std::string> &groups);
-
-	void add(std::shared_ptr<runnable_action> act);
-
-	runnable_action::run_result run();
-
-private:
-
-	typedef std::vector<std::shared_ptr<runnable_action>> ordered_actions_t;
-
-	void sort_groups();
-	runnable_action::run_result run_groups();
-	void deinit_groups();
-
-	// Return true if a is less (e.g. a should run before b)
-	bool compare_actions(const std::shared_ptr<runnable_action> &a, const std::shared_ptr<runnable_action> &b);
-
-	std::list<std::string> m_groups;
-	std::map<std::string, std::shared_ptr<runnable_action>> m_actions;
-	std::map<std::string, ordered_actions_t> m_actions_ordered;
-};
-
-}; // namespace application
-}; // namespace falco

userspace/falco/app_actions/create_signal_handlers.cpp

@@ -1,99 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include <string.h>
-#include <signal.h>
-
-#include "create_signal_handlers.h"
-
-static void signal_callback(int signal)
-{
-	falco::app::application::get().state().terminate = true;
-}
-
-static void reopen_outputs(int signal)
-{
-	falco::app::application::get().state().reopen_outputs = true;
-}
-
-static void restart_falco(int signal)
-{
-	falco::app::application::get().state().restart = true;
-}
-
-namespace falco {
-namespace app {
-
-act_create_signal_handlers::act_create_signal_handlers(application &app)
-	: init_action(app), m_name("create signal handlers")
-{
-}
-
-act_create_signal_handlers::~act_create_signal_handlers()
-{
-}
-
-const std::string &act_create_signal_handlers::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_create_signal_handlers::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_create_signal_handlers::run()
-{
-	run_result ret = {true, "", true};
-
-	if(! create_handler(SIGINT, signal_callback, ret) ||
-	   ! create_handler(SIGTERM, signal_callback, ret) ||
-	   ! create_handler(SIGUSR1, reopen_outputs, ret) ||
-	   ! create_handler(SIGHUP, restart_falco, ret))
-	{
-		return ret;
-	}
-
-	return ret;
-}
-
-bool act_create_signal_handlers::create_handler(int sig, void (*func)(int), run_result &ret)
-{
-	if(signal(sig, func) == SIG_ERR)
-	{
-		char errbuf[1024];
-
-		if (strerror_r(errno, errbuf, sizeof(errbuf)) != 0)
-		{
-			snprintf(errbuf, sizeof(errbuf)-1, "Errno %d", errno);
-		}
-
-		ret.success = false;
-		ret.errstr = std::string("Could not create signal handler for ") +
-			   strsignal(sig) +
-			   ": " +
-			   errbuf;
-
-		ret.proceed = false;
-	}
-
-	return ret.success;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/create_signal_handlers.h

@@ -1,47 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-class act_create_signal_handlers : public init_action {
-public:
-	act_create_signal_handlers(application &app);
-	virtual ~act_create_signal_handlers();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-
-	bool create_handler(int sig, void (*func)(int), run_result &ret);
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/daemonize.cpp

@@ -1,115 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <fcntl.h>
-
-#include "daemonize.h"
-
-namespace falco {
-namespace app {
-
-act_daemonize::act_daemonize(application &app)
-	: run_action(app), m_name("daemonize"), m_daemonized(false)
-{
-}
-
-act_daemonize::~act_daemonize()
-{
-}
-
-const std::string &act_daemonize::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_daemonize::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_daemonize::run()
-{
-	run_result ret = {true, "", true};
-
-	// If daemonizing, do it here so any init errors will
-	// be returned in the foreground process.
-	if (options().daemon && !m_daemonized) {
-		pid_t pid, sid;
-
-		pid = fork();
-		if (pid < 0) {
-			// error
-			ret.success = false;
-			ret.errstr = "Could not fork.";
-			ret.proceed = false;
-			return ret;
-		} else if (pid > 0) {
-			// parent. Write child pid to pidfile and exit
-			std::ofstream pidfile;
-			pidfile.open(options().pidfilename);
-
-			if (!pidfile.good())
-			{
-				ret.success = false;
-				ret.errstr = string("Could not write pid to pid file ") + options().pidfilename + ".";
-				ret.proceed = false;
-				return ret;
-			}
-			pidfile << pid;
-			pidfile.close();
-			return ret;
-		}
-		// if here, child.
-
-		// Become own process group.
-		sid = setsid();
-		if (sid < 0) {
-			ret.success = false;
-			ret.errstr = string("Could not set session id.");
-			ret.proceed = false;
-			return ret;
-		}
-
-		// Set umask so no files are world anything or group writable.
-		umask(027);
-
-		// Change working directory to '/'
-		if ((chdir("/")) < 0) {
-			ret.success = false;
-			ret.errstr = string("Could not change working directory to '/'.");
-			ret.proceed = false;
-			return ret;
-		}
-
-		// Close stdin, stdout, stderr and reopen to /dev/null
-		close(0);
-		close(1);
-		close(2);
-		open("/dev/null", O_RDONLY);
-		open("/dev/null", O_RDWR);
-		open("/dev/null", O_RDWR);
-
-		m_daemonized = true;
-	}
-
-	return ret;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/daemonize.h

@@ -1,46 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "run_action.h"
-
-namespace falco {
-namespace app {
-
-class act_daemonize : public run_action {
-public:
-	act_daemonize(application &app);
-	virtual ~act_daemonize();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-	bool m_daemonized;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/easyopts_action.cpp

@@ -1,40 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "easyopts_action.h"
-
-namespace falco {
-namespace app {
-
-static std::string easyopts_group = "easyopts";
-
-easyopts_action::easyopts_action(application &app)
-	: action(app)
-{
-}
-
-easyopts_action::~easyopts_action()
-{
-}
-
-const std::string &easyopts_action::group()
-{
-	return easyopts_group;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/easyopts_action.h

@@ -1,37 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include "app_action.h"
-
-namespace falco {
-namespace app {
-
-// All actions in the "easyopts" group derive from this class
-
-class easyopts_action : public action {
-public:
-	easyopts_action(application &app);
-	virtual ~easyopts_action();
-
-	const std::string &group() override;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/init_action.cpp

@@ -1,40 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-static std::string init_group = "init";
-
-init_action::init_action(application &app)
-	: action(app)
-{
-}
-
-init_action::~init_action()
-{
-}
-
-const std::string &init_action::group()
-{
-	return init_group;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/init_action.h

@@ -1,37 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include "app_action.h"
-
-namespace falco {
-namespace app {
-
-// All actions in the "init" group derive from this class
-
-class init_action : public action {
-public:
-	init_action(application &app);
-	virtual ~init_action();
-
-	const std::string &group() override;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/init_falco_engine.cpp

@@ -1,118 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "init_falco_engine.h"
-
-namespace falco {
-namespace app {
-
-act_init_falco_engine::act_init_falco_engine(application &app)
-	: init_action(app), m_name("init falco engine"),
-	  m_prerequsites({"init inspector", "load config"})
-{
-}
-
-act_init_falco_engine::~act_init_falco_engine()
-{
-}
-
-const std::string &act_init_falco_engine::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_init_falco_engine::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_init_falco_engine::run()
-{
-	run_result ret = {true, "", true};
-
-	configure_output_format();
-
-	// Create "factories" that can create filters/formatters for
-	// syscalls and k8s audit events.
-
-	// libs requires raw pointer, we should modify libs to use reference/shared_ptr
-	std::shared_ptr<gen_event_filter_factory> syscall_filter_factory(new sinsp_filter_factory(state().inspector.get()));
-	std::shared_ptr<gen_event_filter_factory> k8s_audit_filter_factory(new json_event_filter_factory());
-
-	// libs requires raw pointer, we should modify libs to use reference/shared_ptr
-	std::shared_ptr<gen_event_formatter_factory> syscall_formatter_factory(new sinsp_evt_formatter_factory(state().inspector.get()));
-	std::shared_ptr<gen_event_formatter_factory> k8s_audit_formatter_factory(new json_event_formatter_factory(k8s_audit_filter_factory));
-
-	state().engine->add_source(application::s_syscall_source, syscall_filter_factory, syscall_formatter_factory);
-	state().engine->add_source(application::s_k8s_audit_source, k8s_audit_filter_factory, k8s_audit_formatter_factory);
-
-	if(state().config->m_json_output)
-	{
-		syscall_formatter_factory->set_output_format(gen_event_formatter::OF_JSON);
-		k8s_audit_formatter_factory->set_output_format(gen_event_formatter::OF_JSON);
-	}
-
-	for(const auto &src : options().disable_sources)
-	{
-		state().enabled_sources.erase(src);
-	}
-
-	// XXX/mstemm technically this isn't right, you could disable syscall *and* k8s_audit and configure a plugin.
-	if(state().enabled_sources.empty())
-	{
-		throw std::invalid_argument("The event source \"syscall\" and \"k8s_audit\" can not be disabled together");
-	}
-
-	state().engine->set_min_priority(state().config->m_min_priority);
-
-	return ret;
-}
-
-void act_init_falco_engine::configure_output_format()
-{
-	std::string output_format;
-	bool replace_container_info = false;
-
-	if(options().print_additional == "c" || options().print_additional == "container")
-	{
-		output_format = "container=%container.name (id=%container.id)";
-		replace_container_info = true;
-	}
-	else if(options().print_additional == "k" || options().print_additional == "kubernetes")
-	{
-		output_format = "k8s.ns=%k8s.ns.name k8s.pod=%k8s.pod.name container=%container.id";
-		replace_container_info = true;
-	}
-	else if(options().print_additional == "m" || options().print_additional == "mesos")
-	{
-		output_format = "task=%mesos.task.name container=%container.id";
-		replace_container_info = true;
-	}
-	else if(!options().print_additional.empty())
-	{
-		output_format = options().print_additional;
-		replace_container_info = false;
-	}
-
-	if(!output_format.empty())
-	{
-		state().engine->set_extra(output_format, replace_container_info);
-	}
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/init_falco_engine.h

@@ -1,47 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-class act_init_falco_engine : public init_action {
-public:
-	act_init_falco_engine(application &app);
-	virtual ~act_init_falco_engine();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-	void configure_output_format();
-
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/init_inspector.cpp

@@ -1,138 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "init_inspector.h"
-
-namespace falco {
-namespace app {
-
-act_init_inspector::act_init_inspector(application &app)
-	: init_action(app), m_name("init inspector"),
-	  m_prerequsites({"load config"})
-{
-}
-
-act_init_inspector::~act_init_inspector()
-{
-}
-
-const std::string &act_init_inspector::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_init_inspector::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_init_inspector::run()
-{
-	run_result ret = {true, "", true};
-
-	state().inspector->set_buffer_format(options().event_buffer_format);
-
-	// If required, set the CRI paths
-	for (auto &p : options().cri_socket_paths)
-	{
-		if (!p.empty())
-		{
-			state().inspector->add_cri_socket_path(p);
-		}
-	}
-
-	// Decide whether to do sync or async for CRI metadata fetch
-	state().inspector->set_cri_async(!options().disable_cri_async);
-
-	//
-	// If required, set the snaplen
-	//
-	if(options().snaplen != 0)
-	{
-		state().inspector->set_snaplen(options().snaplen);
-	}
-
-	if(!options().all_events)
-	{
-		// Drop EF_DROP_SIMPLE_CONS kernel side
-		state().inspector->set_simple_consumer();
-		// Eventually, drop any EF_DROP_SIMPLE_CONS event
-		// that reached userspace (there are some events that are not syscall-based
-		// like signaldeliver, that have the EF_DROP_SIMPLE_CONS flag)
-		state().inspector->set_drop_event_flags(EF_DROP_SIMPLE_CONS);
-	}
-
-	state().inspector->set_hostname_and_port_resolution_mode(false);
-
-#ifndef MINIMAL_BUILD
-
-		falco_logger::log(LOG_DEBUG, "Setting metadata download max size to " + to_string(state().config->m_metadata_download_max_mb) + " MB\n");
-		falco_logger::log(LOG_DEBUG, "Setting metadata download chunk wait time to " + to_string(state().config->m_metadata_download_chunk_wait_us) + " μs\n");
-		falco_logger::log(LOG_DEBUG, "Setting metadata download watch frequency to " + to_string(state().config->m_metadata_download_watch_freq_sec) + " seconds\n");
-		state().inspector->set_metadata_download_params(state().config->m_metadata_download_max_mb * 1024 * 1024, state().config->m_metadata_download_chunk_wait_us, state().config->m_metadata_download_watch_freq_sec);
-
-#endif
-
-#ifndef MINIMAL_BUILD
-	// Initializing k8s/mesos might have to move to open inspector
-	//
-	// Run k8s, if required
-	//
-	char *k8s_api_env = NULL;
-	if(!options().k8s_api.empty() ||
-	   (k8s_api_env = getenv("FALCO_K8S_API")))
-	{
-		// Create string pointers for some config vars
-		// and pass to inspector. The inspector then
-		// owns the pointers.
-		std::string *k8s_api_ptr = new string((!options().k8s_api.empty() ? options().k8s_api : k8s_api_env));
-		std::string *k8s_api_cert_ptr = new string(options().k8s_api_cert);
-		std::string *k8s_node_name_ptr = new string(options().k8s_node_name);
-
-		if(k8s_api_cert_ptr->empty())
-		{
-			if(char* k8s_cert_env = getenv("FALCO_K8S_API_CERT"))
-			{
-				*k8s_api_cert_ptr = k8s_cert_env;
-			}
-		}
-		state().inspector->init_k8s_client(k8s_api_ptr, k8s_api_cert_ptr, k8s_node_name_ptr, options().verbose);
-	}
-
-	//
-	// Run mesos, if required
-	//
-	if(!options().mesos_api.empty())
-	{
-		// Differs from init_k8s_client in that it
-		// passes a pointer but the inspector does
-		// *not* own it and does not use it after
-		// init_mesos_client() returns.
-		state().inspector->init_mesos_client(&(options().mesos_api), options().verbose);
-	}
-	else if(char* mesos_api_env = getenv("FALCO_MESOS_API"))
-	{
-		std::string mesos_api_copy = mesos_api_env;
-		state().inspector->init_mesos_client(&mesos_api_copy, options().verbose);
-	}
-
-#endif
-	return ret;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/init_inspector.h

@@ -1,45 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-class act_init_inspector : public init_action {
-public:
-	act_init_inspector(application &app);
-	virtual ~act_init_inspector();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/init_outputs.cpp

@@ -1,88 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include <stdlib.h>
-#include <unistd.h>
-
-#include "init_outputs.h"
-
-namespace falco {
-namespace app {
-
-act_init_outputs::act_init_outputs(application &app)
-	: init_action(app), m_name("init outputs"),
-	  m_prerequsites({"load config", "init falco engine"})
-{
-}
-
-act_init_outputs::~act_init_outputs()
-{
-}
-
-const std::string &act_init_outputs::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_init_outputs::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_init_outputs::run()
-{
-	run_result ret = {true, "", true};
-
-	// read hostname
-	std::string hostname;
-	if(char* env_hostname = getenv("FALCO_GRPC_HOSTNAME"))
-	{
-		hostname = env_hostname;
-	}
-	else
-	{
-		char c_hostname[256];
-		int err = gethostname(c_hostname, 256);
-		if(err != 0)
-		{
-			ret.success = false;
-			ret.errstr = "Failed to get hostname";
-			ret.proceed = false;
-		}
-		hostname = c_hostname;
-	}
-
-	state().outputs->init(state().engine,
-				    state().config->m_json_output,
-				    state().config->m_json_include_output_property,
-				    state().config->m_json_include_tags_property,
-				    state().config->m_output_timeout,
-				    state().config->m_notifications_rate, state().config->m_notifications_max_burst,
-				    state().config->m_buffered_outputs,
-				    state().config->m_time_format_iso_8601,
-				    hostname);
-
-	for(auto output : state().config->m_outputs)
-	{
-		state().outputs->add_output(output);
-	}
-
-	return ret;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/init_outputs.h

@@ -1,45 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-class act_init_outputs : public init_action {
-public:
-	act_init_outputs(application &app);
-	virtual ~act_init_outputs();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/list_fields.cpp

@@ -1,74 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include <fields_info.h>
-
-#include "list_fields.h"
-
-namespace falco {
-namespace app {
-
-act_list_fields::act_list_fields(application &app)
-	: init_action(app), m_name("list fields"),
-	  m_prerequsites({"load plugins"})
-{
-}
-
-act_list_fields::~act_list_fields()
-{
-}
-
-const std::string &act_list_fields::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_list_fields::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_list_fields::run()
-{
-	run_result ret = {true, "", true};
-
-	if(options().list_fields)
-	{
-		if(options().list_source_fields != "" &&
-		   !state().engine->is_source_valid(options().list_source_fields))
-		{
-			ret.success = false;
-			ret.errstr = "Value for --list must be a valid source type";
-			ret.proceed = false;
-			return ret;
-		}
-		state().engine->list_fields(options().list_source_fields, options().verbose, options().names_only, options().markdown);
-
-		ret.proceed = false;
-	}
-	else if(app().options().list_syscall_events)
-	{
-		list_events(app().state().inspector.get(), app().options().markdown);
-
-		ret.proceed = false;
-	}
-
-	return ret;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/list_fields.h

@@ -1,45 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-class act_list_fields : public init_action {
-public:
-	act_list_fields(application &app);
-	virtual ~act_list_fields();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/list_plugins.cpp

@@ -1,78 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "list_plugins.h"
-
-namespace falco {
-namespace app {
-
-act_list_plugins::act_list_plugins(application &app)
-	: init_action(app), m_name("list plugins"),
-	  m_prerequsites({"load plugins"})
-{
-}
-
-act_list_plugins::~act_list_plugins()
-{
-}
-
-const std::string &act_list_plugins::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_list_plugins::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_list_plugins::run()
-{
-	run_result ret = {true, "", true};
-
-	if(options().list_plugins)
-	{
-		std::ostringstream os;
-
-		for(auto &info : state().plugin_infos)
-		{
-			os << "Name: " << info.name << std::endl;
-			os << "Description: " << info.description << std::endl;
-			os << "Contact: " << info.contact << std::endl;
-			os << "Version: " << info.plugin_version.as_string() << std::endl;
-
-			if(info.type == TYPE_SOURCE_PLUGIN)
-			{
-				os << "Type: source plugin" << std::endl;
-				os << "ID: " << info.id << std::endl;
-			}
-			else
-			{
-				os << "Type: extractor plugin" << std::endl;
-			}
-			os << std::endl;
-		}
-
-		printf("%lu Plugins Loaded:\n\n%s\n", state().plugin_infos.size(), os.str().c_str());
-		ret.proceed = false;
-	}
-
-	return ret;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/list_plugins.h

@@ -1,45 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-class act_list_plugins : public init_action {
-public:
-	act_list_plugins(application &app);
-	virtual ~act_list_plugins();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/load_config.cpp

@@ -1,73 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "load_config.h"
-
-namespace falco {
-namespace app {
-
-act_load_config::act_load_config(application &app)
-	: init_action(app), m_name("load config")
-{
-}
-
-act_load_config::~act_load_config()
-{
-}
-
-const std::string &act_load_config::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_load_config::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_load_config::run()
-{
-	run_result ret = {true, "", true};
-
-	if (options().conf_filename.size())
-	{
-		state().config->init(options().conf_filename, options().cmdline_config_options);
-		falco_logger::set_time_format_iso_8601(state().config->m_time_format_iso_8601);
-
-		// log after config init because config determines where logs go
-		falco_logger::log(LOG_INFO, "Falco version " + std::string(FALCO_VERSION) + " (driver version " + std::string(DRIVER_VERSION) + ")\n");
-		falco_logger::log(LOG_INFO, "Falco initialized with configuration file " + options().conf_filename + "\n");
-	}
-	else
-	{
-		ret.success = false;
-		ret.proceed = false;
-
-#ifndef BUILD_TYPE_RELEASE
-		ret.errstr = std::string("You must create a config file at ")  + FALCO_SOURCE_CONF_FILE + ", " + FALCO_INSTALL_CONF_FILE + " or by passing -c";
-#else
-		ret.errstr = std::string("You must create a config file at ")  + FALCO_INSTALL_CONF_FILE + " or by passing -c";
-#endif
-	}
-
-	state().config->m_buffered_outputs = !options().unbuffered_outputs;
-
-	return ret;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/load_config.h

@@ -1,45 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-class act_load_config : public init_action {
-public:
-	act_load_config(application &app);
-	virtual ~act_load_config();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/load_plugins.cpp

@@ -1,144 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "load_plugins.h"
-
-namespace falco {
-namespace app {
-
-act_load_plugins::act_load_plugins(application &app)
-	: init_action(app), m_name("load plugins"),
-	  m_prerequsites({"init falco engine", "load config"})
-{
-}
-
-act_load_plugins::~act_load_plugins()
-{
-}
-
-const std::string &act_load_plugins::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_load_plugins::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_load_plugins::run()
-{
-	run_result ret = {true, "", true};
-
-	// Factories that can create filters/formatters for
-	// the (single) source supported by the (single) input plugin.
-	// libs requires raw pointer, we should modify libs to use reference/shared_ptr
-	std::shared_ptr<gen_event_filter_factory> plugin_filter_factory(new sinsp_filter_factory(state().inspector.get(), m_plugin_filter_checks));
-	std::shared_ptr<gen_event_formatter_factory> plugin_formatter_factory(new sinsp_evt_formatter_factory(state().inspector.get(), m_plugin_filter_checks));
-
-	if(state().config->m_json_output)
-	{
-		plugin_formatter_factory->set_output_format(gen_event_formatter::OF_JSON);
-	}
-
-	std::shared_ptr<sinsp_plugin> input_plugin;
-	std::list<std::shared_ptr<sinsp_plugin>> extractor_plugins;
-	for(auto &p : state().config->m_plugins)
-	{
-		std::shared_ptr<sinsp_plugin> plugin;
-#ifdef MUSL_OPTIMIZED
-		ret.success = ret.proceed = false;
-		ret.errstr = "Can not load/use plugins with musl optimized build";
-		return ret;
-#else
-		falco_logger::log(LOG_INFO, "Loading plugin (" + p.m_name + ") from file " + p.m_library_path + "\n");
-
-		// libs requires raw pointer, we should modify libs to use reference/shared_ptr
-		plugin = sinsp_plugin::register_plugin(state().inspector.get(),
-						       p.m_library_path,
-						       (p.m_init_config.empty() ? NULL : (char *)p.m_init_config.c_str()),
-						       m_plugin_filter_checks);
-#endif
-
-		if(plugin->type() == TYPE_SOURCE_PLUGIN)
-		{
-			sinsp_source_plugin *splugin = static_cast<sinsp_source_plugin *>(plugin.get());
-
-			if(input_plugin)
-			{
-				ret.success = false;
-				ret.errstr = string("Can not load multiple source plugins. ") + input_plugin->name() + " already loaded";
-				ret.proceed = false;
-				return ret;
-			}
-
-			input_plugin = plugin;
-			state().event_source = splugin->event_source();
-
-			state().inspector->set_input_plugin(p.m_name);
-			if(!p.m_open_params.empty())
-			{
-				state().inspector->set_input_plugin_open_params(p.m_open_params.c_str());
-			}
-
-			state().engine->add_source(state().event_source, plugin_filter_factory, plugin_formatter_factory);
-
-		} else {
-			extractor_plugins.push_back(plugin);
-		}
-	}
-
-	// Ensure that extractor plugins are compatible with the event source.
-	// Also, ensure that extractor plugins don't have overlapping compatible event sources.
-	std::set<std::string> compat_sources_seen;
-	for(auto plugin : extractor_plugins)
-	{
-		// If the extractor plugin names compatible sources,
-		// ensure that the input plugin's source is in the list
-		// of compatible sources.
-		sinsp_extractor_plugin *eplugin = static_cast<sinsp_extractor_plugin *>(plugin.get());
-		const std::set<std::string> &compat_sources = eplugin->extract_event_sources();
-		if(input_plugin &&
-		   !compat_sources.empty())
-		{
-			if (compat_sources.find(state().event_source) == compat_sources.end())
-			{
-				ret.success = ret.proceed = false;
-				ret.errstr = string("Extractor plugin not compatible with event source ") + state().event_source;
-				return ret;
-			}
-
-			for(const auto &compat_source : compat_sources)
-			{
-				if(compat_sources_seen.find(compat_source) != compat_sources_seen.end())
-				{
-					ret.success = ret.proceed = false;
-					ret.errstr = string("Extractor plugins have overlapping compatible event source ") + compat_source;
-					return ret;
-				}
-				compat_sources_seen.insert(compat_source);
-			}
-		}
-	}
-
-	state().plugin_infos = sinsp_plugin::plugin_infos(state().inspector.get());
-
-	return ret;
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/load_plugins.h

@@ -1,52 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-class act_load_plugins : public init_action {
-public:
-	act_load_plugins(application &app);
-	virtual ~act_load_plugins();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-	// All filterchecks created by plugins go in this
-	// list. If we ever support multiple event sources at
-	// the same time, this (and the below factories) will
-	// have to be a map from event source to filtercheck
-	// list.
-	filter_check_list m_plugin_filter_checks;
-
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/load_rules_files.cpp

@@ -1,200 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include "load_rules_files.h"
-
-namespace falco {
-namespace app {
-
-act_load_rules_files::act_load_rules_files(application &app)
-	: init_action(app), m_name("load rules files"),
-	  m_prerequsites({"load plugins"})
-{
-}
-
-act_load_rules_files::~act_load_rules_files()
-{
-}
-
-const std::string &act_load_rules_files::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_load_rules_files::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_load_rules_files::run()
-{
-	run_result ret = {true, "", true};
-
-	string all_rules;
-
-	if (options().rules_filenames.size())
-	{
-		state().config->m_rules_filenames = options().rules_filenames;
-	}
-
-	if(state().config->m_rules_filenames.size() == 0)
-	{
-		ret.success = false;
-		ret.errstr = "You must specify at least one rules file/directory via -r or a rules_file entry in falco.yaml";
-		ret.proceed = false;
-		return ret;
-	}
-
-	falco_logger::log(LOG_DEBUG, "Configured rules filenames:\n");
-	for (auto filename : state().config->m_rules_filenames)
-	{
-		falco_logger::log(LOG_DEBUG, string("   ") + filename + "\n");
-	}
-
-	for (auto filename : state().config->m_rules_filenames)
-	{
-		falco_logger::log(LOG_INFO, "Loading rules from file " + filename + ":\n");
-		uint64_t required_engine_version;
-
-		try {
-			state().engine->load_rules_file(filename, options().verbose, options().all_events, required_engine_version);
-		}
-		catch(falco_exception &e)
-		{
-			ret.success = false;
-			ret.errstr = string("Could not load rules file ") + filename + ": " + e.what();
-			ret.proceed = false;
-			return ret;
-		}
-		state().required_engine_versions[filename] = required_engine_version;
-	}
-
-	// Ensure that all plugins are compatible with the loaded set of rules
-	for(auto &info : state().plugin_infos)
-	{
-		std::string required_version;
-
-		if(!state().engine->is_plugin_compatible(info.name, info.plugin_version.as_string(), required_version))
-		{
-			ret.success = false;
-			ret.errstr = std::string("Plugin ") + info.name + " version " + info.plugin_version.as_string() + " not compatible with required plugin version " + required_version;
-			ret.proceed = false;
-		}
-	}
-
-	for (auto substring : options().disabled_rule_substrings)
-	{
-		falco_logger::log(LOG_INFO, "Disabling rules matching substring: " + substring + "\n");
-		state().engine->enable_rule(substring, false);
-	}
-
-	if(options().disabled_rule_tags.size() > 0)
-	{
-		for(auto &tag : options().disabled_rule_tags)
-		{
-			falco_logger::log(LOG_INFO, "Disabling rules with tag: " + tag + "\n");
-		}
-		state().engine->enable_rule_by_tag(options().disabled_rule_tags, false);
-	}
-
-	if(options().enabled_rule_tags.size() > 0)
-	{
-		// Since we only want to enable specific
-		// rules, first disable all rules.
-		state().engine->enable_rule(all_rules, false);
-		for(auto &tag : options().enabled_rule_tags)
-		{
-			falco_logger::log(LOG_INFO, "Enabling rules with tag: " + tag + "\n");
-		}
-		state().engine->enable_rule_by_tag(options().enabled_rule_tags, true);
-	}
-
-	if(!options().all_events)
-	{
-		// For syscalls, see if any event types used by the
-		// loaded rules are ones with the EF_DROP_SIMPLE_CONS
-		// label.
-		check_for_ignored_events();
-	}
-
-	if (options().describe_all_rules)
-	{
-		state().engine->describe_rule(NULL);
-		ret.proceed = false;
-		return ret;
-	}
-
-	if (!options().describe_rule.empty())
-	{
-		state().engine->describe_rule(&(options().describe_rule));
-		ret.proceed = false;
-		return ret;
-	}
-
-	return ret;
-}
-
-void act_load_rules_files::check_for_ignored_events()
-{
-	std::set<uint16_t> evttypes;
-	sinsp_evttables* einfo = state().inspector->get_event_info_tables();
-	const struct ppm_event_info* etable = einfo->m_event_info;
-
-	state().engine->evttypes_for_ruleset(application::s_syscall_source, evttypes);
-
-	// Save event names so we don't warn for both the enter and exit event.
-	std::set<std::string> warn_event_names;
-
-	for(auto evtnum : evttypes)
-	{
-		if(evtnum == PPME_GENERIC_E || evtnum == PPME_GENERIC_X)
-		{
-			continue;
-		}
-
-		if(!sinsp::simple_consumer_consider_evtnum(evtnum))
-		{
-			std::string name = etable[evtnum].name;
-			if(warn_event_names.find(name) == warn_event_names.end())
-			{
-				warn_event_names.insert(name);
-			}
-		}
-	}
-
-	// Print a single warning with the list of ignored events
-	if (!warn_event_names.empty())
-	{
-		std::string skipped_events;
-		bool first = true;
-		for (const auto& evtname : warn_event_names)
-		{
-			if (first)
-			{
-				skipped_events += evtname;
-				first = false;
-			} else
-			{
-				skipped_events += "," + evtname;
-			}
-		}
-		fprintf(stderr,"Rules match ignored syscall: warning (ignored-evttype):\n         loaded rules match the following events: %s;\n         but these events are not returned unless running falco with -A\n", skipped_events.c_str());
-	}
-}
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/load_rules_files.h

@@ -1,47 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-
-#pragma once
-
-#include <string>
-
-#include "init_action.h"
-
-namespace falco {
-namespace app {
-
-class act_load_rules_files : public init_action {
-public:
-	act_load_rules_files(application &app);
-	virtual ~act_load_rules_files();
-
-	const std::string &name() override;
-
-	const std::list<std::string> &prerequsites() override;
-
-	run_result run() override;
-
-private:
-	void check_for_ignored_events();
-
-	std::string m_name;
-	std::list<std::string> m_prerequsites;
-};
-
-}; // namespace application
-}; // namespace falco
-

userspace/falco/app_actions/open_inspector.cpp

@@ -1,174 +0,0 @@
-/*
-Copyright (C) 2022 The Falco Authors.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-#include <sys/types.h>
-#include <sys/stat.h>
-#include <fcntl.h>
-
-#include "open_inspector.h"
-
-namespace falco {
-namespace app {
-
-act_open_inspector::act_open_inspector(application &app)
-	: run_action(app), m_name("open_inspector"),
-	  m_prerequsites({"daemonize"})
-{
-}
-
-act_open_inspector::~act_open_inspector()
-{
-}
-
-const std::string &act_open_inspector::name()
-{
-	return m_name;
-}
-
-const std::list<std::string> &act_open_inspector::prerequsites()
-{
-	return m_prerequsites;
-}
-
-runnable_action::run_result act_open_inspector::run()
-{
-	run_result ret = {true, "", true};
-
-	if(options().trace_filename.size())
-	{
-		// Try to open the trace file as a
-		// capture file first.
-		try {
-			state().inspector->open(options().trace_filename);
-			falco_logger::log(LOG_INFO, "Reading system call events from file: " + options().trace_filename + "\n");
-		}
-		catch(sinsp_exception &e)
-		{
-			falco_logger::log(LOG_DEBUG, "Could not read trace file \"" + options().trace_filename + "\": " + string(e.what()));
-			state().trace_is_scap=false;
-		}
-
-		if(!state().trace_is_scap)
-		{
-#ifdef MINIMAL_BUILD
-			ret.success = false;
-			ret.errstr = "Cannot use k8s audit events trace file with a minimal Falco build";
-			ret.proceed = false;
-			return ret;
-#else
-			try {
-				string line;
-				nlohmann::json j;
-
-				// Note we only temporarily open the file here.
-				// The read file read loop will be later.
-				ifstream ifs(options().trace_filename);
-				getline(ifs, line);
-				j = nlohmann::json::parse(line);
-
-				falco_logger::log(LOG_INFO, "Reading k8s audit events from file: " + options().trace_filename + "\n");
-			}
-			catch (nlohmann::json::parse_error& e)
-			{
-				ret.success = false;
-				ret.errstr = std::string("Trace filename ") + options().trace_filename + " not recognized as system call events or k8s audit events";
-				ret.proceed = false;
-				return ret;
-
-			}
-			catch (exception &e)
-			{
-				ret.success = false;
-				ret.errstr = std::string("Could not open trace filename ") + options().trace_filename + " for reading: " + e.what();
-				ret.proceed = false;
-				return ret;
-			}
-#endif
-		}
-	}
-	else
-	{
-		open_t open_cb = [this](std::shared_ptr<sinsp> inspector)
-			{
-				if(options().userspace)
-				{
-					// open_udig() is the underlying method used in the capture code to parse userspace events from the kernel.
-					//
-					// Falco uses a ptrace(2) based userspace implementation.
-					// Regardless of the implementation, the underlying method remains the same.
-					inspector->open_udig();
-					return;
-				}
-				inspector->open();
-			};
-		open_t open_nodriver_cb = [](std::shared_ptr<sinsp> inspector) {
-			inspector->open_nodriver();
-		};
-		open_t open_f;
-
-		// Default mode: both event sources enabled
-		if (state().enabled_sources.find(application::s_syscall_source) != state().enabled_sources.end() &&
-		    state().enabled_sources.find(application::s_k8s_audit_source) != state().enabled_sources.end())
-		{
-			open_f = open_cb;
-		}
-		if (state().enabled_sources.find(application::s_syscall_source) == state().enabled_sources.end())
-		{
-			open_f = open_nodriver_cb;
-		}
-		if (state().enabled_sources.find(application::s_k8s_audit_source) == state().enabled_sources.end())
-		{
-			open_f = open_cb;
-		}
-
-		try
-		{
-			open_f(state().inspector);
-		}
-		catch(sinsp_exception &e)
-		{
-			// If syscall input source is enabled and not through userspace instrumentation
-			if (state().enabled_sources.find(application::s_syscall_source) != state().enabled_sources.end() && !options().userspace)
-			{
-				// Try to insert the Falco kernel module
-				if(system("modprobe " DRIVER_NAME " > /dev/null 2> /dev/null"))
-				{
-					falco_logger::log(LOG_ERR, "Unable to load the driver.\n");
-				}
-				open_f(state().inspector);
-			}
-			else
-			{
-				ret.success = false;
-				ret.errstr = e.what();
-				ret.proceed = false;
-				return ret;
-			}
-		}
-	}
-
-	// This must be done after the open
-	if(!options().all_events)
-	{
-		state().inspector->start_dropping_mode(1);
-	}
-
-	return ret;
-}
-
-}; // namespace application
-}; // namespace falco
-