udf: Fix crash after seekdir
authorJan Kara <jack@suse.cz>
Thu, 4 Nov 2021 14:22:35 +0000 (15:22 +0100)
committerJan Kara <jack@suse.cz>
Tue, 9 Nov 2021 11:53:58 +0000 (12:53 +0100)
commita48fc69fe6588b48d878d69de223b91a386a7cb4
tree93f5770ca7067581d329e24259fa6eeee98ae455
parent6b75d88fa81b122cce37ebf17428a849ccd3d0f1
udf: Fix crash after seekdir

udf_readdir() didn't validate the directory position it should start
reading from. Thus when user uses lseek(2) on directory file descriptor
it can trick udf_readdir() into reading from a position in the middle of
directory entry which then upsets directory parsing code resulting in
errors or even possible kernel crashes. Similarly when the directory is
modified between two readdir calls, the directory position need not be
valid anymore.

Add code to validate current offset in the directory. This is actually
rather expensive for UDF as we need to read from the beginning of the
directory and parse all directory entries. This is because in UDF a
directory is just a stream of data containing directory entries and
since file names are fully under user's control we cannot depend on
detecting magic numbers and checksums in the header of directory entry
as a malicious attacker could fake them. We skip this step if we detect
that nothing changed since the last readdir call.

Reported-by: Nathan Wilson <nate@chickenbrittle.com>
CC: stable@vger.kernel.org
Signed-off-by: Jan Kara <jack@suse.cz>
fs/udf/dir.c
fs/udf/namei.c
fs/udf/super.c