[libvirt] [PATCH 07/11] Remove bogus port handling code in virsh

Daniel P. Berrange berrange at redhat.com
Tue Nov 23 14:27:29 UTC 2010


On Fri, Nov 19, 2010 at 11:23:00AM -0700, Eric Blake wrote:
> On 11/12/2010 09:22 AM, Daniel P. Berrange wrote:
> > The "find-storage-pool-sources-as" command takes two arguments,
> > a hostname and a port number. For some reason the code would
> > also then look for a port number appended to the hostname
> > string by searching for ':'. This totally breaks if the user
> > gives an IPv6 address, and is redundant, since you can already
> > provide a port as a separate argument
> > 
> > * tools/virsh.c: Remove bogus port number handling code
> > ---
> >  tools/virsh.c |   11 ++---------
> >  1 files changed, 2 insertions(+), 9 deletions(-)
> > 


> 
> >          virBuffer buf = VIR_BUFFER_INITIALIZER;
> >          virBufferAddLit(&buf, "<source>\n");
> >          virBufferVSprintf(&buf, "  <host name='%.*s'",(int)hostlen, host);
> 
> However, given that we are no longer computing hostlen as anything other
> than strlen(host), I would recommend that you respin this patch to
> completely get rid of hostlen, and use %s instead of %.*s.

Yep I'll kill that.

Daniel




More information about the libvir-list mailing list