Whamcloud - gitweb
filefrag: handle invalid st_dev and blksize cases
authorLuis Henriques <lhenriques@suse.de>
Wed, 28 Oct 2020 15:55:50 +0000 (15:55 +0000)
committerTheodore Ts'o <tytso@mit.edu>
Wed, 20 Jan 2021 04:43:29 +0000 (23:43 -0500)
commitc57857a514d46535bf81f4fd3d0635ffc9df7490
treed73e3bd3f0f0450bd2868e9690855231b3f1074f
parent7616fd6a599e44c5700c2c3a2e08979c6c5c747e
filefrag: handle invalid st_dev and blksize cases

It is possible to crash filefrag with a "Floating point exception" in
two different scenarios:

1. When fstat() returns a device ID set to 0
2. When FIGETBSZ ioctl returns a blocksize of 0

In both scenarios a divide-by-zero will occur in frag_report() because
variable blksize will be set to zero.

I've managed to trigger this crash with an old CephFS kernel client,
using xfstest generic/519.  The first scenario has been fixed by kernel
commit 75c9627efb72 ("ceph: map snapid to anonymous bdev ID").  The
second scenario is also fixed with commit 8f97d1e99149 ("vfs: fix
FIGETBSZ ioctl on an overlayfs file").

However, it is desirable to handle these two scenarios gracefully by
checking these conditions explicitly.

Signed-off-by: Luis Henriques <lhenriques@suse.de>
Signed-off-by: Theodore Ts'o <tytso@mit.edu>
misc/filefrag.c