diff options
author | Denys Vlasenko <vda.linux@googlemail.com> | 2017-07-09 00:39:15 +0200 |
---|---|---|
committer | Denys Vlasenko <vda.linux@googlemail.com> | 2017-07-09 00:39:15 +0200 |
commit | 9de9c871bf44b931c7a1bb66d3134e2deb811f88 (patch) | |
tree | 68f7f95835b7e2c1c60825e2745d00f5aacfd92b /miscutils/devmem.c | |
parent | d18b2000967cddd0b84091d90a914aec58025310 (diff) | |
download | busybox-9de9c871bf44b931c7a1bb66d3134e2deb811f88.tar.gz |
shuf: fix random line selection. Closes 9971
"""
For example, given input file:
foo
bar
baz
after shuffling the input file, foo will never end up back on the first line.
This came to light when I ran into a use-case where someone was selecting
a random line from a file using shuf | head -n 1, and the results on busybox
were showing a statistical anomaly (as in, the first line would never ever
be picked) vs the same process running on environments that had gnu coreutils
installed.
On line https://git.busybox.net/busybox/tree/coreutils/shuf.c#n56 it uses
r %= i, which will result in 0 <= r < i, while the algorithm specifies
0 <= r <= i.
"""
Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
Diffstat (limited to 'miscutils/devmem.c')
0 files changed, 0 insertions, 0 deletions