diff options
| author | Denis Vlasenko <vda.linux@googlemail.com> | 2007-09-28 13:41:41 +0000 |
|---|---|---|
| committer | Denis Vlasenko <vda.linux@googlemail.com> | 2007-09-28 13:41:41 +0000 |
| commit | 0afdfdf649005b50c1fffd663537b405129ae4d1 (patch) | |
| tree | 0b39935bb8aa9d5aacc570879963643f4e288227 /miscutils/hdparm.c | |
| parent | 920c52a5c80969ab0c831e56cb26971e83942f0a (diff) | |
| download | busybox-0afdfdf649005b50c1fffd663537b405129ae4d1.tar.gz | |
hdparm: paranoia-induced fix against theoretical div-by-0 case
Diffstat (limited to 'miscutils/hdparm.c')
| -rw-r--r-- | miscutils/hdparm.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/miscutils/hdparm.c b/miscutils/hdparm.c index 906dd7a0c..cb6db78cd 100644 --- a/miscutils/hdparm.c +++ b/miscutils/hdparm.c @@ -1378,8 +1378,8 @@ static void print_timing(unsigned m, unsigned elapsed_us) printf("%5u MB in %u.%02u seconds = %u kB/s\n", m, sec, hs, - /* + 1 prevents div-by-0 */ - (unsigned) ((unsigned long long)m * (1024 * 1000000) / (elapsed_us + 1)) + /* "| 1" prevents div-by-0 */ + (unsigned) ((unsigned long long)m * (1024 * 1000000) / (elapsed_us | 1)) // ~= (m * 1024) / (elapsed_us / 1000000) // = kb / elapsed_sec ); |