]> git.ipfire.org Git - thirdparty/systemd.git/blame - docs/HACKING.md
docs: permit user/group services that do not support enumeration
[thirdparty/systemd.git] / docs / HACKING.md
CommitLineData
c3e270f4
FB
1---
2title: Hacking on systemd
4cdca0af 3category: Contributing
b41a3f66 4layout: default
c3e270f4
FB
5---
6
5a8a9dee
FA
7# Hacking on systemd
8
9We welcome all contributions to systemd. If you notice a bug or a missing
10feature, please feel invited to fix it, and submit your work as a GitHub Pull
11Request (PR) at https://github.com/systemd/systemd/pull/new.
12
13Please make sure to follow our [Coding Style](CODING_STYLE.md) when submitting patches.
14Also have a look at our [Contribution Guidelines](CONTRIBUTING.md).
15
16When adding new functionality, tests should be added. For shared functionality
17(in `src/basic/` and `src/shared/`) unit tests should be sufficient. The general
18policy is to keep tests in matching files underneath `src/test/`,
19e.g. `src/test/test-path-util.c` contains tests for any functions in
20`src/basic/path-util.c`. If adding a new source file, consider adding a matching
21test executable. For features at a higher level, tests in `src/test/` are very
1e268f42 22strongly recommended. If that is not possible, integration tests in `test/` are
5a8a9dee
FA
23encouraged.
24
25Please also have a look at our list of [code quality tools](CODE_QUALITY.md) we have setup for systemd,
26to ensure our codebase stays in good shape.
27
28Please always test your work before submitting a PR. For many of the components
29of systemd testing is straight-forward as you can simply compile systemd and
30run the relevant tool from the build directory.
31
32For some components (most importantly, systemd/PID1 itself) this is not
33possible, however. In order to simplify testing for cases like this we provide
34a set of `mkosi` build files directly in the source tree. `mkosi` is a tool for
35building clean OS images from an upstream distribution in combination with a
36fresh build of the project in the local working directory. To make use of this,
37please acquire `mkosi` from https://github.com/systemd/mkosi first, unless your
38distribution has packaged it already and you can get it from there. After the
39tool is installed it is sufficient to type `mkosi` in the systemd project
40directory to generate a disk image `image.raw` you can boot either in
41`systemd-nspawn` or in an UEFI-capable VM:
42
43```
44# systemd-nspawn -bi image.raw
45```
46
47or:
48
49```
50# qemu-system-x86_64 -enable-kvm -m 512 -smp 2 -bios /usr/share/edk2/ovmf/OVMF_CODE.fd -hda image.raw
51```
52
53Every time you rerun the `mkosi` command a fresh image is built, incorporating
54all current changes you made to the project tree.
55
56Alternatively, you may install the systemd version from your git check-out
57directly on top of your host system's directory tree. This mostly works fine,
58but of course you should know what you are doing as you might make your system
59unbootable in case of a bug in your changes. Also, you might step into your
60package manager's territory with this. Be careful!
61
62And never forget: most distributions provide very simple and convenient ways to
63install all development packages necessary to build systemd. For example, on
64Fedora the following command line should be sufficient to install all of
65systemd's build dependencies:
66
67```
68# dnf builddep systemd
69```
70
71Putting this all together, here's a series of commands for preparing a patch
72for systemd (this example is for Fedora):
73
74```sh
75$ sudo dnf builddep systemd # install build dependencies
76$ sudo dnf install mkosi # install tool to quickly build images
77$ git clone https://github.com/systemd/systemd.git
78$ cd systemd
79$ vim src/core/main.c # or wherever you'd like to make your changes
80$ meson build # configure the build
81$ ninja -C build # build it locally, see if everything compiles fine
82$ ninja -C build test # run some simple regression tests
83$ (umask 077; echo 123 > mkosi.rootpw) # set root password used by mkosi
84$ sudo mkosi # build a test image
85$ sudo systemd-nspawn -bi image.raw # boot up the test image
86$ git add -p # interactively put together your patch
87$ git commit # commit it
88$ git push REMOTE HEAD:refs/heads/BRANCH
89 # where REMOTE is your "fork" on GitHub
90 # and BRANCH is a branch name.
91```
92
93And after that, head over to your repo on GitHub and click "Compare & pull request"
94
95Happy hacking!
96
97
98## Fuzzers
99
100systemd includes fuzzers in `src/fuzz/` that use libFuzzer and are automatically
135a1add 101run by [OSS-Fuzz](https://github.com/google/oss-fuzz) with sanitizers.
53a42e62 102To add a fuzz target, create a new `src/fuzz/fuzz-foo.c` file with a `LLVMFuzzerTestOneInput`
5a8a9dee
FA
103function and add it to the list in `src/fuzz/meson.build`.
104
105Whenever possible, a seed corpus and a dictionary should also be added with new
106fuzz targets. The dictionary should be named `src/fuzz/fuzz-foo.dict` and the seed
107corpus should be built and exported as `$OUT/fuzz-foo_seed_corpus.zip` in
108`tools/oss-fuzz.sh`.
109
110The fuzzers can be built locally if you have libFuzzer installed by running
111`tools/oss-fuzz.sh`. You should also confirm that the fuzzer runs in the
112OSS-Fuzz environment by checking out the OSS-Fuzz repo, and then running
113commands like this:
114
115```
116python infra/helper.py build_image systemd
117python infra/helper.py build_fuzzers --sanitizer memory systemd ../systemd
118python infra/helper.py run_fuzzer systemd fuzz-foo
119```
120
121If you find a bug that impacts the security of systemd, please follow the
122guidance in [CONTRIBUTING.md](CONTRIBUTING.md) on how to report a security vulnerability.
123
124For more details on building fuzzers and integrating with OSS-Fuzz, visit:
125
6cec69fc
LK
126- [Setting up a new project - OSS-Fuzz](https://google.github.io/oss-fuzz/getting-started/new-project-guide/)
127- [Tutorials - OSS-Fuzz](https://google.github.io/oss-fuzz/reference/useful-links/#tutorials)