PR/57721 CVS commit: [netbsd-10] src

["Martin Husemann" <martin%netbsd.org@localhost>]

The following reply was made to PR lib/57721; it has been noted by GNATS.

From: "Martin Husemann" <martin%netbsd.org@localhost>
To: gnats-bugs%gnats.NetBSD.org@localhost
Cc: 
Subject: PR/57721 CVS commit: [netbsd-10] src
Date: Tue, 28 Nov 2023 13:17:11 +0000

 Module Name:	src
 Committed By:	martin
 Date:		Tue Nov 28 13:17:11 UTC 2023
 
 Modified Files:
 	src/distrib/sets/lists/debug [netbsd-10]: mi
 	src/distrib/sets/lists/tests [netbsd-10]: mi
 	src/lib/libpthread [netbsd-10]: pthread.c
 	src/tests/lib/libpthread [netbsd-10]: Makefile
 Added Files:
 	src/tests/lib/libpthread [netbsd-10]: t_stack.c
 
 Log Message:
 Pull up following revision(s) (requested by riastradh in ticket #478):
 
 	tests/lib/libpthread/Makefile: revision 1.16
 	lib/libpthread/pthread.c: revision 1.184
 	distrib/sets/lists/debug/mi: revision 1.424
 	distrib/sets/lists/tests/mi: revision 1.1297
 	tests/lib/libpthread/t_stack.c: revision 1.1
 	tests/lib/libpthread/t_stack.c: revision 1.2
 	tests/lib/libpthread/t_stack.c: revision 1.3
 	tests/lib/libpthread/t_stack.c: revision 1.4
 	tests/lib/libpthread/t_stack.c: revision 1.5
 	tests/lib/libpthread/t_stack.c: revision 1.6
 
 pthread: Add tests for pthread user stack allocation.
 PR lib/57721
 
 libpthread/t_stack: Make this more robust to the guard size bug.
 Make sure to allocate enough space for the thread's stack for a guard
 even though there shouldn't be one, so that when we run the thread,
 it doesn't start with the stack pointer pointing into someone else's
 allocation (like malloc) causing stack frames to trash another data
 structure -- or causing the user of that data structure to trash the
 stack frames.
 PR lib/57721
 
 libpthread/t_stack: Omit needless cast in previous.
 Arose from an earlier draft of the change.
 PR lib/57721
 
 libpthread/t_stack: Appease gcc12 maybe-uninitialized warning.
 The jmp_buf is not, in fact, uninitialized at the point of use, but
 it doesn't hurt to narrow the scope a bit to between when the jmp_buf
 is initialized by setjmp, and when the signal handler might be called
 after sigaction.
 Noted by prlw1.
 PR lib/57721
 
 libpthread/t_stack: Fix format string for size_t.
 Tested this on i386 since that had been crashing before, but i386
 doesn't see %zu for unsigned int as a problem.
 PR lib/57721
 
 pthread: Don't adjust user-allocated stack addresses by guardsize.
 PR lib/57721
 
 
 To generate a diff of this commit:
 cvs rdiff -u -r1.394.2.3 -r1.394.2.4 src/distrib/sets/lists/debug/mi
 cvs rdiff -u -r1.1238.2.4 -r1.1238.2.5 src/distrib/sets/lists/tests/mi
 cvs rdiff -u -r1.181 -r1.181.2.1 src/lib/libpthread/pthread.c
 cvs rdiff -u -r1.15 -r1.15.6.1 src/tests/lib/libpthread/Makefile
 cvs rdiff -u -r0 -r1.6.2.2 src/tests/lib/libpthread/t_stack.c
 
 Please note that diffs are not public domain; they are subject to the
 copyright notices on the relevant files.