diff options
| author | cao <[email protected]> | 2016-11-09 06:28:17 +0800 |
|---|---|---|
| committer | Brian Behlendorf <[email protected]> | 2016-11-08 14:28:17 -0800 |
| commit | a36cc8d2428259f5cfefb653bff39ca7afc66518 (patch) | |
| tree | c76a5c3dda816ad8e2194bfbb72cdda69bc5e1b9 /module/zfs/zfs_vnops.c | |
| parent | ffb78320c8040e10e754f1358476beb04c943510 (diff) | |
Fix coverity defects: CID 147626, 147628
CID 147626: Type:Dereference before null check
CID 147628: Type:Dereference before null check
Reviewed-by: Brian Behlendorf <[email protected]
Reviewed-by: Chunwei Chen <[email protected]>
Signed-off-by: cao.xuewen <[email protected]>
Closes #5304
Diffstat (limited to 'module/zfs/zfs_vnops.c')
| -rw-r--r-- | module/zfs/zfs_vnops.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/module/zfs/zfs_vnops.c b/module/zfs/zfs_vnops.c index 6b5cda958..4a0f20166 100644 --- a/module/zfs/zfs_vnops.c +++ b/module/zfs/zfs_vnops.c @@ -4256,8 +4256,7 @@ zfs_putpage(struct inode *ip, struct page *pp, struct writeback_control *wbc) * writepages() normally handles the entire commit for * performance reasons. */ - if (zsb->z_log != NULL) - zil_commit(zsb->z_log, zp->z_id); + zil_commit(zsb->z_log, zp->z_id); } ZFS_EXIT(zsb); |