mirror of
				https://github.com/torvalds/linux.git
				synced 2025-10-31 00:28:52 +02:00 
			
		
		
		
	audit: move kaudit thread start from auditd registration to kaudit init
The kauditd_thread() task was started only after the auditd userspace daemon registers itself with kaudit. This was fine when only auditd consumed messages from the kaudit netlink unicast socket. With the addition of a multicast group to that socket it is more convenient to have the thread start on init of the kaudit kernel subsystem. Signed-off-by: Richard Guy Briggs <rbriggs@redhat.com> Signed-off-by: Eric Paris <eparis@redhat.com>
This commit is contained in:
		
							parent
							
								
									3320c5133d
								
							
						
					
					
						commit
						6ff5e45985
					
				
					 1 changed files with 4 additions and 10 deletions
				
			
		|  | @ -676,16 +676,6 @@ static int audit_receive_msg(struct sk_buff *skb, struct nlmsghdr *nlh) | |||
| 	if (err) | ||||
| 		return err; | ||||
| 
 | ||||
| 	/* As soon as there's any sign of userspace auditd,
 | ||||
| 	 * start kauditd to talk to it */ | ||||
| 	if (!kauditd_task) | ||||
| 		kauditd_task = kthread_run(kauditd_thread, NULL, "kauditd"); | ||||
| 	if (IS_ERR(kauditd_task)) { | ||||
| 		err = PTR_ERR(kauditd_task); | ||||
| 		kauditd_task = NULL; | ||||
| 		return err; | ||||
| 	} | ||||
| 
 | ||||
| 	loginuid = audit_get_loginuid(current); | ||||
| 	sessionid = audit_get_sessionid(current); | ||||
| 	security_task_getsecid(current, &sid); | ||||
|  | @ -974,6 +964,10 @@ static int __init audit_init(void) | |||
| 	else | ||||
| 		audit_sock->sk_sndtimeo = MAX_SCHEDULE_TIMEOUT; | ||||
| 
 | ||||
| 	kauditd_task = kthread_run(kauditd_thread, NULL, "kauditd"); | ||||
| 	if (IS_ERR(kauditd_task)) | ||||
| 		return PTR_ERR(kauditd_task); | ||||
| 
 | ||||
| 	skb_queue_head_init(&audit_skb_queue); | ||||
| 	skb_queue_head_init(&audit_skb_hold_queue); | ||||
| 	audit_initialized = AUDIT_INITIALIZED; | ||||
|  |  | |||
		Loading…
	
		Reference in a new issue
	
	 Richard Guy Briggs
						Richard Guy Briggs