[apparmor] [PATCH] clean up many warnings

Seth Arnold seth.arnold at gmail.com
Mon Jul 26 09:34:56 BST 2010


On Sun, Jul 25, 2010 at 6:59 PM, Kees Cook <kees at ubuntu.com> wrote:
>  .PHONY: checkin
>  checkin:
>        if cvs -q up -d | grep -q "^\?" ; then echo "Hey! You have" \

Might as well get rid of our ancient cvs magic while you're here. (It
was clever magic, which was why I didn't remove it when we moved to
subversion.)

> -static struct keyword_table address_family[] = {
> -/*     {"unix",        AF_UNIX},
> -       {"local",       AF_LOCAL},      */
> -       {"inet",        AF_INET},
> -/*     {"ax25",        AF_AX25},
> -       {"ipx",         AF_IPX},
> -       {"appletalk",   AF_APPLETALK},
> -       {"netrom",      AF_NETROM},
> -       {"bridge",      AF_BRIDGE},
> -       {"atmpvc",      AF_ATMPVC},
> -       {"x25",         AF_X25}, */
> -       {"inet6",       AF_INET6},
> -/*     {"rose",        AF_ROSE},
> -       {"decnet",      AF_DECnet},
> -       {"netbeui",     AF_NETBEUI},
> -       {"security",    AF_SECURITY},
> -       {"key",         AF_KEY},
> -       {"netlink",     AF_NETLINK},
> -       {"route",       AF_ROUTE},
> -       {"packet",      AF_PACKET},
> -       {"ash",         AF_ASH},
> -       {"econet",      AF_ECONET},
> -       {"atmsvc",      AF_ATMSVC},
> -       {"sna",         AF_SNA},
> -       {"irda",        AF_IRDA},
> -       {"pppox",       AF_PPPOX},
> -       {"wanpipe",     AF_WANPIPE},
> -       {"llc",         AF_LLC},
> -       {"tipc",        AF_TIPC},
> -       {"bluetooth",   AF_BLUETOOTH},
> -       {"iucv",        AF_IUCV},
> -       {"rxrpc",       AF_RXRPC}, */
> -       /* terminate */
> -       {NULL, 0}
> -};

Seems a shame to get rid of such a beautiful table. (No idea if it was
still used, but it's a neat table. :)

Thanks!



More information about the AppArmor mailing list