mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	signal/sparc32: In setup_rt_frame and setup_fram use force_fatal_sig
Modify the 32bit version of setup_rt_frame and setup_frame to act similar to the 64bit version of setup_rt_frame and fail with a signal instead of calling do_exit. Replacing do_exit(SIGILL) with force_fatal_signal(SIGILL) ensures that the process will be terminated cleanly when the stack frame is invalid, instead of just killing off a single thread and leaving the process is a weird state. Cc: David Miller <davem@davemloft.net> Cc: sparclinux@vger.kernel.org Link: https://lkml.kernel.org/r/20211020174406.17889-16-ebiederm@xmission.com Signed-off-by: Eric W. Biederman <ebiederm@xmission.com>
This commit is contained in:
		
							parent
							
								
									c317d306d5
								
							
						
					
					
						commit
						086ec444f8
					
				
					 1 changed files with 2 additions and 2 deletions
				
			
		| 
						 | 
					@ -244,7 +244,7 @@ static int setup_frame(struct ksignal *ksig, struct pt_regs *regs,
 | 
				
			||||||
		get_sigframe(ksig, regs, sigframe_size);
 | 
							get_sigframe(ksig, regs, sigframe_size);
 | 
				
			||||||
 | 
					
 | 
				
			||||||
	if (invalid_frame_pointer(sf, sigframe_size)) {
 | 
						if (invalid_frame_pointer(sf, sigframe_size)) {
 | 
				
			||||||
		do_exit(SIGILL);
 | 
							force_fatal_sig(SIGILL);
 | 
				
			||||||
		return -EINVAL;
 | 
							return -EINVAL;
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					@ -336,7 +336,7 @@ static int setup_rt_frame(struct ksignal *ksig, struct pt_regs *regs,
 | 
				
			||||||
	sf = (struct rt_signal_frame __user *)
 | 
						sf = (struct rt_signal_frame __user *)
 | 
				
			||||||
		get_sigframe(ksig, regs, sigframe_size);
 | 
							get_sigframe(ksig, regs, sigframe_size);
 | 
				
			||||||
	if (invalid_frame_pointer(sf, sigframe_size)) {
 | 
						if (invalid_frame_pointer(sf, sigframe_size)) {
 | 
				
			||||||
		do_exit(SIGILL);
 | 
							force_fatal_sig(SIGILL);
 | 
				
			||||||
		return -EINVAL;
 | 
							return -EINVAL;
 | 
				
			||||||
	}
 | 
						}
 | 
				
			||||||
 | 
					
 | 
				
			||||||
| 
						 | 
					
 | 
				
			||||||
		Loading…
	
		Reference in a new issue