mirror of
				https://github.com/torvalds/linux.git
				synced 2025-11-04 10:40:15 +02:00 
			
		
		
		
	X25: Dont let x25_bind use addresses containing characters
Addresses should be all digits. Stops x25_bind using addresses containing characters. Signed-off-by: Andrew Hendry <andrew.hendry@gmail.com> Signed-off-by: David S. Miller <davem@davemloft.net>
This commit is contained in:
		
							parent
							
								
									b18e7a0685
								
							
						
					
					
						commit
						a9288525d2
					
				
					 1 changed files with 10 additions and 1 deletions
				
			
		| 
						 | 
				
			
			@ -55,6 +55,7 @@
 | 
			
		|||
#include <linux/notifier.h>
 | 
			
		||||
#include <linux/init.h>
 | 
			
		||||
#include <linux/compat.h>
 | 
			
		||||
#include <linux/ctype.h>
 | 
			
		||||
 | 
			
		||||
#include <net/x25.h>
 | 
			
		||||
#include <net/compat.h>
 | 
			
		||||
| 
						 | 
				
			
			@ -648,7 +649,7 @@ static int x25_bind(struct socket *sock, struct sockaddr *uaddr, int addr_len)
 | 
			
		|||
{
 | 
			
		||||
	struct sock *sk = sock->sk;
 | 
			
		||||
	struct sockaddr_x25 *addr = (struct sockaddr_x25 *)uaddr;
 | 
			
		||||
	int rc = 0;
 | 
			
		||||
	int len, i, rc = 0;
 | 
			
		||||
 | 
			
		||||
	lock_kernel();
 | 
			
		||||
	if (!sock_flag(sk, SOCK_ZAPPED) ||
 | 
			
		||||
| 
						 | 
				
			
			@ -658,6 +659,14 @@ static int x25_bind(struct socket *sock, struct sockaddr *uaddr, int addr_len)
 | 
			
		|||
		goto out;
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	len = strlen(addr->sx25_addr.x25_addr);
 | 
			
		||||
	for (i = 0; i < len; i++) {
 | 
			
		||||
		if (!isdigit(addr->sx25_addr.x25_addr[i])) {
 | 
			
		||||
			rc = -EINVAL;
 | 
			
		||||
			goto out;
 | 
			
		||||
		}
 | 
			
		||||
	}
 | 
			
		||||
 | 
			
		||||
	x25_sk(sk)->source_addr = addr->sx25_addr;
 | 
			
		||||
	x25_insert_socket(sk);
 | 
			
		||||
	sock_reset_flag(sk, SOCK_ZAPPED);
 | 
			
		||||
| 
						 | 
				
			
			
 | 
			
		|||
		Loading…
	
		Reference in a new issue