blob: 07ae0c06dfc34493688fd36f70c4dcb7285891f4 [file] [log] [blame]
Trent Jaegerd28d1e02005-12-13 23:12:40 -08001/*
2 * NSA Security-Enhanced Linux (SELinux) security module
3 *
4 * This file contains the SELinux XFRM hook function implementations.
5 *
6 * Authors: Serge Hallyn <sergeh@us.ibm.com>
7 * Trent Jaeger <jaegert@us.ibm.com>
8 *
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -07009 * Updated: Venkat Yekkirala <vyekkirala@TrustedCS.com>
10 *
11 * Granular IPSec Associations for use in MLS environments.
12 *
Trent Jaegerd28d1e02005-12-13 23:12:40 -080013 * Copyright (C) 2005 International Business Machines Corporation
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -070014 * Copyright (C) 2006 Trusted Computer Solutions, Inc.
Trent Jaegerd28d1e02005-12-13 23:12:40 -080015 *
16 * This program is free software; you can redistribute it and/or modify
17 * it under the terms of the GNU General Public License version 2,
18 * as published by the Free Software Foundation.
19 */
20
21/*
22 * USAGE:
23 * NOTES:
24 * 1. Make sure to enable the following options in your kernel config:
25 * CONFIG_SECURITY=y
26 * CONFIG_SECURITY_NETWORK=y
27 * CONFIG_SECURITY_NETWORK_XFRM=y
28 * CONFIG_SECURITY_SELINUX=m/y
29 * ISSUES:
30 * 1. Caching packets, so they are not dropped during negotiation
31 * 2. Emulating a reasonable SO_PEERSEC across machines
32 * 3. Testing addition of sk_policy's with security context via setsockopt
33 */
Trent Jaegerd28d1e02005-12-13 23:12:40 -080034#include <linux/kernel.h>
35#include <linux/init.h>
36#include <linux/security.h>
37#include <linux/types.h>
38#include <linux/netfilter.h>
39#include <linux/netfilter_ipv4.h>
40#include <linux/netfilter_ipv6.h>
Tejun Heo5a0e3ad2010-03-24 17:04:11 +090041#include <linux/slab.h>
Trent Jaegerd28d1e02005-12-13 23:12:40 -080042#include <linux/ip.h>
43#include <linux/tcp.h>
44#include <linux/skbuff.h>
45#include <linux/xfrm.h>
46#include <net/xfrm.h>
47#include <net/checksum.h>
48#include <net/udp.h>
Arun Sharma600634972011-07-26 16:09:06 -070049#include <linux/atomic.h>
Trent Jaegerd28d1e02005-12-13 23:12:40 -080050
51#include "avc.h"
52#include "objsec.h"
53#include "xfrm.h"
54
Paul Moored621d352008-01-29 08:43:36 -050055/* Labeled XFRM instance counter */
56atomic_t selinux_xfrm_refcount = ATOMIC_INIT(0);
Trent Jaegerd28d1e02005-12-13 23:12:40 -080057
58/*
59 * Returns true if an LSM/SELinux context
60 */
61static inline int selinux_authorizable_ctx(struct xfrm_sec_ctx *ctx)
62{
63 return (ctx &&
64 (ctx->ctx_doi == XFRM_SC_DOI_LSM) &&
65 (ctx->ctx_alg == XFRM_SC_ALG_SELINUX));
66}
67
68/*
69 * Returns true if the xfrm contains a security blob for SELinux
70 */
71static inline int selinux_authorizable_xfrm(struct xfrm_state *x)
72{
73 return selinux_authorizable_ctx(x->security);
74}
75
76/*
Paul Moore2e5aa862013-07-23 17:38:38 -040077 * Allocates a xfrm_sec_state and populates it using the supplied security
78 * xfrm_user_sec_ctx context.
79 */
80static int selinux_xfrm_alloc_user(struct xfrm_sec_ctx **ctxp,
81 struct xfrm_user_sec_ctx *uctx)
82{
83 int rc;
84 const struct task_security_struct *tsec = current_security();
85 struct xfrm_sec_ctx *ctx = NULL;
86 u32 str_len;
87
88 if (ctxp == NULL || uctx == NULL ||
89 uctx->ctx_doi != XFRM_SC_DOI_LSM ||
90 uctx->ctx_alg != XFRM_SC_ALG_SELINUX)
91 return -EINVAL;
92
93 str_len = uctx->ctx_len;
94 if (str_len >= PAGE_SIZE)
95 return -ENOMEM;
96
97 ctx = kmalloc(sizeof(*ctx) + str_len + 1, GFP_KERNEL);
98 if (!ctx)
99 return -ENOMEM;
100
101 ctx->ctx_doi = XFRM_SC_DOI_LSM;
102 ctx->ctx_alg = XFRM_SC_ALG_SELINUX;
103 ctx->ctx_len = str_len;
104 memcpy(ctx->ctx_str, &uctx[1], str_len);
105 ctx->ctx_str[str_len] = '\0';
106 rc = security_context_to_sid(ctx->ctx_str, str_len, &ctx->ctx_sid);
107 if (rc)
108 goto err;
109
110 rc = avc_has_perm(tsec->sid, ctx->ctx_sid,
111 SECCLASS_ASSOCIATION, ASSOCIATION__SETCONTEXT, NULL);
112 if (rc)
113 goto err;
114
115 *ctxp = ctx;
116 atomic_inc(&selinux_xfrm_refcount);
117 return 0;
118
119err:
120 kfree(ctx);
121 return rc;
122}
123
124/*
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700125 * LSM hook implementation that authorizes that a flow can use
126 * a xfrm policy rule.
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800127 */
Paul Moore03e1ad72008-04-12 19:07:52 -0700128int selinux_xfrm_policy_lookup(struct xfrm_sec_ctx *ctx, u32 fl_secid, u8 dir)
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800129{
Venkat Yekkirala5b368e62006-10-05 15:42:18 -0500130 int rc;
131 u32 sel_sid;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800132
133 /* Context sid is either set to label or ANY_ASSOC */
Paul Moore03e1ad72008-04-12 19:07:52 -0700134 if (ctx) {
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800135 if (!selinux_authorizable_ctx(ctx))
136 return -EINVAL;
137
138 sel_sid = ctx->ctx_sid;
Paul Moore03e1ad72008-04-12 19:07:52 -0700139 } else
Venkat Yekkirala5b368e62006-10-05 15:42:18 -0500140 /*
141 * All flows should be treated as polmatch'ing an
142 * otherwise applicable "non-labeled" policy. This
143 * would prevent inadvertent "leaks".
144 */
145 return 0;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800146
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700147 rc = avc_has_perm(fl_secid, sel_sid, SECCLASS_ASSOCIATION,
148 ASSOCIATION__POLMATCH,
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800149 NULL);
150
Venkat Yekkirala5b368e62006-10-05 15:42:18 -0500151 if (rc == -EACCES)
Paul Moore03e1ad72008-04-12 19:07:52 -0700152 return -ESRCH;
Venkat Yekkirala5b368e62006-10-05 15:42:18 -0500153
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800154 return rc;
155}
156
157/*
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700158 * LSM hook implementation that authorizes that a state matches
159 * the given policy, flow combo.
160 */
161
162int selinux_xfrm_state_pol_flow_match(struct xfrm_state *x, struct xfrm_policy *xp,
David S. Millere33f7702011-02-22 18:13:15 -0800163 const struct flowi *fl)
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700164{
165 u32 state_sid;
Venkat Yekkirala67f83cb2006-11-08 17:04:26 -0600166 int rc;
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700167
Venkat Yekkirala67f83cb2006-11-08 17:04:26 -0600168 if (!xp->security)
Venkat Yekkirala5b368e62006-10-05 15:42:18 -0500169 if (x->security)
170 /* unlabeled policy and labeled SA can't match */
171 return 0;
172 else
173 /* unlabeled policy and unlabeled SA match all flows */
174 return 1;
Venkat Yekkirala67f83cb2006-11-08 17:04:26 -0600175 else
176 if (!x->security)
177 /* unlabeled SA and labeled policy can't match */
178 return 0;
179 else
180 if (!selinux_authorizable_xfrm(x))
181 /* Not a SELinux-labeled SA */
182 return 0;
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700183
Venkat Yekkirala67f83cb2006-11-08 17:04:26 -0600184 state_sid = x->security->ctx_sid;
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700185
David S. Miller1d28f422011-03-12 00:29:39 -0500186 if (fl->flowi_secid != state_sid)
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700187 return 0;
188
David S. Miller1d28f422011-03-12 00:29:39 -0500189 rc = avc_has_perm(fl->flowi_secid, state_sid, SECCLASS_ASSOCIATION,
Venkat Yekkirala5b368e62006-10-05 15:42:18 -0500190 ASSOCIATION__SENDTO,
191 NULL)? 0:1;
192
Venkat Yekkirala67f83cb2006-11-08 17:04:26 -0600193 /*
194 * We don't need a separate SA Vs. policy polmatch check
195 * since the SA is now of the same label as the flow and
196 * a flow Vs. policy polmatch check had already happened
197 * in selinux_xfrm_policy_lookup() above.
198 */
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700199
200 return rc;
201}
202
203/*
Venkat Yekkirala6b877692006-11-08 17:04:09 -0600204 * LSM hook implementation that checks and/or returns the xfrm sid for the
205 * incoming packet.
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700206 */
207
Venkat Yekkiralabeb8d132006-08-04 23:12:42 -0700208int selinux_xfrm_decode_session(struct sk_buff *skb, u32 *sid, int ckall)
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700209{
210 struct sec_path *sp;
211
Venkat Yekkiralabeb8d132006-08-04 23:12:42 -0700212 *sid = SECSID_NULL;
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700213
214 if (skb == NULL)
215 return 0;
216
217 sp = skb->sp;
218 if (sp) {
219 int i, sid_set = 0;
220
221 for (i = sp->len-1; i >= 0; i--) {
222 struct xfrm_state *x = sp->xvec[i];
223 if (selinux_authorizable_xfrm(x)) {
224 struct xfrm_sec_ctx *ctx = x->security;
225
226 if (!sid_set) {
Venkat Yekkiralabeb8d132006-08-04 23:12:42 -0700227 *sid = ctx->ctx_sid;
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700228 sid_set = 1;
Venkat Yekkiralabeb8d132006-08-04 23:12:42 -0700229
230 if (!ckall)
231 break;
Eric Paris3c1c88a2008-04-18 17:38:27 -0400232 } else if (*sid != ctx->ctx_sid)
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700233 return -EINVAL;
234 }
235 }
236 }
237
238 return 0;
239}
240
241/*
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800242 * LSM hook implementation that allocs and transfers uctx spec to
243 * xfrm_policy.
244 */
Paul Moore03e1ad72008-04-12 19:07:52 -0700245int selinux_xfrm_policy_alloc(struct xfrm_sec_ctx **ctxp,
246 struct xfrm_user_sec_ctx *uctx)
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800247{
Paul Moore2e5aa862013-07-23 17:38:38 -0400248 return selinux_xfrm_alloc_user(ctxp, uctx);
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800249}
250
251
252/*
253 * LSM hook implementation that copies security data structure from old to
254 * new for policy cloning.
255 */
Paul Moore03e1ad72008-04-12 19:07:52 -0700256int selinux_xfrm_policy_clone(struct xfrm_sec_ctx *old_ctx,
257 struct xfrm_sec_ctx **new_ctxp)
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800258{
Paul Moore03e1ad72008-04-12 19:07:52 -0700259 struct xfrm_sec_ctx *new_ctx;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800260
261 if (old_ctx) {
Paul Moore03e1ad72008-04-12 19:07:52 -0700262 new_ctx = kmalloc(sizeof(*old_ctx) + old_ctx->ctx_len,
Dan Carpenter45024032013-03-16 12:48:11 +0300263 GFP_ATOMIC);
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800264 if (!new_ctx)
265 return -ENOMEM;
266
267 memcpy(new_ctx, old_ctx, sizeof(*new_ctx));
268 memcpy(new_ctx->ctx_str, old_ctx->ctx_str, new_ctx->ctx_len);
Paul Mooree4e85362013-05-29 07:36:32 +0000269 atomic_inc(&selinux_xfrm_refcount);
Paul Moore03e1ad72008-04-12 19:07:52 -0700270 *new_ctxp = new_ctx;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800271 }
272 return 0;
273}
274
275/*
Paul Moore03e1ad72008-04-12 19:07:52 -0700276 * LSM hook implementation that frees xfrm_sec_ctx security information.
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800277 */
Paul Moore03e1ad72008-04-12 19:07:52 -0700278void selinux_xfrm_policy_free(struct xfrm_sec_ctx *ctx)
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800279{
Paul Mooree4e85362013-05-29 07:36:32 +0000280 atomic_dec(&selinux_xfrm_refcount);
Eric Paris3c1c88a2008-04-18 17:38:27 -0400281 kfree(ctx);
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800282}
283
284/*
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700285 * LSM hook implementation that authorizes deletion of labeled policies.
286 */
Paul Moore03e1ad72008-04-12 19:07:52 -0700287int selinux_xfrm_policy_delete(struct xfrm_sec_ctx *ctx)
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700288{
David Howells86a264a2008-11-14 10:39:18 +1100289 const struct task_security_struct *tsec = current_security();
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700290
Paul Mooree4e85362013-05-29 07:36:32 +0000291 if (!ctx)
292 return 0;
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700293
Paul Mooree4e85362013-05-29 07:36:32 +0000294 return avc_has_perm(tsec->sid, ctx->ctx_sid,
295 SECCLASS_ASSOCIATION, ASSOCIATION__SETCONTEXT,
296 NULL);
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700297}
298
299/*
Paul Moore2e5aa862013-07-23 17:38:38 -0400300 * LSM hook implementation that allocates a xfrm_sec_state, populates it using
301 * the supplied security context, and assigns it to the xfrm_state.
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800302 */
Paul Moore2e5aa862013-07-23 17:38:38 -0400303int selinux_xfrm_state_alloc(struct xfrm_state *x,
304 struct xfrm_user_sec_ctx *uctx)
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800305{
Paul Moore2e5aa862013-07-23 17:38:38 -0400306 return selinux_xfrm_alloc_user(&x->security, uctx);
307}
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800308
Paul Moore2e5aa862013-07-23 17:38:38 -0400309/*
310 * LSM hook implementation that allocates a xfrm_sec_state and populates based
311 * on a secid.
312 */
313int selinux_xfrm_state_alloc_acquire(struct xfrm_state *x,
314 struct xfrm_sec_ctx *polsec, u32 secid)
315{
316 int rc;
317 struct xfrm_sec_ctx *ctx;
318 char *ctx_str = NULL;
319 int str_len;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800320
Paul Moore2e5aa862013-07-23 17:38:38 -0400321 if (!polsec)
322 return 0;
323
324 if (secid == 0)
325 return -EINVAL;
326
327 rc = security_sid_to_context(secid, &ctx_str, &str_len);
328 if (rc)
329 return rc;
330
331 ctx = kmalloc(sizeof(*ctx) + str_len, GFP_ATOMIC);
332 if (!ctx)
333 return -ENOMEM;
334
335 ctx->ctx_doi = XFRM_SC_DOI_LSM;
336 ctx->ctx_alg = XFRM_SC_ALG_SELINUX;
337 ctx->ctx_sid = secid;
338 ctx->ctx_len = str_len;
339 memcpy(ctx->ctx_str, ctx_str, str_len);
340 kfree(ctx_str);
341
342 x->security = ctx;
343 atomic_inc(&selinux_xfrm_refcount);
344 return 0;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800345}
346
347/*
348 * LSM hook implementation that frees xfrm_state security information.
349 */
350void selinux_xfrm_state_free(struct xfrm_state *x)
351{
Paul Mooree4e85362013-05-29 07:36:32 +0000352 atomic_dec(&selinux_xfrm_refcount);
353 kfree(x->security);
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800354}
355
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700356 /*
357 * LSM hook implementation that authorizes deletion of labeled SAs.
358 */
359int selinux_xfrm_state_delete(struct xfrm_state *x)
360{
David Howells86a264a2008-11-14 10:39:18 +1100361 const struct task_security_struct *tsec = current_security();
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700362 struct xfrm_sec_ctx *ctx = x->security;
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700363
Paul Mooree4e85362013-05-29 07:36:32 +0000364 if (!ctx)
365 return 0;
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700366
Paul Mooree4e85362013-05-29 07:36:32 +0000367 return avc_has_perm(tsec->sid, ctx->ctx_sid,
368 SECCLASS_ASSOCIATION, ASSOCIATION__SETCONTEXT,
369 NULL);
Catherine Zhangc8c05a82006-06-08 23:39:49 -0700370}
371
Catherine Zhang2c7946a2006-03-20 22:41:23 -0800372/*
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800373 * LSM hook that controls access to unlabelled packets. If
374 * a xfrm_state is authorizable (defined by macro) then it was
375 * already authorized by the IPSec process. If not, then
376 * we need to check for unlabelled access since this may not have
377 * gone thru the IPSec process.
378 */
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700379int selinux_xfrm_sock_rcv_skb(u32 isec_sid, struct sk_buff *skb,
Thomas Liu2bf49692009-07-14 12:14:09 -0400380 struct common_audit_data *ad)
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800381{
382 int i, rc = 0;
383 struct sec_path *sp;
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700384 u32 sel_sid = SECINITSID_UNLABELED;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800385
386 sp = skb->sp;
387
388 if (sp) {
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800389 for (i = 0; i < sp->len; i++) {
Dave Jones67644722006-04-02 23:34:19 -0700390 struct xfrm_state *x = sp->xvec[i];
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800391
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700392 if (x && selinux_authorizable_xfrm(x)) {
393 struct xfrm_sec_ctx *ctx = x->security;
394 sel_sid = ctx->ctx_sid;
395 break;
396 }
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800397 }
398 }
399
Venkat Yekkirala67f83cb2006-11-08 17:04:26 -0600400 /*
401 * This check even when there's no association involved is
402 * intended, according to Trent Jaeger, to make sure a
403 * process can't engage in non-ipsec communication unless
404 * explicitly allowed by policy.
405 */
406
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700407 rc = avc_has_perm(isec_sid, sel_sid, SECCLASS_ASSOCIATION,
408 ASSOCIATION__RECVFROM, ad);
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800409
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800410 return rc;
411}
412
413/*
414 * POSTROUTE_LAST hook's XFRM processing:
415 * If we have no security association, then we need to determine
416 * whether the socket is allowed to send to an unlabelled destination.
417 * If we do have a authorizable security association, then it has already been
Venkat Yekkirala67f83cb2006-11-08 17:04:26 -0600418 * checked in the selinux_xfrm_state_pol_flow_match hook above.
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800419 */
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700420int selinux_xfrm_postroute_last(u32 isec_sid, struct sk_buff *skb,
Thomas Liu2bf49692009-07-14 12:14:09 -0400421 struct common_audit_data *ad, u8 proto)
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800422{
423 struct dst_entry *dst;
424 int rc = 0;
425
Eric Dumazetadf30902009-06-02 05:19:30 +0000426 dst = skb_dst(skb);
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800427
428 if (dst) {
429 struct dst_entry *dst_test;
430
Stephen Hemmingerc80544d2007-10-18 03:07:05 -0700431 for (dst_test = dst; dst_test != NULL;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800432 dst_test = dst_test->child) {
433 struct xfrm_state *x = dst_test->xfrm;
434
435 if (x && selinux_authorizable_xfrm(x))
James Morris4e5ab4c2006-06-09 00:33:33 -0700436 goto out;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800437 }
438 }
439
Venkat Yekkirala67f83cb2006-11-08 17:04:26 -0600440 switch (proto) {
441 case IPPROTO_AH:
442 case IPPROTO_ESP:
443 case IPPROTO_COMP:
444 /*
445 * We should have already seen this packet once before
446 * it underwent xfrm(s). No need to subject it to the
447 * unlabeled check.
448 */
449 goto out;
450 default:
451 break;
452 }
453
454 /*
455 * This check even when there's no association involved is
456 * intended, according to Trent Jaeger, to make sure a
457 * process can't engage in non-ipsec communication unless
458 * explicitly allowed by policy.
459 */
460
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800461 rc = avc_has_perm(isec_sid, SECINITSID_UNLABELED, SECCLASS_ASSOCIATION,
Venkat Yekkiralae0d1caa2006-07-24 23:29:07 -0700462 ASSOCIATION__SENDTO, ad);
James Morris4e5ab4c2006-06-09 00:33:33 -0700463out:
464 return rc;
Trent Jaegerd28d1e02005-12-13 23:12:40 -0800465}