On 10/14/2011 04:12 PM, Roopa Prabhu wrote:

From: Roopa Prabhu<roprabhu@cisco.com>

getPhysfnDev was conditionally allocating memory for physfndev on success. Recently added unconditional free in the caller of getPhysfnDev can result in a crash. So fix getPhysfnDev to unconditionally allocate memory for physfndev on success

Signed-off-by: Roopa Prabhu<roprabhu@cisco.com> Acked-by: Eric Blake<eblake@redhat.com> --- src/util/macvtap.c | 6 +++++- 1 files changed, 5 insertions(+), 1 deletions(-)

Already pushed. -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org