On Monday 09 February 2009 04:37:14 pm Eric Paris wrote: > If CONFIG_BUG is not set places in the code where BUG is called will not > end execution. Look for places where this might result is system problems > and keep the system running. > > Signed-off-by: Eric Paris <eparis@xxxxxxxxxx> ... > diff --git a/security/selinux/netnode.c b/security/selinux/netnode.c > index 7100072..e002d5a 100644 > --- a/security/selinux/netnode.c > +++ b/security/selinux/netnode.c ... > @@ -240,6 +242,7 @@ static int sel_netnode_sid_slow(void *addr, u16 family, > u32 *sid) break; > default: > BUG(); > + ret = -EINVAL; > } > if (ret != 0) > goto out; I'm just being picky here, but how about EPROTONOSUPPORT instead of EINVAL here? Otherwise it looks good to me. -- paul moore linux @ hp -- This message was distributed to subscribers of the selinux mailing list. If you no longer wish to subscribe, send mail to majordomo@xxxxxxxxxxxxx with the words "unsubscribe selinux" without quotes as the message.
