From mboxrd@z Thu Jan 1 00:00:00 1970 From: Steve Grubb Subject: Re: [PATCH] auvirt: Remove workaround for VM name searching Date: Sat, 11 Feb 2012 09:47:56 -0500 Message-ID: <201202110947.57134.sgrubb@redhat.com> References: <1328815120-6691-1-git-send-email-mhcerri@linux.vnet.ibm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Return-path: In-Reply-To: <1328815120-6691-1-git-send-email-mhcerri@linux.vnet.ibm.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Sender: linux-audit-bounces@redhat.com Errors-To: linux-audit-bounces@redhat.com To: linux-audit@redhat.com Cc: gcwilson@us.ibm.com, bryntcor@us.ibm.com List-Id: linux-audit@redhat.com On Thursday, February 09, 2012 02:18:40 PM Marcelo Cerri wrote: > With this patch, the workaround for creating the search criteria is removed > and escaped fields are properly retrieved. Applied, but one comment down below... > The unexpected single quote at the beginning of MAC addresses is fixed by a > patch in libvirt: > > https://www.redhat.com/archives/libvir-list/2012-February/msg00502.html > --- > tools/auvirt/auvirt.c | 39 +++++++++++++++------------------------ > 1 files changed, 15 insertions(+), 24 deletions(-) > > diff --git a/tools/auvirt/auvirt.c b/tools/auvirt/auvirt.c > index c04780a..a89b097 100644 > --- a/tools/auvirt/auvirt.c > +++ b/tools/auvirt/auvirt.c > @@ -781,7 +768,9 @@ int process_resource_event(auparse_state_t *au) > } > } else if (strcmp("cgroup", res_type) == 0) { > auparse_first_record(au); > - const char *cgroup = auparse_find_field(au, "cgroup"); > + const char *cgroup = NULL; > + if (auparse_find_field(au, "cgroup")) > + cgroup = auparse_interpret_field(au); > rc += add_resource(au, uuid, uid, time, name, success, reason, > res_type, cgroup); So, if cgroup is NULL here, does anything go boom later? -Steve