Re: [busybox/utility.c:recursiveAction] ? - followLinks, stat, lstat
On Mon Jan 24, 2000 at 11:05:59PM -0800, Karl M. Hegbloom wrote:
>
> % cvs diff -w utility.c
> Index: utility.c
> ===================================================================
> RCS file: /cvs/debian-boot/boot-floppies/utilities/busybox/utility.c,v
> retrieving revision 1.30
> diff -u -w -r1.30 utility.c
> --- utility.c 2000/01/23 02:13:50 1.30
> +++ utility.c 2000/01/25 07:00:37
> @@ -406,7 +406,6 @@
> else
> status = lstat(fileName, &statbuf);
>
> - status = lstat(fileName, &statbuf);
> if (status < 0) {
> perror(fileName);
> return (FALSE);
Yikes! Now I understand. This is certainly something
leftover in the code, and (as you noticed) is obviously
wrong. Of course, the chance of my seeing a bug is inversely
proportional to the number of times I look at it... In
your first email, I didn't even _see_ the last line. I guess
it sortof looked like a sig.
Anyways, thank-you, and good spotting. I Will apply this fix,
-Erik
--
Erik B. Andersen Web: http://www.xmission.com/~andersen/
email: andersee@debian.org
--This message was written using 73% post-consumer electrons--
Reply to: