On Tue, Dec 02, 2014 at 02:19:49PM +0000, Ian Campbell wrote: > On Mon, 2014-12-01 at 11:31 +0000, Wei Liu wrote: > > Free strings returned by libxl_basename after used. > > > > Signed-off-by: Wei Liu <wei.l...@citrix.com> > > Cc: Ian Campbell <ian.campb...@citrix.com> > > Cc: Ian Jackson <ian.jack...@eu.citrix.com> > > --- > > tools/libxl/xl_cmdimpl.c | 8 ++++++-- > > 1 file changed, 6 insertions(+), 2 deletions(-) > > > > diff --git a/tools/libxl/xl_cmdimpl.c b/tools/libxl/xl_cmdimpl.c > > index 0e754e7..fe3034f 100644 > > --- a/tools/libxl/xl_cmdimpl.c > > +++ b/tools/libxl/xl_cmdimpl.c > > @@ -920,6 +920,7 @@ static void parse_config_data(const char *config_source, > > int pci_permissive = 0; > > int pci_seize = 0; > > int i, e; > > + char *basename; > > For some reason on arm32 (only) this causes: > xl_cmdimpl.c: In function ‘parse_config_data’: > xl_cmdimpl.c:929:11: error: declaration of ‘basename’ shadows a global > declaration [-Werror=shadow] > > basename(3) is defined by libgen.h which we don't include, so I suspect > this is a libc issue on armhf (unless Ian has any other ideas?). > > How about I s/basename/kernel_basename/ or some such on commit? >
That's OK. Wei. > Ian. _______________________________________________ Xen-devel mailing list Xen-devel@lists.xen.org http://lists.xen.org/xen-devel