On Tue, Dec 02, 2014 at 02:19:49PM +, Ian Campbell wrote:
> On Mon, 2014-12-01 at 11:31 +, Wei Liu wrote:
> > Free strings returned by libxl_basename after used.
> >
> > Signed-off-by: Wei Liu
> > Cc: Ian Campbell
> > Cc: Ian Jackson
> > ---
> > 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