[U-Boot] [PATCH] Avoid using GNU basename

Wolfgang Denk wd at denx.de
Fri Mar 2 08:16:10 CET 2012


Dear Keith Mok,

In message <CAHjoi4eR+n=qQ8vZAKwwgLiPSFfrBWHRHwYNvfJn4y0F9EqO7g at mail.gmail.com> you wrote:
> There is no GNU basename support in MacOS.
> Use generic POSIX basename defined in libgen.h instead.
...
>         int fp, ep;
>         const char *prg;
> +       char *prog_pathname;
> 
> -       prg = basename(argv[0]);
> +       prog_pathname = strdup(argv[0]);
> +       prg = basename(prog_pathname);

free() missing.  Actually the strdup should not be needed at all,
as we don't use argv[0] after that - at least we shouldn't.  The
remaining "usage(argv[0]);" should be fixed instead.

Besides - I agree with Mike's comment how your MacOS issue should be
fixed.

Best regards,

Wolfgang Denk

-- 
DENX Software Engineering GmbH,     MD: Wolfgang Denk & Detlev Zundel
HRB 165235 Munich, Office: Kirchenstr.5, D-82194 Groebenzell, Germany
Phone: (+49)-8142-66989-10 Fax: (+49)-8142-66989-80 Email: wd at denx.de
Drun'? 'm not drun'! You woudn' dare call m' drun' if I was sober!
                                     - Terry Pratchett, _Men at Arms_


More information about the U-Boot mailing list