[PR #4594] [MERGED] build: allow building with address sanitizer #5200

New issue

Closed

opened 2026-05-05 10:34:06 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-05 10:34:06 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/netblue30/firejail/pull/4594
Author: @reinerh
Created: 10/6/2021
Status: ✅ Merged
Merged: 10/13/2021
Merged by: @reinerh

Base: master ← Head: asan

📝 Commits (1)

6b1c5ce build: allow building with sanitizer

📊 Changes

2 files changed (+63 additions, -0 deletions)

View changed files

📝 configure (+54 -0)
📝 configure.ac (+9 -0)

📄 Description

This allows building firejail (and other binaries) with address sanitizer (with --enable-asan).
I noticed that some were interested in this in #4592 (ping @rusty-snake @smitsohu).

I didn't rebuild configure, as I have a different autoconf version installed and it would only blow up the diff. If someone wants to test it, please run autoreconf -vfi before. (In my humble opinion generated files like configure should anyway not be part of the repository.)

Right now running some of the tools only finds memory leaks. Running firejail itself does not work for me, it fails with this error:

Error: proc 17925 cannot sync with peer: unexpected EOF

Running with --noprofile works, but I didn't figure out yet which setting is the one that breaks it.
Also so far I only tested it with gcc. clang might give different results.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/netblue30/firejail/pull/4594 **Author:** [@reinerh](https://github.com/reinerh) **Created:** 10/6/2021 **Status:** ✅ Merged **Merged:** 10/13/2021 **Merged by:** [@reinerh](https://github.com/reinerh) **Base:** `master` ← **Head:** `asan` --- ### 📝 Commits (1) - [`6b1c5ce`](https://github.com/netblue30/firejail/commit/6b1c5ceea16cc85298111fd9c8f28d2f2d36fb48) build: allow building with sanitizer ### 📊 Changes **2 files changed** (+63 additions, -0 deletions) <details> <summary>View changed files</summary> 📝 `configure` (+54 -0) 📝 `configure.ac` (+9 -0) </details> ### 📄 Description This allows building firejail (and other binaries) with address sanitizer (with `--enable-asan`). I noticed that some were interested in this in #4592 (ping @rusty-snake @smitsohu). I didn't rebuild `configure`, as I have a different autoconf version installed and it would only blow up the diff. If someone wants to test it, please run `autoreconf -vfi` before. (In my humble opinion generated files like `configure` should anyway not be part of the repository.) Right now running some of the tools only finds memory leaks. Running firejail itself does not work for me, it fails with this error: ``` Error: proc 17925 cannot sync with peer: unexpected EOF ``` Running with `--noprofile` works, but I didn't figure out yet which setting is the one that breaks it. Also so far I only tested it with gcc. clang might give different results. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

gitea-mirror

2026-05-05 10:34:06 -06:00

closed this issue
added the
pull-request
label

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: github-starred/firejail#5200

No description provided.

Rows
Columns