Message ID | 20191207221818.3641-1-sebastian.scherbel@fau.de (mailing list archive) |
---|---|
Headers | show |
Series | btrfs: Move dereference behind null checks | expand |
On 07/12/2019 23:18, Sebastian wrote: > From: Sebastian Scherbel <sebastian.scherbel@fau.de> > > 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(-) > Hi, I've already submitted a series addressing these and David merged it: https://lore.kernel.org/linux-btrfs/20191205131959.19184-1-jth@kernel.org/
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@fau.de> > > > > 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@kernel.org/ 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.
From: Sebastian Scherbel <sebastian.scherbel@fau.de> 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(-)