Skip site navigation (1)Skip section navigation (2)
Date:      Fri, 3 Aug 2018 14:22:16 +0000 (UTC)
From:      Alan Somers <asomers@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   svn commit: r337251 - stable/10/sys/security/audit
Message-ID:  <201808031422.w73EMGC5076151@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: asomers
Date: Fri Aug  3 14:22:16 2018
New Revision: 337251
URL: https://svnweb.freebsd.org/changeset/base/337251

Log:
  MFC r334296:
  
  Fix "Bad tailq" panic when auditing auditon(A_SETCLASS, ...)
  
  Due to an oversight in r195280, auditon(A_SETCLASS, ...) would cause a tailq
  element to get added to the tailq twice, resulting in a circular tailq. This
  panics when INVARIANTS are on.
  
  Differential Revision:	https://reviews.freebsd.org/D15381

Modified:
  stable/10/sys/security/audit/audit_bsm.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/sys/security/audit/audit_bsm.c
==============================================================================
--- stable/10/sys/security/audit/audit_bsm.c	Fri Aug  3 14:19:56 2018	(r337250)
+++ stable/10/sys/security/audit/audit_bsm.c	Fri Aug  3 14:22:16 2018	(r337251)
@@ -418,7 +418,6 @@ audit_sys_auditon(struct audit_record *ar, struct au_r
 		break;
 
 	case A_SETCLASS:
-		kau_write(rec, tok);
 		tok = au_to_arg32(2, "setclass:ec_event",
 		    ar->ar_arg_auditon.au_evclass.ec_number);
 		kau_write(rec, tok);



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