On 02/15/2012 03:08 AM, Phantomcircuit wrote:

# libvirtd --version libvirtd (libvirt) 0.9.8 # which ebtables /sbin/ebtables

2012-02-15 09:40:59.083+0000: 17675: error : ebtablesCreateRuleInstance:1943 : internal error cannot create rule since ebtables tool is missing. 2012-02-15 09:40:59.107+0000: 17675: error : virNetDevGetIndex:656 : Unable to get index for interface vnet0: No such device

ebtables_cmd_path = virFindFileInPath("ebtables");

I can't figure out why ebtables_cmd_path is NULL when the call to ebtablesCreateRuleInstance is made.

Does the $PATH in use by libvirtd include /sbin by default on your system? Not all systems put /sbin on the path. Hmm, looks like we have a couple bugs: 1. configure.ac sets EBTABLES_PATH, but only adds /usr/sbin, and not /sbin, to the PATH used to find it, but then defaults to /sbin/ebtables if the PATH search failed. This is fragile (the fallback value should be tested for existence before hard-coding it into the executable). 2. src/util/ebtables.c uses EBTABLES_PATH, while src/nwfilter/nwfilter_ebiptables_driver.c only uses a PATH lookup of the raw "ebtables". These should be consolidated to use the same program (that is, if the user did ./configure EBTABLES=/path/to/alternate, we should prefer that name rather than half-and-half). Patches welcome. -- Eric Blake eblake@redhat.com +1-919-301-3266 Libvirt virtualization library http://libvirt.org