On Mon, Dec 09, 2019 at 10:30:06AM +0100, Johannes Thumshirn wrote: > On 07/12/2019 23:18, Sebastian wrote: > > From: Sebastian Scherbel <sebastian.scherbel@xxxxxx> > > > > Regarding Bug 205003, points 1 and 2 > > This patch series moves two dereferences after the null check to avoid > > a possible null pointer dereference. > > > > Sebastian Scherbel (2): > > btrfs: Move dereference behind null check in check integrity > > btrfs: Move dereference behind null check in check volumes > > > > fs/btrfs/check-integrity.c | 4 +++- > > fs/btrfs/volumes.c | 4 +++- > > 2 files changed, 6 insertions(+), 2 deletions(-) > > > > I've already submitted a series addressing these and David merged it: > https://lore.kernel.org/linux-btrfs/20191205131959.19184-1-jth@xxxxxxxxxx/ Yes, that's been in misc-next since ~friday, I'm not sure when I pushed the branch. Sebastian, thanks for the patches, this sometimes happens that the work is duplicated. Johannes removed the BUG_ON and WARN_ON completely though your change is also correct assuming that the two are not redundant.
