IDEChannel: Fix wait_until_not_busy() (SerenityOS#7084)

The time_elapsed variable would count until milliseconds_timeout + 1, so a != comparision won't work.
cubiclove · May 13, 2021 · 28a6a9a · 28a6a9a
1 parent 8693c92
commit 28a6a9a
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/Kernel/Storage/IDEChannel.cpp b/Kernel/Storage/IDEChannel.cpp
@@ -269,7 +269,7 @@ bool IDEChannel::wait_until_not_busy(bool slave, size_t milliseconds_timeout)
  IO::delay(1000);
  time_elapsed++;
  }
- return time_elapsed != milliseconds_timeout;
+ return time_elapsed <= milliseconds_timeout;
 }
 
 bool IDEChannel::wait_until_not_busy(size_t milliseconds_timeout)
@@ -279,7 +279,7 @@ bool IDEChannel::wait_until_not_busy(size_t milliseconds_timeout)
  IO::delay(1000);
  time_elapsed++;
  }
- return time_elapsed != milliseconds_timeout;
+ return time_elapsed <= milliseconds_timeout;
 }
 
 String IDEChannel::channel_type_string() const