diff options
author | Joe Perches <joe@perches.com> | 2010-01-26 06:40:07 -0500 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2010-01-27 00:15:59 -0500 |
commit | 5ff7ef7911d100b6568c731b1d078f819da82d03 (patch) | |
tree | d723b682b0828f48af7e0184dbd0e5da64d8412c /net/atm/ioctl.c | |
parent | a8147d737bd37bd51bb01737ac9c17a2cfc02a38 (diff) |
net/atm/ioctl.c: checkpatch cleanups
Spacing cleanups
Moved EXPORT_SYMBOL
Mostly 80 column wrapped.
switch/case cleanups
Signed-off-by: Joe Perches <joe@perches.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'net/atm/ioctl.c')
-rw-r--r-- | net/atm/ioctl.c | 195 |
1 files changed, 97 insertions, 98 deletions
diff --git a/net/atm/ioctl.c b/net/atm/ioctl.c index b75afba1f72b..62dc8bfe6fe7 100644 --- a/net/atm/ioctl.c +++ b/net/atm/ioctl.c | |||
@@ -37,6 +37,7 @@ void register_atm_ioctl(struct atm_ioctl *ioctl) | |||
37 | list_add_tail(&ioctl->list, &ioctl_list); | 37 | list_add_tail(&ioctl->list, &ioctl_list); |
38 | mutex_unlock(&ioctl_mutex); | 38 | mutex_unlock(&ioctl_mutex); |
39 | } | 39 | } |
40 | EXPORT_SYMBOL(register_atm_ioctl); | ||
40 | 41 | ||
41 | void deregister_atm_ioctl(struct atm_ioctl *ioctl) | 42 | void deregister_atm_ioctl(struct atm_ioctl *ioctl) |
42 | { | 43 | { |
@@ -44,129 +45,128 @@ void deregister_atm_ioctl(struct atm_ioctl *ioctl) | |||
44 | list_del(&ioctl->list); | 45 | list_del(&ioctl->list); |
45 | mutex_unlock(&ioctl_mutex); | 46 | mutex_unlock(&ioctl_mutex); |
46 | } | 47 | } |
47 | |||
48 | EXPORT_SYMBOL(register_atm_ioctl); | ||
49 | EXPORT_SYMBOL(deregister_atm_ioctl); | 48 | EXPORT_SYMBOL(deregister_atm_ioctl); |
50 | 49 | ||
51 | static int do_vcc_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg, int compat) | 50 | static int do_vcc_ioctl(struct socket *sock, unsigned int cmd, |
51 | unsigned long arg, int compat) | ||
52 | { | 52 | { |
53 | struct sock *sk = sock->sk; | 53 | struct sock *sk = sock->sk; |
54 | struct atm_vcc *vcc; | 54 | struct atm_vcc *vcc; |
55 | int error; | 55 | int error; |
56 | struct list_head * pos; | 56 | struct list_head *pos; |
57 | void __user *argp = (void __user *)arg; | 57 | void __user *argp = (void __user *)arg; |
58 | 58 | ||
59 | vcc = ATM_SD(sock); | 59 | vcc = ATM_SD(sock); |
60 | switch (cmd) { | 60 | switch (cmd) { |
61 | case SIOCOUTQ: | 61 | case SIOCOUTQ: |
62 | if (sock->state != SS_CONNECTED || | 62 | if (sock->state != SS_CONNECTED || |
63 | !test_bit(ATM_VF_READY, &vcc->flags)) { | 63 | !test_bit(ATM_VF_READY, &vcc->flags)) { |
64 | error = -EINVAL; | 64 | error = -EINVAL; |
65 | goto done; | 65 | goto done; |
66 | } | 66 | } |
67 | error = put_user(sk->sk_sndbuf - sk_wmem_alloc_get(sk), | 67 | error = put_user(sk->sk_sndbuf - sk_wmem_alloc_get(sk), |
68 | (int __user *) argp) ? -EFAULT : 0; | 68 | (int __user *)argp) ? -EFAULT : 0; |
69 | goto done; | ||
70 | case SIOCINQ: | ||
71 | { | ||
72 | struct sk_buff *skb; | ||
73 | |||
74 | if (sock->state != SS_CONNECTED) { | ||
75 | error = -EINVAL; | ||
69 | goto done; | 76 | goto done; |
70 | case SIOCINQ: | 77 | } |
71 | { | 78 | skb = skb_peek(&sk->sk_receive_queue); |
72 | struct sk_buff *skb; | 79 | error = put_user(skb ? skb->len : 0, |
73 | 80 | (int __user *)argp) ? -EFAULT : 0; | |
74 | if (sock->state != SS_CONNECTED) { | 81 | goto done; |
75 | error = -EINVAL; | 82 | } |
76 | goto done; | 83 | case SIOCGSTAMP: /* borrowed from IP */ |
77 | } | ||
78 | skb = skb_peek(&sk->sk_receive_queue); | ||
79 | error = put_user(skb ? skb->len : 0, | ||
80 | (int __user *)argp) ? -EFAULT : 0; | ||
81 | goto done; | ||
82 | } | ||
83 | case SIOCGSTAMP: /* borrowed from IP */ | ||
84 | #ifdef CONFIG_COMPAT | 84 | #ifdef CONFIG_COMPAT |
85 | if (compat) | 85 | if (compat) |
86 | error = compat_sock_get_timestamp(sk, argp); | 86 | error = compat_sock_get_timestamp(sk, argp); |
87 | else | 87 | else |
88 | #endif | 88 | #endif |
89 | error = sock_get_timestamp(sk, argp); | 89 | error = sock_get_timestamp(sk, argp); |
90 | goto done; | 90 | goto done; |
91 | case SIOCGSTAMPNS: /* borrowed from IP */ | 91 | case SIOCGSTAMPNS: /* borrowed from IP */ |
92 | #ifdef CONFIG_COMPAT | 92 | #ifdef CONFIG_COMPAT |
93 | if (compat) | 93 | if (compat) |
94 | error = compat_sock_get_timestampns(sk, argp); | 94 | error = compat_sock_get_timestampns(sk, argp); |
95 | else | 95 | else |
96 | #endif | 96 | #endif |
97 | error = sock_get_timestampns(sk, argp); | 97 | error = sock_get_timestampns(sk, argp); |
98 | goto done; | ||
99 | case ATM_SETSC: | ||
100 | if (net_ratelimit()) | ||
101 | pr_warning("ATM_SETSC is obsolete; used by %s:%d\n", | ||
102 | current->comm, task_pid_nr(current)); | ||
103 | error = 0; | ||
104 | goto done; | ||
105 | case ATMSIGD_CTRL: | ||
106 | if (!capable(CAP_NET_ADMIN)) { | ||
107 | error = -EPERM; | ||
98 | goto done; | 108 | goto done; |
99 | case ATM_SETSC: | 109 | } |
100 | if (net_ratelimit()) | 110 | /* |
101 | pr_warning("ATM_SETSC is obsolete; used by %s:%d\n", | 111 | * The user/kernel protocol for exchanging signalling |
102 | current->comm, task_pid_nr(current)); | 112 | * info uses kernel pointers as opaque references, |
103 | error = 0; | 113 | * so the holder of the file descriptor can scribble |
114 | * on the kernel... so we should make sure that we | ||
115 | * have the same privileges that /proc/kcore needs | ||
116 | */ | ||
117 | if (!capable(CAP_SYS_RAWIO)) { | ||
118 | error = -EPERM; | ||
104 | goto done; | 119 | goto done; |
105 | case ATMSIGD_CTRL: | 120 | } |
106 | if (!capable(CAP_NET_ADMIN)) { | ||
107 | error = -EPERM; | ||
108 | goto done; | ||
109 | } | ||
110 | /* | ||
111 | * The user/kernel protocol for exchanging signalling | ||
112 | * info uses kernel pointers as opaque references, | ||
113 | * so the holder of the file descriptor can scribble | ||
114 | * on the kernel... so we should make sure that we | ||
115 | * have the same privileges that /proc/kcore needs | ||
116 | */ | ||
117 | if (!capable(CAP_SYS_RAWIO)) { | ||
118 | error = -EPERM; | ||
119 | goto done; | ||
120 | } | ||
121 | #ifdef CONFIG_COMPAT | 121 | #ifdef CONFIG_COMPAT |
122 | /* WTF? I don't even want to _think_ about making this | 122 | /* WTF? I don't even want to _think_ about making this |
123 | work for 32-bit userspace. TBH I don't really want | 123 | work for 32-bit userspace. TBH I don't really want |
124 | to think about it at all. dwmw2. */ | 124 | to think about it at all. dwmw2. */ |
125 | if (compat) { | 125 | if (compat) { |
126 | if (net_ratelimit()) | 126 | if (net_ratelimit()) |
127 | pr_warning("32-bit task cannot be atmsigd\n"); | 127 | pr_warning("32-bit task cannot be atmsigd\n"); |
128 | error = -EINVAL; | 128 | error = -EINVAL; |
129 | goto done; | 129 | goto done; |
130 | } | 130 | } |
131 | #endif | 131 | #endif |
132 | error = sigd_attach(vcc); | 132 | error = sigd_attach(vcc); |
133 | if (!error) | 133 | if (!error) |
134 | sock->state = SS_CONNECTED; | 134 | sock->state = SS_CONNECTED; |
135 | goto done; | ||
136 | case ATM_SETBACKEND: | ||
137 | case ATM_NEWBACKENDIF: | ||
138 | { | ||
139 | atm_backend_t backend; | ||
140 | error = get_user(backend, (atm_backend_t __user *)argp); | ||
141 | if (error) | ||
135 | goto done; | 142 | goto done; |
136 | case ATM_SETBACKEND: | 143 | switch (backend) { |
137 | case ATM_NEWBACKENDIF: | 144 | case ATM_BACKEND_PPP: |
138 | { | 145 | request_module("pppoatm"); |
139 | atm_backend_t backend; | ||
140 | error = get_user(backend, (atm_backend_t __user *) argp); | ||
141 | if (error) | ||
142 | goto done; | ||
143 | switch (backend) { | ||
144 | case ATM_BACKEND_PPP: | ||
145 | request_module("pppoatm"); | ||
146 | break; | ||
147 | case ATM_BACKEND_BR2684: | ||
148 | request_module("br2684"); | ||
149 | break; | ||
150 | } | ||
151 | } | ||
152 | break; | ||
153 | case ATMMPC_CTRL: | ||
154 | case ATMMPC_DATA: | ||
155 | request_module("mpoa"); | ||
156 | break; | ||
157 | case ATMARPD_CTRL: | ||
158 | request_module("clip"); | ||
159 | break; | 146 | break; |
160 | case ATMLEC_CTRL: | 147 | case ATM_BACKEND_BR2684: |
161 | request_module("lec"); | 148 | request_module("br2684"); |
162 | break; | 149 | break; |
150 | } | ||
151 | break; | ||
152 | } | ||
153 | case ATMMPC_CTRL: | ||
154 | case ATMMPC_DATA: | ||
155 | request_module("mpoa"); | ||
156 | break; | ||
157 | case ATMARPD_CTRL: | ||
158 | request_module("clip"); | ||
159 | break; | ||
160 | case ATMLEC_CTRL: | ||
161 | request_module("lec"); | ||
162 | break; | ||
163 | } | 163 | } |
164 | 164 | ||
165 | error = -ENOIOCTLCMD; | 165 | error = -ENOIOCTLCMD; |
166 | 166 | ||
167 | mutex_lock(&ioctl_mutex); | 167 | mutex_lock(&ioctl_mutex); |
168 | list_for_each(pos, &ioctl_list) { | 168 | list_for_each(pos, &ioctl_list) { |
169 | struct atm_ioctl * ic = list_entry(pos, struct atm_ioctl, list); | 169 | struct atm_ioctl *ic = list_entry(pos, struct atm_ioctl, list); |
170 | if (try_module_get(ic->owner)) { | 170 | if (try_module_get(ic->owner)) { |
171 | error = ic->ioctl(sock, cmd, arg); | 171 | error = ic->ioctl(sock, cmd, arg); |
172 | module_put(ic->owner); | 172 | module_put(ic->owner); |
@@ -185,7 +185,6 @@ done: | |||
185 | return error; | 185 | return error; |
186 | } | 186 | } |
187 | 187 | ||
188 | |||
189 | int vcc_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg) | 188 | int vcc_ioctl(struct socket *sock, unsigned int cmd, unsigned long arg) |
190 | { | 189 | { |
191 | return do_vcc_ioctl(sock, cmd, arg, 0); | 190 | return do_vcc_ioctl(sock, cmd, arg, 0); |
@@ -288,8 +287,8 @@ static int do_atmif_sioc(struct socket *sock, unsigned int cmd, | |||
288 | sioc = compat_alloc_user_space(sizeof(*sioc)); | 287 | sioc = compat_alloc_user_space(sizeof(*sioc)); |
289 | sioc32 = compat_ptr(arg); | 288 | sioc32 = compat_ptr(arg); |
290 | 289 | ||
291 | if (copy_in_user(&sioc->number, &sioc32->number, 2 * sizeof(int)) | 290 | if (copy_in_user(&sioc->number, &sioc32->number, 2 * sizeof(int)) || |
292 | || get_user(data, &sioc32->arg)) | 291 | get_user(data, &sioc32->arg)) |
293 | return -EFAULT; | 292 | return -EFAULT; |
294 | datap = compat_ptr(data); | 293 | datap = compat_ptr(data); |
295 | if (put_user(datap, &sioc->arg)) | 294 | if (put_user(datap, &sioc->arg)) |