[lvc-project] [PATCH 6.1 1/1] fs/ntfs3: Fix shift-out-of-bounds in ntfs_fill_super

Roman Smirnov r.smirnov at omp.ru
Mon May 6 10:19:42 MSK 2024


On Wed, 2024-04-24 at 20:41 +0300, Fedor Pchelkin wrote:
> Добрый вечер, Роман! Спасибо за подготовленный бэкпорт-патч!
> 
> Маленький комментарий ниже.
> 
> On Wed, 24. Apr 13:11, Roman Smirnov wrote:
> > From: Konstantin Komarov <almaz.alexandrovich at paragon-software.com>
> > 
> > commit 91a4b1ee78cb100b19b70f077c247f211110348f upstream. 
> > 
> > Reported-by: syzbot+478c1bf0e6bf4a8f3a04 at syzkaller.appspotmail.com
> > Signed-off-by: Konstantin Komarov <almaz.alexandrovich at paragon-software.com>
> > [Roman: added string hint, replaced cluster_bits with sbi->cluster_bits, 
> > removed redundant checks near "Check MFT record size" and "Check index record 
> > size".]
> > Signed-off-by: Roman Smirnov <r.smirnov at omp.ru>
> > ---
> 
> hint был добавлен коммитом 6a4cd3ea7d77 ("fs/ntfs3: Alternative boot if
> primary boot is corrupted") [1] и имеет ценность только после им внесённого
> варианта альтернативной загрузки. В более ранние стабильные ветки, где нет
> указанного коммита, не думаю, что hint стоит внедрять.
> 
> [1]:
> https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=6a4cd3ea7d77
> 

> 
Добрый день, Фёдор. Спасибо за ревью.

Как думаете, в связи с долгой реакцией мейнтейнера есть ли смысл подготовить вторую версию
патча без этой строки?

[...]


More information about the lvc-project mailing list