diff options
| author | Marcel Huber <[email protected]> | 2016-05-11 21:23:42 +0200 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2016-05-17 11:03:11 -0700 |
| commit | 2587cd8f93ec2ec774e49aafbf536b918f77cff2 (patch) | |
| tree | 1058870475566adc1c02ab3fdd9a773cd663a150 /module | |
| parent | ebecfcd6991bebe71511cb8fd409112798f203b2 (diff) | |
Fixes subtle bug in zio_handle_io_delay()
Fixed bug introduced in commit #c35b1882. Hinted by gcc:
zio_inject.c: In function ‘zio_handle_io_delay’:
zio_inject.c:382:3: warning: this ‘if’ clause does not guard... [-Wmisleading-indentation]
if (handler->zi_record.zi_freq != 0 &&
^~
zio_inject.c:384:4: note: ...this statement, but the latter is misleadingly indented as if it is guarded by the ‘if’
continue;
^~~~~~~~
Signed-off-by: Marcel Huber <[email protected]>
Signed-off-by: Chunwei Chen <[email protected]>
Signed-off-by: Brian Behlendorf <[email protected]>
Closes #4632
Diffstat (limited to 'module')
| -rw-r--r-- | module/zfs/zio_inject.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/module/zfs/zio_inject.c b/module/zfs/zio_inject.c index a6eca8715..1458be477 100644 --- a/module/zfs/zio_inject.c +++ b/module/zfs/zio_inject.c @@ -380,8 +380,9 @@ zio_handle_io_delay(zio_t *zio) continue; if (handler->zi_record.zi_freq != 0 && - spa_get_random(100) >= handler->zi_record.zi_freq); + spa_get_random(100) >= handler->zi_record.zi_freq) { continue; + } if (vd->vdev_guid == handler->zi_record.zi_guid) { seconds = handler->zi_record.zi_timer; |