Re: [PATCH] linux-user: rt_sigprocmask, check read perms first

2022-01-26 Thread Patrick Venture
On Tue, Jan 11, 2022 at 3:06 PM Patrick Venture wrote: > > > On Tue, Jan 11, 2022 at 12:50 PM Laurent Vivier wrote: > >> Hi Patrick, >> >> Le 11/01/2022 à 21:14, Patrick Venture a écrit : >> > >> > >> > On Sat, Jan 8, 2022 at 10:16 AM Laurent Vivier > > wrote: >> > >>

Re: [PATCH] linux-user: rt_sigprocmask, check read perms first

2022-01-11 Thread Patrick Venture
On Tue, Jan 11, 2022 at 12:50 PM Laurent Vivier wrote: > Hi Patrick, > > Le 11/01/2022 à 21:14, Patrick Venture a écrit : > > > > > > On Sat, Jan 8, 2022 at 10:16 AM Laurent Vivier > wrote: > > > > Le 06/01/2022 à 23:00, Patrick Venture a écrit : > > > From:

Re: [PATCH] linux-user: rt_sigprocmask, check read perms first

2022-01-11 Thread Laurent Vivier
Hi Patrick, Le 11/01/2022 à 21:14, Patrick Venture a écrit : On Sat, Jan 8, 2022 at 10:16 AM Laurent Vivier mailto:laur...@vivier.eu>> wrote: Le 06/01/2022 à 23:00, Patrick Venture a écrit : > From: Shu-Chun Weng mailto:s...@google.com>> > > Linux kernel does it this way

Re: [PATCH] linux-user: rt_sigprocmask, check read perms first

2022-01-11 Thread Patrick Venture
On Sat, Jan 8, 2022 at 10:16 AM Laurent Vivier wrote: > Le 06/01/2022 à 23:00, Patrick Venture a écrit : > > From: Shu-Chun Weng > > > > Linux kernel does it this way (checks read permission before validating > `how`) > > and the latest version of ABSL's `AddressIsReadable()` depends on this >

Re: [PATCH] linux-user: rt_sigprocmask, check read perms first

2022-01-08 Thread Laurent Vivier
Le 06/01/2022 à 23:00, Patrick Venture a écrit : From: Shu-Chun Weng Linux kernel does it this way (checks read permission before validating `how`) and the latest version of ABSL's `AddressIsReadable()` depends on this behavior. c.f.

[PATCH] linux-user: rt_sigprocmask, check read perms first

2022-01-06 Thread Patrick Venture
From: Shu-Chun Weng Linux kernel does it this way (checks read permission before validating `how`) and the latest version of ABSL's `AddressIsReadable()` depends on this behavior. c.f. https://github.com/torvalds/linux/blob/9539ba4308ad5bdca6cb41c7b73cbb9f796dcdd7/kernel/signal.c#L3147