summaryrefslogtreecommitdiff
path: root/mozilla-1673202.patch
diff options
context:
space:
mode:
authorMartin Stransky <stransky@redhat.com>2020-10-27 10:47:08 +0100
committerMartin Stransky <stransky@redhat.com>2020-10-27 10:47:08 +0100
commit8f7fa14c2786e50657d3667943443850e5a63318 (patch)
tree644ea1f51309bd3ee5172fad161223b11b3e82bc /mozilla-1673202.patch
parentAdded fix for rawhide crashes (rhbz#1891234) (diff)
downloadlibrewolf-fedora-ff-8f7fa14c2786e50657d3667943443850e5a63318.tar.gz
librewolf-fedora-ff-8f7fa14c2786e50657d3667943443850e5a63318.tar.bz2
librewolf-fedora-ff-8f7fa14c2786e50657d3667943443850e5a63318.zip
Fixed mozilla-1673202.patch patch to apply
Diffstat (limited to 'mozilla-1673202.patch')
-rw-r--r--mozilla-1673202.patch38
1 files changed, 4 insertions, 34 deletions
diff --git a/mozilla-1673202.patch b/mozilla-1673202.patch
index 711a975..b8695a1 100644
--- a/mozilla-1673202.patch
+++ b/mozilla-1673202.patch
@@ -1,31 +1,7 @@
-# HG changeset patch
-# User Jed Davis <jld@mozilla.com>
-
-Bug 1673202 - Call fstat directly in Linux sandbox fstatat interception. r?gcp
-
-Sandbox policies handle the case of `fstatat(fd, "", AT_EMPTY_PATH|...)`
-by invoking the SIGSYS handler (because seccomp-bpf can't tell if the
-string will be empty when the syscall would use it), which makes the
-equivalent call to `fstat`.
-
-Unfortunately, recent development versions of glibc implement `fstat` by
-calling `fstatat`, which causes unbounded recursion and stack overflow.
-(This depends on the headers present at build time; see the bug for more
-details.) This patch switches it to use the `fstat` (or `fstat64` on
-32-bit) syscall directly.
-
-Differential Revision: https://phabricator.services.mozilla.com/D94798
-
-diff --git a/security/sandbox/linux/SandboxFilter.cpp b/security/sandbox/linux/SandboxFilter.cpp
-index 9bdb10c49e085..a128cce7b266c 100644
---- a/security/sandbox/linux/SandboxFilter.cpp
-+++ b/security/sandbox/linux/SandboxFilter.cpp
-@@ -294,17 +294,21 @@ class SandboxPolicyCommon : public SandboxPolicyBase {
- auto broker = static_cast<SandboxBrokerClient*>(aux);
- auto fd = static_cast<int>(aArgs.args[0]);
- auto path = reinterpret_cast<const char*>(aArgs.args[1]);
- auto buf = reinterpret_cast<statstruct*>(aArgs.args[2]);
- auto flags = static_cast<int>(aArgs.args[3]);
+diff -up firefox-82.0/security/sandbox/linux/SandboxFilter.cpp.1673202 firefox-82.0/security/sandbox/linux/SandboxFilter.cpp
+--- firefox-82.0/security/sandbox/linux/SandboxFilter.cpp.1673202 2020-10-27 10:40:54.903158025 +0100
++++ firefox-82.0/security/sandbox/linux/SandboxFilter.cpp 2020-10-27 10:43:19.024883597 +0100
+@@ -246,7 +246,11 @@ class SandboxPolicyCommon : public Sandb
if (fd != AT_FDCWD && (flags & AT_EMPTY_PATH) != 0 &&
strcmp(path, "") == 0) {
@@ -38,9 +14,3 @@ index 9bdb10c49e085..a128cce7b266c 100644
}
if (fd != AT_FDCWD && path[0] != '/') {
- SANDBOX_LOG_ERROR("unsupported fd-relative fstatat(%d, \"%s\", %p, 0x%x)",
- fd, path, buf, flags);
- return BlockedSyscallTrap(aArgs, nullptr);
- }
-
-
bgstack15