snapshot-20080618 'ls' inverted return status

[George Boudreau]

Hi,
  While building busybox-20080618, with defconfig, I stumbled upon a
problem with 'ls'.

./busybox ls; echo $?
1
./busybox ls qwert; echo $?
0

Looks like the status logic is inverted.

    if (ENABLE_FEATURE_CLEAN_UP)
        dfree(dnp, nfiles);
    return (exit_failure == 0);

should be

    return (exit_failure == 1);
or
    return (exit_failure);


regards
  George