Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 3 Jan 2022 23:30:00 GMT
From:      Cy Schubert <cy@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-branches@FreeBSD.org
Subject:   git: 12c711f167ae - stable/13 - ntp: Revert "Disable ntpd stack gap. When ASLR with STACK GAP != 0 ntpd suffers SIGSEGV."
Message-ID:  <202201032330.203NU0KG028681@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch stable/13 has been updated by cy:

URL: https://cgit.FreeBSD.org/src/commit/?id=12c711f167ae6a3d9db66d7a59d4556b9a4ffbb5

commit 12c711f167ae6a3d9db66d7a59d4556b9a4ffbb5
Author:     Cy Schubert <cy@FreeBSD.org>
AuthorDate: 2021-08-16 15:05:50 +0000
Commit:     Cy Schubert <cy@FreeBSD.org>
CommitDate: 2022-01-03 23:04:05 +0000

    ntp: Revert "Disable ntpd stack gap. When ASLR with STACK GAP != 0 ntpd suffers SIGSEGV."
    
    This reverts commit 55263180628a386ad38b4b6621919220b137f770 because the
    underlying problem was fixed by 889b56c8cd84c9a9f2d9e3b019c154d6f14d9021.
    
    (cherry picked from commit fc393054398ea50fb0cee52704e9385afe888b48)
---
 contrib/ntp/ntpd/ntpd.c | 23 -----------------------
 1 file changed, 23 deletions(-)

diff --git a/contrib/ntp/ntpd/ntpd.c b/contrib/ntp/ntpd/ntpd.c
index aac315c430da..a5591de79ff5 100644
--- a/contrib/ntp/ntpd/ntpd.c
+++ b/contrib/ntp/ntpd/ntpd.c
@@ -145,17 +145,6 @@
 # include <seccomp.h>
 #endif /* LIBSECCOMP and KERN_SECCOMP */
 
-#ifdef __FreeBSD__
-#include <sys/procctl.h>
-#ifndef PROC_STACKGAP_CTL
-/*
- * Even if we compile on an older system we can still run on a newer one.
- */
-#define	PROC_STACKGAP_CTL	17
-#define	PROC_STACKGAP_DISABLE	0x0002
-#endif
-#endif
-
 #ifdef HAVE_DNSREGISTRATION
 # include <dns_sd.h>
 DNSServiceRef mdns;
@@ -438,18 +427,6 @@ main(
 	char *argv[]
 	)
 {
-#   ifdef __FreeBSD__
-	{
-		/*
-		 * We Must disable ASLR stack gap on FreeBSD to avoid a
-		 * segfault. See PR/241421 and PR/241960.
-		 */
-		int aslr_var = PROC_STACKGAP_DISABLE;
-
-		pid_t my_pid = getpid();
-		procctl(P_PID, my_pid, PROC_STACKGAP_CTL, &aslr_var); 
-	}
-#   endif
 	return ntpdmain(argc, argv);
 }
 #endif /* !SYS_WINNT */



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202201032330.203NU0KG028681>