[patch] add a trivial xml parser for the setuid addon
Havoc Pennington
hp at redhat.com
Mon Jun 18 07:56:32 PDT 2007
Hi,
My main question here is whether I can think of a way to refactor
config-parser.c itself to nuke the dependencies. I will look at that a
bit later.
In any case we'll need tests for the trivial config parser, including
the OOM handling, and it should be easy to do them.
Havoc
Richard Hughes wrote:
> This patch adds a trivially small (i.e. nothing clever, no selinux, no
> policy, no included files, nothing remotely risky) xml config file
> parser. It also introduces config-parser-common.{h|h} files to abstract
> out the common element definitions and conversion code. If this is okay,
> then I'll migrate config-parser.c to use the config-parser-common.c
> definitions, and then we get little duplication between the trivial and
> the proper parser.
>
> Not using the config-parser.c file in the setuid addon means we can drop
> a metric _TON_ of dependencies, making the addon much smaller and much
> easier to audit.
>
More information about the dbus
mailing list