linux-kernel - Re: [PATCH v2 1/2] signal.7: Clearly describe ucontext kernel dump to userspace

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date: Mon, 17 Jun 2024 12:56:11 +0200
From: Alejandro Colomar <alx@...nel.org>
To: Dev Jain <dev.jain@....com>
Cc: linux-man@...r.kernel.org, mingo@...nel.org, tglx@...utronix.de, 
	mark.rutland@....com, ryan.roberts@....com, broonie@...nel.org, suzuki.poulose@....com, 
	Anshuman.Khandual@....com, DeepakKumar.Mishra@....com, aneesh.kumar@...nel.org, 
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH v2 1/2] signal.7: Clearly describe ucontext kernel dump
 to userspace

On Mon, Jun 17, 2024 at 12:54:10PM GMT, Alejandro Colomar wrote:
> On Tue, Jun 11, 2024 at 02:38:22PM GMT, Dev Jain wrote:
> > The set of blocked signals is updated from struct sigaction after the
> > kernel dumps ucontext. Mention this to avoid misunderstanding.
> > 
> > Signed-off-by: Dev Jain <dev.jain@....com>
> 
> Hi Dev,
> 
> I've applied this patch, with some minor tweaks:
> <https://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/commit/?h=contrib&id=9e1f221b2099ee2c936ce4d6562a61e1e253af47>

Whoops; I found another issue, and amended:
<https://www.alejandro-colomar.es/src/alx/linux/man-pages/man-pages.git/commit/?h=contrib&id=b8136d84eb3b9b1db0dd6f54f47c1c104682f44f>


> 
> Have a lovely day!
> Alex
> 
> > ---
> >  man/man7/signal.7 | 8 +++++++-
> >  1 file changed, 7 insertions(+), 1 deletion(-)
> > 
> > diff --git a/man/man7/signal.7 b/man/man7/signal.7
> > index 4ba5306c8..7d22a7cfe 100644
> > --- a/man/man7/signal.7
> > +++ b/man/man7/signal.7
> > @@ -282,7 +282,13 @@ the thread's alternate signal stack settings.
> >  .B SA_SIGINFO
> >  flag, then the above information is accessible via the
> >  .I ucontext_t
> > -object that is pointed to by the third argument of the signal handler.)
> > +object that is pointed to by the third argument of the signal handler.
> > +This object reflects the state at which the signal is delivered, rather
> > +than in the handler;
> > +for example, the mask of blocked signals stored in this object will not
> > +contain the mask of new signals blocked through
> > +.BR sigaction(2)).
> > +
> >  .IP (1.4)
> >  Any signals specified in
> >  .I act\->sa_mask
> > -- 
> > 2.34.1
> > 
> > 
> 
> -- 
> <https://www.alejandro-colomar.es/>



-- 
<https://www.alejandro-colomar.es/>

Download attachment "signature.asc" of type "application/pgp-signature" (834 bytes)