On Wed, Sep 11, 2013 at 01:47:44AM -0700, Tyler Hicks wrote:
> From: John Johansen <[email protected]>
> 
> This conversion is nothing more than what is required to get it to
> compile. Further improvements will come as the code is refactored.
> 
> Unfortunately due to C++ not supporting designated initializers, the auto
> generation of af names needed to be reworked, and "netlink" and "unix"
> domain socket keywords leaked in. Since these where going to be added in
> separate patches I have not bothered to do the extra work to replace them
> with a temporary place holder.
> 
> Signed-off-by: John Johansen <[email protected]>
> [tyhicks: merged with dbus changes and memory leak fixes]
> Signed-off-by: Tyler Hicks <[email protected]>
> Acked-by: Seth Arnold <[email protected]>

This doesn't apply to trunk due to the conflicting DUP_STRING changes in
parser_misc.c. The following patch snippet:

> diff --git a/parser/parser_misc.c b/parser/parser_misc.c
> index 24dd53d..10c41ec 100644
> --- a/parser/parser_misc.c
> +++ b/parser/parser_misc.c
> @@ -841,7 +841,7 @@ struct cod_entry *copy_cod_entry(struct cod_entry *orig)
>       if (!entry)
>               return NULL;
>  
> -     entry->namespace = orig->namespace ? strdup(orig->namespace) : NULL;
> +     entry->ns = orig->ns ? strdup(orig->ns) : NULL;
>       entry->name = strdup(orig->name);
>       entry->link_name = orig->link_name ? strdup(orig->link_name) : NULL;
>       entry->mode = orig->mode;

needs to be replaced with the following:

@@ -841,7 +841,7 @@ struct cod_entry *copy_cod_entry(struct cod_entry *orig)
        if (!entry)
                return NULL;
 
-       DUP_STRING(orig, entry, namespace, err);
+       DUP_STRING(orig, entry, ns, err);
        DUP_STRING(orig, entry, name, err);
        DUP_STRING(orig, entry, link_name, err);
        entry->mode = orig->mode;

for the patch to apply.
-- 
Steve Beattie
<[email protected]>
http://NxNW.org/~steve/

Attachment: signature.asc
Description: Digital signature

-- 
AppArmor mailing list
[email protected]
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/apparmor

Reply via email to