Re: [libvirt] [PATCH] Remove hard dependency on DMI

On 03/04/2010 07:49 PM, Ed Swierk wrote: > On Thu, Mar 4, 2010 at 10:31 AM, Dave Allan<dallan(a)redhat.com> wrote: >> I reworked udevSetupSystemDev into this style (which also fixes the >> bug you >> pointed out that it didn't properly free resources on error). The patch >> also makes failure to find DMI data non-fatal. > > Your patch works fine. Thanks. > > Acked-by: Ed Swierk<eswierk(a)aristanetworks.com> Thanks for the ACK; anybody else willing to provide the second?

>From 23526641083527139548c68b4637bae8350d2f98 Mon Sep 17 00:00:00 2001 From: David Allan <dallan(a)redhat.com&gt; Date: Thu, 4 Mar 2010 13:17:24 -0500 Subject: [PATCH 1/1] Free resources on error in udev startup * The udev driver didn't properly free resources that it allocates when setting up the 'computer' device in the error case.