On 11/12/2013 03:36 PM, Daniel P. Berrange wrote: > On Tue, Nov 12, 2013 at 01:52:10PM +0100, Ján Tomko wrote: >> When opening a new connection to the driver, nwfilterOpen >> only succeeds if the driverState has been allocated. >> >> Move the privilege check in driver initialization before >> the state allocation to disable the driver. >> >> This changes the nwfilter-define error from: >> error: cannot create config directory (null): Bad address >> To: >> this function is not supported by the connection driver: >> virNWFilterDefineXML >> >> https://bugzilla.redhat.com/show_bug.cgi?id=1029266 >> --- >> v1: https://www.redhat.com/archives/libvir-list/2013-November/msg00368.html >> v2: https://www.redhat.com/archives/libvir-list/2013-November/msg00374.html >> forbid everything instead of just virNWFilterDefineXML >> v3: cut a pair of holes into my brown paper bag >> >> src/nwfilter/nwfilter_driver.c | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> > > ACK > > > Suitable for -maint branches too. > I've pushed it to master and v0.10.2, v1.0.5 and v1.1.0 ~ v1.1.3 -maint branches. Thank you for the reviews. Jan
Attachment:
signature.asc
Description: OpenPGP digital signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list