Skip to content

nbd: clamp the timeout before converting it to jiffies - #1149

Open
blktests-ci[bot] wants to merge 1 commit into
linus-master_basefrom
series/1147249=>linus-master
Open

nbd: clamp the timeout before converting it to jiffies#1149
blktests-ci[bot] wants to merge 1 commit into
linus-master_basefrom
series/1147249=>linus-master

Conversation

@blktests-ci

@blktests-ci blktests-ci Bot commented Aug 17, 2026

Copy link
Copy Markdown

Pull request for series with
subject: nbd: clamp the timeout before converting it to jiffies
version: 1
url: https://patchwork.kernel.org/project/linux-block/list/?series=1147249

@blktests-ci

blktests-ci Bot commented Aug 17, 2026

Copy link
Copy Markdown
Author

Upstream branch: 8d3ae59
series: https://patchwork.kernel.org/project/linux-block/list/?series=1147249
version: 1

@blktests-ci

blktests-ci Bot commented Aug 18, 2026

Copy link
Copy Markdown
Author

Upstream branch: 8d3ae59
series: https://patchwork.kernel.org/project/linux-block/list/?series=1147249
version: 1

@blktests-ci
blktests-ci Bot force-pushed the series/1147249=>linus-master branch from d227a60 to c8a794b Compare August 18, 2026 07:21
@blktests-ci
blktests-ci Bot force-pushed the linus-master_base branch from 60442a3 to 3df366e Compare August 19, 2026 05:25
nbd_set_cmd_timeout() stores timeout * HZ into tag_set.timeout, an
unsigned int, and passes the same product to blk_queue_rq_timeout()
without any range check.  A large timeout (2^32 seconds, say)
truncates to 0 instead of being a very long timeout.

Clamp the timeout to UINT_MAX / HZ seconds first.

Setting NBD_SET_TIMEOUT to 2^32 left the debugfs timeout at 0 before
this change and at 4294967000 after it.

Fixes: 55313e9 ("nbd: add set cmd timeout helper")
Signed-off-by: Tao Cui <cuitao@kylinos.cn>
Reviewed-by: Yu Kuai <yukuai@fygo.io>
@blktests-ci

blktests-ci Bot commented Aug 19, 2026

Copy link
Copy Markdown
Author

Upstream branch: bd5f485
series: https://patchwork.kernel.org/project/linux-block/list/?series=1147249
version: 1

@blktests-ci
blktests-ci Bot force-pushed the series/1147249=>linus-master branch from c8a794b to 33bd540 Compare August 19, 2026 05:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants