New issue
Advanced search Search tips

Issue 731780 link

Starred by 0 users

Issue metadata

Status: Fixed
Owner:
Closed: Jun 2017
Cc:
Components:
EstimatedDays: ----
NextAction: ----
OS: Chrome
Pri: 3
Type: Feature

Blocking:
issue 726372



Sign in to add a comment

Uprev minijail for signal forwarding

Project Member Reported by la...@chromium.org, Jun 9 2017

Issue description

Update minijail to 98f4a938cbc4acaff5f06009af50bb4bf19e173b where it will forward signals to its child process by default.
 

Comment 1 by la...@chromium.org, Jun 9 2017

jorgelo@: Is this something you would like to do? If it doesn't seem too dangerous I'm happy to take it too.

Comment 2 by la...@chromium.org, Jun 9 2017

Blocking: 726372
Feel free to give it a shot and send me the CL!

We usually list the new commits in the uprev CL commit message with something like:

git log --no-merges --oneline

Comment 4 by la...@chromium.org, Jun 9 2017

Owner: la...@chromium.org
Status: Assigned (was: Untriaged)
Project Member

Comment 5 by bugdroid1@chromium.org, Jun 12 2017

The following revision refers to this bug:
  https://chromium.googlesource.com/chromiumos/overlays/chromiumos-overlay/+/bd78c80fda2e18716871cee26aeadd060ffb527d

commit bd78c80fda2e18716871cee26aeadd060ffb527d
Author: Lann Martin <lannm@chromium.org>
Date: Mon Jun 12 19:30:36 2017

minijail: Uprev for signal forwarding functionality.

98f4a93 minijail0: adjust how we stop processing non-opts
b9a7b16 support creating new uts namespaces (and setting hostnames)
dba6209 Reland "Add optional signal forwarder and '-z' option to cli"
7151582 Revert "Add optional signal forwarder and '-z' option to cli"
74b9027 Add optional signal forwarder and '-z' option to cli
d253835 minijail0: update -i usage

BUG= chromium:731780 
TEST=Deploy updated package, test signal forwarding.
TEST=security_Minijail0

Change-Id: I390785f741c15571eb8a3fae1facadd7bf7d198e
Reviewed-on: https://chromium-review.googlesource.com/530086
Commit-Ready: Lann Martin <lannm@chromium.org>
Tested-by: Lann Martin <lannm@chromium.org>
Reviewed-by: Mike Frysinger <vapier@chromium.org>

[rename] https://crrev.com/bd78c80fda2e18716871cee26aeadd060ffb527d/chromeos-base/chromeos-minijail/chromeos-minijail-0.0.1-r1479.ebuild

Comment 6 by la...@chromium.org, Jun 12 2017

Status: Fixed (was: Assigned)

Comment 7 by sjg@google.com, Jul 5 2017

Labels: Team-BLD

Comment 8 by dchan@chromium.org, Jan 22 2018

Status: Archived (was: Fixed)

Comment 9 by vapier@chromium.org, Jun 21 2018

Components: -OS>Packages OS>Systems>Minijail
Status: Fixed (was: Archived)

Sign in to add a comment