On Sun, Mar 20, 2011 at 4:21 PM, Denys Vlasenko <[email protected]> wrote: > On Tuesday 15 March 2011 03:07, [email protected] wrote: >> Hi, >> >> I have just noticed a recent wget bug. Busybox wget doesn't parse the >> URL correctly in some circumstances; long guessed output filenames && >> HTTP redirection. >> >> e.g. >> $./busybox wget >> http://cdimage.debian.org/debian-cd/6.0.0/kfreebsd-i386/iso-cd/debian-6.0.0-kfreebsd-i386-CD-1.iso >> Connecting to cdimage.debian.org (130.239.18.173:80) >> Connecting to hammurabi.acc.umu.se (130.239.18.165:80) >> wget: can't open '': No such file or directory > > Can't reproduce. What causes this on your machine? > >
Like Cristian says, now there's a 404 error. Debian 6.0.1 was released yesterday... the return was: wget: can't open '': No such file or directory wget tried to output to "". If I try with other url (current iso of debian), wget tries to output to: ebian-6.0.1-kfreebsd-i386-CD-1.iso (first letter of filename is missing) ./busybox wget http://cdimage.debian.org/debian-cd/6.0.1/kfreebsd-i386/iso-cd/debian-6.0.1-kfreebsd-i386-CD-1.iso Connecting to cdimage.debian.org (130.239.18.163:80) Connecting to caesar.acc.umu.se (130.239.18.142:80) ebian-6.0.1-kfreebsd 1% | | 7769k 1:34:38 ETA ^----------------------------- d is missing [...] > I don't want to leak target.allocated in this case. > -- > vda > Ok.I did not understand why, but removing the free fixed the problem for me. ¿? I'm using 1.19.git on linux x86 (glibc) Wouldn't be better: else { free(target.allocated); parse_url(str, &target); if (!use_proxy) { free(server.allocated); server.allocated = NULL; ... Regards, -- Pere _______________________________________________ busybox mailing list [email protected] http://lists.busybox.net/mailman/listinfo/busybox
