[libvirt] [firewalld PATCHv3] firewalld PATCH v3
Eric Blake
eblake at redhat.com
Wed Aug 15 15:18:36 UTC 2012
On 08/14/2012 12:59 PM, Thomas Woerner wrote:
> * configura.ac, spec file: firewalld now defaults to enabled, depends on
> dbus
> * fixed comment for with_firewalld define
> * bridge_driver, nwfilter_driver: new dbus filters to get FirewallD1.Reloaded
> signal and DBus.NameOwnerChanged on org.fedoraproject.FirewallD1
> * iptables, ebtables, nwfilter_ebiptables_driver: use firewall-cmd direct
> passthrough interface
> * spec file changed as requested
In spite of Dan's ACK,
> +++ b/src/util/ebtables.c
> @@ -176,11 +176,34 @@ ebtablesAddRemoveRule(ebtRules *rules, int action, const char *arg, ...)
> const char *s;
> int n, command_idx;
>
> +#if HAVE_FIREWALLD
> + int ret;
> + char *firewall_cmd_path = NULL;
> + virCommandPtr cmd = NULL;
> +
> + firewall_cmd_path = virFindFileInPath("firewall-cmd");
this is rather inefficient - doing a PATH lookup for every call to
ebtablesAddRemoveRule. Can we do the lookup just once and remember the
result?
--
Eric Blake eblake at redhat.com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 620 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20120815/27417590/attachment-0001.sig>
More information about the libvir-list
mailing list